From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 93-97-173-237.zone5.bethere.co.uk ([93.97.173.237] helo=tim.rpsys.net) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1T5Ipt-0003P7-G5 for openembedded-core@lists.openembedded.org; Sat, 25 Aug 2012 18:03:45 +0200 Received: from localhost (localhost [127.0.0.1]) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q7PFpX5f008332; Sat, 25 Aug 2012 16:51:33 +0100 Received: from tim.rpsys.net ([127.0.0.1]) by localhost (tim.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 08209-02; Sat, 25 Aug 2012 16:51:29 +0100 (BST) Received: from [192.168.3.10] ([192.168.3.10]) (authenticated bits=0) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q7PFpOoZ008326 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO); Sat, 25 Aug 2012 16:51:26 +0100 Message-ID: <1345909886.14369.115.camel@ted> From: Richard Purdie To: Koen Kooi Date: Sat, 25 Aug 2012 16:51:26 +0100 In-Reply-To: <1A25F2D9-E229-44B5-A7D4-AA363EB1EA57@dominion.thruhere.net> References: <65d8351bbcb9fb6012c296222795f3c938dc5624.1345867875.git.tom.zanussi@intel.com> <3BDCB1BC-7393-4B42-BE04-3BE9A8C90123@dominion.thruhere.net> <1345895796.14369.106.camel@ted> <1A25F2D9-E229-44B5-A7D4-AA363EB1EA57@dominion.thruhere.net> X-Mailer: Evolution 3.2.3-0ubuntu6 Mime-Version: 1.0 X-Virus-Scanned: amavisd-new at rpsys.net Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCH 1/1] ia32-base.inc: new include file 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: Sat, 25 Aug 2012 16:03:45 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Sat, 2012-08-25 at 16:17 +0200, Koen Kooi wrote: > Op 25 aug. 2012, om 13:56 heeft Richard Purdie het volgende geschreven: > > > On Sat, 2012-08-25 at 09:45 +0200, Koen Kooi wrote: > >> Op 25 aug. 2012, om 06:34 heeft tom.zanussi@intel.com het volgende geschreven: > >> > >>> From: Tom Zanussi > >>> > >>> This is the ia32-base.inc moved over from meta-intel. See meta-intel > >>> for the complete history of contributions to this file. > >>> > >>> Here's the initial commit text that explains the purpose of this file: > >>> > >>> The meta-intel BSPs currently have a number of machine settings common > >>> to all - factor these out into a common include file. > >>> > >>> Also add several new intel-specific XSERVER variables for building > >>> XSERVER variables in BSPs. > >>> > >>> Signed-off-by: Tom Zanussi > >>> --- > >>> meta/conf/machine/include/ia32-base.inc | 58 +++++++++++++++++++++++++++++++++ > >>> > >>> +# glibc-related variables > >>> +# > >>> +GLIBC_ADDONS ?= "nptl" > >> > >> That's a distro setting > > > > Yes and no. It makes sense for architectures to have sane defaults and I > > class nptl on ia32 as that. We want OE-Core to function distroless but > > allow distros to change things where it makes sense to them. So I'd > > argue this is reasonable. > > So put it in default-distrovars, not in a machine 'tune' file. This was only seemingly needed on x86 for reasons which I no longer understand and are likely historical and irrelevant now. So yes, we could add it there with an arch override. The likely outcome is the thing will just get removed so I'm finding it hard to get worked up about it though. Cheers, Richard