From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dan.rpsys.net (dan.rpsys.net [93.97.175.187]) by mail.openembedded.org (Postfix) with ESMTP id 2C4EB615A9 for ; Wed, 16 Oct 2013 16:32:56 +0000 (UTC) Received: from localhost (dan.rpsys.net [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r9GGWSXK008243; Wed, 16 Oct 2013 17:32:28 +0100 X-Virus-Scanned: Debian amavisd-new at dan.rpsys.net Received: from dan.rpsys.net ([127.0.0.1]) by localhost (dan.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 3EPZdWGQcQgi; Wed, 16 Oct 2013 17:32:27 +0100 (BST) Received: from [192.168.3.10] (rpvlan0 [192.168.3.10]) (authenticated bits=0) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r9GGWPHf008228 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NOT); Wed, 16 Oct 2013 17:32:26 +0100 Message-ID: <1381941141.29912.494.camel@ted> From: Richard Purdie To: Darren Hart Date: Wed, 16 Oct 2013 17:32:21 +0100 In-Reply-To: <1381939546.23041.123.camel@dvhart-mobl4.amr.corp.intel.com> References: <7df25a781aad32cb4f7deaa1c4af839e8dc604a2.1381898855.git.tom.zanussi@linux.intel.com> <3e3f7120140dc606369f19fbfca3e222db06274d.1381898855.git.tom.zanussi@linux.intel.com> <1381925340.29912.461.camel@ted> <1381936247.23041.119.camel@dvhart-mobl4.amr.corp.intel.com> <1381939546.23041.123.camel@dvhart-mobl4.amr.corp.intel.com> X-Mailer: Evolution 3.6.4-0ubuntu1 Mime-Version: 1.0 Cc: Tom Zanussi , openembedded-core@lists.openembedded.org Subject: Re: [PATCH 3/3] wic: add fakeroot to the populate-extfs step X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 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, 16 Oct 2013 16:32:57 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Wed, 2013-10-16 at 09:05 -0700, Darren Hart wrote: > On Wed, 2013-10-16 at 08:10 -0700, Darren Hart wrote: > > On Wed, 2013-10-16 at 13:09 +0100, Richard Purdie wrote: > > > On Wed, 2013-10-16 at 00:14 -0500, Tom Zanussi wrote: > > > > Without this, files in the generated filesystem pick up the wrong > > > > ownership. > > > > > > > > Signed-off-by: Tom Zanussi > > > > --- > > > > scripts/lib/mic/kickstart/custom_commands/partition.py | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/scripts/lib/mic/kickstart/custom_commands/partition.py b/scripts/lib/mic/kickstart/custom_commands/partition.py > > > > index 302cace..309abee 100644 > > > > --- a/scripts/lib/mic/kickstart/custom_commands/partition.py > > > > +++ b/scripts/lib/mic/kickstart/custom_commands/partition.py > > > > @@ -216,7 +216,7 @@ class Wic_PartData(Mic_PartData): > > > > """ > > > > Prepare content for an ext2/3/4 rootfs partition. > > > > """ > > > > - populate_script = "%s/usr/bin/populate-extfs.sh" % native_sysroot > > > > + populate_script = "fakeroot %s/usr/bin/populate-extfs.sh" % native_sysroot > > > > image_extra_space = 10240 > > > > > > > > image_rootfs = rootfs_dir > > > > > > We don't use fakeroot, please use pseudo. If pseudo doesn't work, we > > > need to figure out why and fix it. > > > > > > > Ultimately this needs to work independently from bitbake. Is that > > practical if we use pseudo here? > > Starting to get a bit out of scope for this discussion, but it does > impact the utility of the tool. I don't see a package for pseudo in > either Ubuntu or Fedora. The lack of a package would definitely increase > the barrier to use (lots of people really dislike having to build their > own sources and muck up their filesystem with non-package files). > > Has any work already been kicked off to try and get pseudo packaged in > the major distributions? Who would the right people be to work with on > that? > > BTW, I'm not arguing for using fakeroot here, just trying to look at the > big picture of using wic and what additional effort might be required to > make it successful. I understand that, I'd suggest one solution of pointing people at our standalone buildtools tarball which we can easily include pseudo in. It would also be nice to see it as a tool in its own right in distros. My main worry is that we did used to run into a lot of weird fakeroot bugs, I really don't want to end up there again... Cheers, Richard