PATCH RFA: libstdc++ lives in src/.libs

Ping for build maintainers.

Ian

On Thu, Oct 7, 2010 at 4:54 PM, Ian Lance Taylor <iant@google.com> wrote:
> In the top level Makefile.def file, the lib_path of a target module
> gives the directory where the library can be found at run time.  This is
> used to set LD_LIBRARY_PATH (or equivalent) when bootstrapping.  The
> value for libstdc++-v3 is .libs, but actually the library is built in
> src/.libs.  This patch corrects this error.
>
> Bootstrapped on x86_64-unknown-linux-gnu.  Passed the libstdc++-v3 and
> g++ testsuites.
>
> OK for mainline?
>
> Ian
>
>
> 2010-10-07  Ian Lance Taylor  <iant@google.com>
>
>        * Makefile.def (target_modules): Set lib_path to src/.libs for
>        libstdc++-v3 module.
>        * Makefile.tpl: Fix typo in TARGET_LIB_PATH comment.
>        * Makefile.in: Rebuild.
>
>
>

Hello Ian,

* Ian Lance Taylor wrote on Mon, Oct 18, 2010 at 10:08:21PM CEST:
> Ping for build maintainers.

I cannot approve this but I think this can be considered as obvious.
One question is why it didn't show up earlier.

Cheers,
Ralf

> On Thu, Oct 7, 2010 at 4:54 PM, Ian Lance Taylor <iant@google.com> wrote:
> >        * Makefile.def (target_modules): Set lib_path to src/.libs for
> >        libstdc++-v3 module.
> >        * Makefile.tpl: Fix typo in TARGET_LIB_PATH comment.
> >        * Makefile.in: Rebuild.

> --- Makefile.def	(revision 165058)
> +++ Makefile.def	(working copy)
> @@ -149,7 +149,7 @@ host_modules= { module= lto-plugin; boot
>  
>  target_modules = { module= libstdc++-v3;
>  		   bootstrap=true;
> -		   lib_path=.libs;
> +		   lib_path=src/.libs;
>  		   raw_cxx=true; };
>  target_modules = { module= libmudflap; lib_path=.libs; };
>  target_modules = { module= libssp; lib_path=.libs; };

On 10/20/2010 07:24 AM, Ralf Wildenhues wrote:
> Hello Ian,
>
> * Ian Lance Taylor wrote on Mon, Oct 18, 2010 at 10:08:21PM CEST:
>> Ping for build maintainers.
>
> I cannot approve this but I think this can be considered as obvious.

Indeed.

Paolo

Ralf Wildenhues <Ralf.Wildenhues@gmx.de> writes:

> * Ian Lance Taylor wrote on Mon, Oct 18, 2010 at 10:08:21PM CEST:
>> Ping for build maintainers.
>
> I cannot approve this but I think this can be considered as obvious.
> One question is why it didn't show up earlier.

Thanks.  Committed.  I think it didn't show up earlier simply because
there are no gcc programs that are written in C++ and run during the
build.  The testsuite uses other mechanisms to set LD_LIBRARY_PATH.

Ian