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

[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' \