From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.pbcl.net ([88.198.119.4] helo=hetzner.pbcl.net) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1TGoZa-00029B-6y for openembedded-core@lists.openembedded.org; Wed, 26 Sep 2012 12:10:30 +0200 Received: from elite.brightsigndigital.co.uk ([81.142.160.137] helo=[172.30.1.145]) by hetzner.pbcl.net with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.72) (envelope-from ) id 1TGoNC-0006LR-66; Wed, 26 Sep 2012 11:57:42 +0200 From: Phil Blundell To: Martin Jansa Date: Wed, 26 Sep 2012 10:57:41 +0100 In-Reply-To: <20120926095459.GE3313@jama.jama.net> References: <1348652719.31293.90.camel@phil-desktop> <20120926095459.GE3313@jama.jama.net> X-Mailer: Evolution 3.0.2- Message-ID: <1348653462.31293.92.camel@phil-desktop> Mime-Version: 1.0 Cc: OE-core Subject: Re: [PATCH] gcc-configure-cross: factor out --enable-threads argument into ${GCCTHREADS} X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 26 Sep 2012 10:10:30 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit 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.