[OE-core] [PATCH] gcc-configure-cross: factor out --enable-threads argument into ${GCCTHREADS}

Phil Blundell philb at gnu.org
Wed Sep 26 09:57:41 UTC 2012


On Wed, 2012-09-26 at 11:54 +0200, Martin Jansa wrote:
> On Wed, Sep 26, 2012 at 10:45:18AM +0100, Phil Blundell wrote:
> > diff --git a/meta/recipes-devtools/gcc/gcc-configure-common.inc b/meta/recipes-devtools/gcc/gcc-configure-common.inc
> > index 908ad3e..504bfaa 100644
> > --- a/meta/recipes-devtools/gcc/gcc-configure-common.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-configure-common.inc
> > @@ -26,12 +26,13 @@ EXTRA_OECONF_INITIAL ?= ""
> >  EXTRA_OECONF_INTERMEDIATE ?= ""
> >  
> >  GCCMULTILIB = "--disable-multilib"
> > +GCCTHREADS = "posix"
> >  
> >  EXTRA_OECONF = "${@['--enable-clocale=generic', ''][d.getVar('USE_NLS', True) != 'no']} \
> >                  --with-gnu-ld \
> >                  --enable-shared \
> >                  --enable-languages=${LANGUAGES} \
> > -                --enable-threads=posix \
> > +                --enable-threads=${GCCTHREADS} \
> >                  ${GCCMULTILIB} \
> >                  --enable-c99 \
> >                  --enable-long-long \
> > -- 
> > 1.7.10.4
> 
> Missing PR bump

Does it really need one?  It doesn't have any impact on the value that
EXTRA_OECONF ends up with, and bumping the PRs on all the various .bb
files would make the patch about ten times the size.  I can do that if
you insist but it seems like a bit of a waste of time.

p.






More information about the Openembedded-core mailing list