[FFmpeg-devel] [PATCH] configure: use -r, not -E, for sed

[Lauri Kasanen]

On Fri, 16 Nov 2018 22:36:16 +0100
Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2018-11-15 15:00 GMT+01:00, Lauri Kasanen <cand at gmx.com>:
> > Old versions of sed do not support the -E option.
> 
> > -        VERSION_SCRIPT_POSTPROCESS_CMD='tr " " "\n" | sed -n
> > /global:/,/local:/p | grep ";" | tr ";" "\n" | sed -E "s/(.+)/_\1/g" | sed
> > -E "s/(.+[^*])$$$$/\1*/"'
> > +        VERSION_SCRIPT_POSTPROCESS_CMD='tr " " "\n" | sed -n
> > /global:/,/local:/p | grep ";" | tr ";" "\n" | sed -r "s/(.+)/_\1/g" | sed
> > -r "s/(.+[^*])$$$$/\1*/"'
> 
> Could you try to replace the current command with one that
> neither needs "-E" nor "-r"?
> Your suggestions fixes antique Linux systems but not current
> non-Linux Posix systems (and contradicts the documentation).

Regexes tend to be write-only. Not sure I can parse what that tries to
do, to rewrite it in basic RE that posix sed supports.

What do you mean by contradicts docs?

- Lauri