Re: Bug in gnulib-tools prevents bison from bootstrapping

[Ralf Wildenhues]

Hello Fernando,

* Fernando Ferreira wrote on Fri, Jun 22, 2007 at 01:42:25PM CEST:
[...]
>  # You need to invoke gt_JAVACOMP yourself, possibly with arguments.
>  AC_CONFIG_FILES( :build-aux/ )
>  # You need to invoke gt_JAVAEXEC yourself, possibly with arguments.
>  AC_CONFIG_FILES( :build-aux/ )
>
> These seemed to be the offending lines, but I would then discover, when 
> opening the file on gedit, that the supposed whitespaces before and after 
> ':build-aux/' were not whitespaces, but the ASCII characters 1 and 2 ('\1' 
> and '\2', C-like notation). To discover how did them got there was my next 
> quest.

Thank you for the nice and complete bug report and analysis.  Please try
this patch.  OK to install?

Cheers,
Ralf

        * gnulib-tool (func_add_or_update, func_create_testdir): Do not
        simplify `sed_replace_build_aux' scripts, they are portable but
        echoing them with `echo' is not.
        Report and analysis by Fernando Ferreira <address@hidden>.

Index: gnulib-tool
===================================================================
RCS file: /cvsroot/gnulib/gnulib/gnulib-tool,v
retrieving revision 1.233
diff -u -r1.233 gnulib-tool
--- gnulib-tool 28 May 2007 15:46:55 -0000      1.233
+++ gnulib-tool 22 Jun 2007 12:43:48 -0000
@@ -2174,7 +2184,6 @@
           
s|AC_CONFIG_FILES(\(.*\):build-aux/\(.*\))|AC_CONFIG_FILES(\1:'"$auxdir"'/\2)|
           ba
         }'
-      sed_replace_build_aux=`echo "$sed_replace_build_aux" | sed -e 1d -e 's/^ 
*//'`
     else
       sed_replace_build_aux=
     fi
@@ -2623,7 +2632,6 @@
         
s|AC_CONFIG_FILES(\(.*\):build-aux/\(.*\))|AC_CONFIG_FILES(\1:../'"$auxdir"'/\2)|
         ba
        }'
-     sed_replace_build_aux=`echo "$sed_replace_build_aux" | sed -e 1d -e 's/^ 
*//'`
      # We don't have explicit ordering constraints between the various
      # autoconf snippets. It's cleanest to put those of the library before
      # those of the tests.
@@ -2744,7 +2752,6 @@
          
s|AC_CONFIG_FILES(\(.*\):build-aux/\(.*\))|AC_CONFIG_FILES(\1:'"$auxdir"'/\2)|
          ba
        }'
-     sed_replace_build_aux=`echo "$sed_replace_build_aux" | sed -e 1d -e 's/^ 
*//'`
    else
      sed_replace_build_aux=
    fi