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 <[EMAIL PROTECTED]>.

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!


_______________________________________________
help-bison@gnu.org http://lists.gnu.org/mailman/listinfo/help-bison

Reply via email to