On Mon, 9 Jan 2023 at 18:16, <[email protected]> wrote: > > From: Marc-André Lureau <[email protected]> > > Let's try to reduce our Perl usage during config/build-time. > > Note: this patch might be dropped if "configure: remove > backwards-compatibility code" is merged earlier. > > Signed-off-by: Marc-André Lureau <[email protected]> > --- > configure | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/configure b/configure > index 2281892657..e45acde404 100755 > --- a/configure > +++ b/configure > @@ -2571,11 +2571,9 @@ else > if test -f meson-private/cmd_line.txt; then > # Adjust old command line options whose type was changed > # Avoids having to use "setup --wipe" when Meson is upgraded > - perl -i -ne ' > - s/^gettext = true$/gettext = auto/; > - s/^gettext = false$/gettext = disabled/; > - /^b_staticpic/ && next; > - print;' meson-private/cmd_line.txt > + sed -i.bak 's/^gettext = true/gettext = auto/g' -e \ > + 's/^gettext = false/gettext = disabled/g' -e \ > + '/^b_staticpic/d' meson-private/cmd_line.txt
This doesn't work on macos sed, because POSIX sed allows sed command file or sed -e command -e command file but not sed command -e command file The fix is easy, prefix the first command with '-e'. -i is not POSIX sed, but luckily the BSD sed implements it. The two gettext matches have lost the trailing '$' in the regex. thanks -- PMM
