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