Patchwork gcc-configure-common: Use build time sysroot for target gcc

login
register
mail settings
Submitter Bogdan Marinescu
Date June 13, 2012, 8:26 a.m.
Message ID <1339575975-22078-1-git-send-email-bogdan.a.marinescu@intel.com>
Download mbox | patch
Permalink /patch/29779/
State New
Headers show

Comments

Bogdan Marinescu - June 13, 2012, 8:26 a.m.
From: Khem Raj <raj.khem@gmail.com>

Otherwise it assumes / as sysroot on build host and starts
searching for directories in there.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
--
cgit v0.9.0.2-39-g756e
Saul Wold - June 14, 2012, 5:14 p.m.
On 06/13/2012 01:26 AM, Bogdan Marinescu wrote:
> From: Khem Raj<raj.khem@gmail.com>
>
> Otherwise it assumes / as sysroot on build host and starts
> searching for directories in there.
>
> Signed-off-by: Khem Raj<raj.khem@gmail.com>
> ---
> diff --git a/meta/recipes-devtools/gcc/gcc-configure-target.inc b/meta/recipes-devtools/gcc/gcc-configure-target.inc
> index 1ee7504..5bca2b7 100644
> --- a/meta/recipes-devtools/gcc/gcc-configure-target.inc
> +++ b/meta/recipes-devtools/gcc/gcc-configure-target.inc
> @@ -1,4 +1,6 @@
>   require gcc-configure-common.inc
>
>   EXTRA_OECONF_PATHS = " \
> +    --with-sysroot \

I can't seem to find the reply to this, but RP added a "=" to the
end of this and that seems to have re-introduced the problem we were 
attempting to fix.

Apparently this is a bug in the configure script then also.

Sau!

> +    --with-build-sysroot=${STAGING_DIR_TARGET} \
>       --with-gxx-include-dir=${includedir}/c++/"
> --
> cgit v0.9.0.2-39-g756e
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
Richard Purdie - June 14, 2012, 6:51 p.m.
On Thu, 2012-06-14 at 10:14 -0700, Saul Wold wrote:
> On 06/13/2012 01:26 AM, Bogdan Marinescu wrote:
> > From: Khem Raj<raj.khem@gmail.com>
> >
> > Otherwise it assumes / as sysroot on build host and starts
> > searching for directories in there.
> >
> > Signed-off-by: Khem Raj<raj.khem@gmail.com>
> > ---
> > diff --git a/meta/recipes-devtools/gcc/gcc-configure-target.inc b/meta/recipes-devtools/gcc/gcc-configure-target.inc
> > index 1ee7504..5bca2b7 100644
> > --- a/meta/recipes-devtools/gcc/gcc-configure-target.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-configure-target.inc
> > @@ -1,4 +1,6 @@
> >   require gcc-configure-common.inc
> >
> >   EXTRA_OECONF_PATHS = " \
> > +    --with-sysroot \
> 
> I can't seem to find the reply to this, but RP added a "=" to the
> end of this and that seems to have re-introduced the problem we were 
> attempting to fix.
> 
> Apparently this is a bug in the configure script then also.

I screwed up, I've fixed it. Sorry about that.

Cheers,

Richard

Patch

diff --git a/meta/recipes-devtools/gcc/gcc-configure-target.inc b/meta/recipes-devtools/gcc/gcc-configure-target.inc
index 1ee7504..5bca2b7 100644
--- a/meta/recipes-devtools/gcc/gcc-configure-target.inc
+++ b/meta/recipes-devtools/gcc/gcc-configure-target.inc
@@ -1,4 +1,6 @@ 
 require gcc-configure-common.inc
 
 EXTRA_OECONF_PATHS = " \
+    --with-sysroot \
+    --with-build-sysroot=${STAGING_DIR_TARGET} \
     --with-gxx-include-dir=${includedir}/c++/"