Re: sed: -e expression #6, char 53: unknown option to `s'

[Tim Ruehsen]

Am Samstag, den 03.02.2018, 19:56 +0100 schrieb Bruno Haible:
> Tim Rühsen wrote:
> > sed: -e expression #6, char 53: unknown option to `s'
> 
> The 6th sed expression and following here are these:
> 
> >       -e 's|@''GNULIB_PTHREAD_SIGMASK''@|(IN_LIBIDN_GNULIB_TESTS ||
> > IN_LIBIDN_GNULIB_TESTS)|g' \
> >       -e 's|@''GNULIB_RAISE''@|(IN_LIBIDN_GNULIB_TESTS ||
> > IN_LIBIDN_GNULIB_TESTS)|g' \
> >       -e 's/@''GNULIB_SIGNAL_H_SIGPIPE''@/0/g' \
> >       -e 's/@''GNULIB_SIGPROCMASK''@/(IN_LIBIDN_GNULIB_TESTS ||
> > IN_LIBIDN_GNULIB_TESTS)/g' \
> >       -e 's/@''GNULIB_SIGACTION''@/IN_LIBIDN_GNULIB_TESTS/g' \
> 
> The template for this comes from modules/signal-h. But the values
> come from Automake variables in the Makefile. It looks like gnulib
> has substituted values that contain the C operator ||. The syntax
> error comes from the fact that 2 out of 5 of these sed expressions
> use the '|' character. They should probably use the '/' character.
> 
> Please try this patch:
> 
> diff --git a/modules/signal-h b/modules/signal-h
> index fe9af81..810e2bf 100644
> --- a/modules/signal-h
> +++ b/modules/signal-h
> @@ -28,8 +28,8 @@ signal.h: signal.in.h $(top_builddir)/config.status
> $(CXXDEFS_H) $(ARG_NONNULL_H
>             -e 's|@''PRAGMA_SYSTEM_HEADER''@|@PRAGMA_SYSTEM_HEADER
> @|g' \
>             -e 's|@''PRAGMA_COLUMNS''@|@PRAGMA_COLUMNS@|g' \
>             -e 's|@''NEXT_SIGNAL_H''@|$(NEXT_SIGNAL_H)|g' \
> -           -e
> 's|@''GNULIB_PTHREAD_SIGMASK''@|$(GNULIB_PTHREAD_SIGMASK)|g' \
> -           -e 's|@''GNULIB_RAISE''@|$(GNULIB_RAISE)|g' \
> +           -e
> 's/@''GNULIB_PTHREAD_SIGMASK''@/$(GNULIB_PTHREAD_SIGMASK)/g' \
> +           -e 's/@''GNULIB_RAISE''@/$(GNULIB_RAISE)/g' \
>             -e
> 's/@''GNULIB_SIGNAL_H_SIGPIPE''@/$(GNULIB_SIGNAL_H_SIGPIPE)/g' \
>             -e
> 's/@''GNULIB_SIGPROCMASK''@/$(GNULIB_SIGPROCMASK)/g' \
>             -e 's/@''GNULIB_SIGACTION''@/$(GNULIB_SIGACTION)/g' \
> 

Hi Bruno,

thanks, with this patch the build completes !

Regards, Tim