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 1QsvvI-0006Q8-IU for openembedded-core@lists.openembedded.org; Mon, 15 Aug 2011 14:05:40 +0200 Received: from localhost (localhost [127.0.0.1]) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id p7FC15xB016628 for ; Mon, 15 Aug 2011 13:01:05 +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 16215-04 for ; Mon, 15 Aug 2011 13:01:01 +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 p7FC0xnr016622 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Mon, 15 Aug 2011 13:00:59 +0100 From: Richard Purdie To: Patches and discussions about the oe-core layer In-Reply-To: References: Date: Mon, 15 Aug 2011 13:00:56 +0100 Message-ID: <1313409656.14274.573.camel@rex> Mime-Version: 1.0 X-Mailer: Evolution 2.32.2 X-Virus-Scanned: amavisd-new at rpsys.net Subject: Re: [PATCH 0/2] Populate site config files content to rootfs X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: Patches and discussions about the oe-core layer List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 15 Aug 2011 12:05:40 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Thu, 2011-08-11 at 22:31 +0800, Lianhao Lu wrote: > This series of patches fixed the bug #892. Currently, the site config > files used in the cross compiling environment generated by meta-toolchain > partly come from SITECONFIG_SYSROOT_CACHE, where each package will put > their own site config cache file during their populate-sysroot task. > > However, there is no way to guarantee the meta-toolchain to collect these > site config cache files after all other packages have finished their > populate-sysroot tasks. Since we now encourage users to use the same rootfs > for both running and cross-compiling, by populating the site config files' > content into rootfs and having the CONFIG_SITE pointing to the rootfs, we > can gurantee the cross compiling environment not missing any site config > cache files' content. I'm afraid I'm not very keen on this patch since it complicates the situation and I'm not sure its an improvement. We strive very hard never to add files to a rootfs which aren't packaged and this approach breaks that (and always adds them to certain image types which again, I worry about). Can't we use the no_cache option to siteinfo_get_files() to address this bug, yet leave the cache files where they are currently located as part of the toolchain? Ultimately we could specific extra cache files to add (such as eglibc) and ensure we have the correct dependencies to ensure the caches are generated. Cheers, Richard