On Thu, Feb 24, 2022 at 11:46 AM Glen Choo <chooglen@xxxxxxxxxx> wrote:
> Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes:
> > On Thu, Feb 24 2022, Glen Choo wrote:
> >> + sed -E 's/[0-9a-f]+\.\./OLD_HEAD\.\./' $ACTUAL_ERR >actual.err.cmp &&
> >
> > I think this is unportable per check-non-portable-shell.pl:
> >
> > /\bsed\s+-[^efn]\s+/ and err 'sed option not portable (use only -n, -e, -f)';
>
> I used that because I wanted +, but I found what I needed from the sed
> manpage i.e. that + is equivalent to \{1,\}).
This isn't necessarily going to be portable either for older sed
implementations. Most portable would be:
[0-9a-f][0-9a-f]*
(Whether or not we need to worry about those older sed impmenetations
is a different question...)
