[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Bug in gnulib-tools prevents bison from bootstrapping
From: |
Ralf Wildenhues |
Subject: |
Re: Bug in gnulib-tools prevents bison from bootstrapping |
Date: |
Fri, 22 Jun 2007 14:47:32 +0200 |
User-agent: |
Mutt/1.5.15 (2007-05-20) |
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
Re: Bug in gnulib-tools prevents bison from bootstrapping,
Ralf Wildenhues <=
Re: Bug in gnulib-tools prevents bison from bootstrapping, Bruno Haible, 2007/06/22