Fix the fix. (was Re: Fix quoting of program-transform-name in toplevel configure)
Geoffrey Keating
geoffk@apple.com
Sat Aug 23 08:26:00 GMT 2003
On Friday, August 22, 2003, at 09:17 PM, Phil Edwards wrote:
> On Fri, Aug 22, 2003 at 10:17:56PM -0400, Phil Edwards wrote:
>> On Thu, Aug 21, 2003 at 03:09:15PM +0200, Gerald Pfeifer wrote:
>>>
>>> Even that different shell we recommend, /bin/ksh, fails to bootstrap
>>> current CVS due to the problem above. But I understood that in this
>>> our sources can easily work around the problem, right?
>>
>> Yes. The fix is trivial.
>
> So I did it.
This is OK; if you haven't committed it already, please do!
> 2003-08-23 Phil Edwards <pme@gcc.gnu.org>
>
> * configure.in: Use newline instead of semicolon when assuming
> shell arguments in a for loop.
> * configure: Regenerated.
>
>
> Index: configure.in
> ===================================================================
> RCS file: /cvs/gcc/gcc/configure.in,v
> retrieving revision 1.252
> diff -u -3 -p -r1.252 configure.in
> --- configure.in 21 Aug 2003 02:19:20 -0000 1.252
> +++ configure.in 23 Aug 2003 04:15:04 -0000
> @@ -74,7 +74,8 @@ cat <<\EOF_SED > conftestsed
> s,\\,\\\\,g; s,\$,$$,g
> EOF_SED
> tmp="'$progname'"
> -for ac_arg ; do
> +for ac_arg
> +do
> tmp="$tmp '"`echo "$ac_arg" | sed -fconftestsed`
> done
> rm -f conftestsed
More information about the Gcc-patches
mailing list