Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > Also as another aside (but your patch is fine as it is), my suggestion > used Perl+Perl RX but you switched it to sed+BRE. Do we want to avoid > "sed -E"? I wondered that for something else the other day, we have > this: > > t/check-non-portable-shell.pl: /\bsed\s+-[^efn]\s+/ and err 'sed option not portable (use only -n, -e, -f)'; > > So maybe it means "nothing but -nef, or maybe "don't use -efn". The ERE > (-E and -r) options aren't mentioned, and a naïve log search of of "sed > -E" and "sed -r" in t/ returns nothing. Correct. We currently do not use "sed -E", and the script says "Let's not use it; it's not even in POSIX" for things other than 'n', 'e', and 'f'. https://pubs.opengroup.org/onlinepubs/9699919799/utilities/sed.html Thanks.
