bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Bug in gnulib-tools prevents bison from bootstrapping


From: Fernando Ferreira
Subject: Re: Bug in gnulib-tools prevents bison from bootstrapping
Date: Fri, 22 Jun 2007 14:26:33 +0100
User-agent: Thunderbird 2.0.0.0 (X11/20070525)

Ralf Wildenhues wrote:
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


The patch worked, and everything installed flawless. Thanks a lot!




reply via email to

[Prev in Thread] Current Thread [Next in Thread]