This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: r150960 changed ltmain.sh and broke the build
- From: Dave Korn <dave dot korn dot cygwin at googlemail dot com>
- To: Paolo Bonzini <bonzini at gnu dot org>
- Cc: Dave Korn <dave dot korn dot cygwin at googlemail dot com>, NightStrike <nightstrike at gmail dot com>, gcc at gcc dot gnu dot org, davek at gcc dot gnu dot org, Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>, Mook - <mook dot gcc at gmail dot com>, Kai Tietz <ktietz70 at googlemail dot com>
- Date: Wed, 26 Aug 2009 11:38:50 +0100
- Subject: Re: r150960 changed ltmain.sh and broke the build
- References: <b609cb3b0908260211q4ca347efw94f4e420d1ac5efc@mail.gmail.com> <4A950CFE.7060709@gmail.com> <4A950C2A.8040204@gnu.org>
Paolo Bonzini wrote:
>> Argh. Very sorry for the breakage, I'll get straight onto it. What
>> versions of the mingw+msys tools are you using? Can you please send
>> me a copy
>> of the generated libtool script, and the output you get when
>> re-running the
>> failing libtool command manually after adding the "--debug"
>> (non-modal) option?
>
> This is the problem:
>
> removedotparts="s,/\(\./\)\{1\,\},/,g;s,/\.$,/,"
>
> \, should be , (there is another occurrence).
Oops. Looks like there are different versions of SED that have different
requirements, because on cygwin SED 4.1.5, you *have* to quote the comma in
the curly-braces {min,max} repetitions specifier or it is interpreted as a
separator.
> $ sed -e 's,/\(\./\)\{1\,\},/,g;s,/\.$,/,'
> ./foo/bar/./baz
> ./foo/bar/baz
>
> $ sed -e 's,/\(\./\)\{1,\},/,g;s,/\.$,/,'
> sed: -e expression #1, char 18: unknown option to `s'
>
> $
Maybe the best thing would be to change it to use a different separator?
Ralf, have we discovered a new item for the autoconf man page "portable shell"
chapter?
> Also please single-quote the commands.
Ok, why?
cheers,
DaveK