Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Paul Eggleton
On Friday 14 August 2015 11:47:16 Burton, Ross wrote:
 On 11 August 2015 at 09:44, Patrick Ohly patrick.o...@intel.com wrote:
  Installing tar into the sysroot leads to race conditions
  (tasks which do not depend on tar-replacement-native may already
  call tar while it's installation is incomplete). Avoid those
  by installing only the tar binary under the name tar-native.
 
 So the point of tar-replacement-native was that it's a *replacement* for
 the host tar, so it needs the same name.  If it was to be installed as an
 opt-in alternative there's a mechanism for doing that already (see
 EXTRANATIVEPATH) but that would break the purpose of tar-replacement-native.
 
 I think for this situation you should show how to enable xattrs for images,
 with the caveat that if your host tar isn't suitable the user should
 bitbake just tar-replacement-native *first*.

Or install the buildtools perhaps.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Burton, Ross
On 14 August 2015 at 12:01, Paul Eggleton paul.eggle...@linux.intel.com
wrote:

 Or install the buildtools perhaps.


Yes, I had that in my head but someone failed to write it down.  There's a
fair chance that a host with a sufficiently old tar also needs other
updates, and that's when buildtools is used (which includes tar).

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Paul Eggleton
On Friday 14 August 2015 16:52:08 Patrick Ohly wrote:
 On Fri, 2015-08-14 at 12:03 +0100, Burton, Ross wrote:
  On 14 August 2015 at 12:01, Paul Eggleton
  
  paul.eggle...@linux.intel.com wrote:
  Or install the buildtools perhaps.
  
  Yes, I had that in my head but someone failed to write it down.
  There's a fair chance that a host with a sufficiently old tar also
  needs other updates, and that's when buildtools is used (which
  includes tar).
 
 The hosts we talk about (Debian 7, CentOS 6) work fine for normal usage
 without having to install buildtools.

Really? You must have added *something* on top of CentOS 6 because last I 
checked its git, tar and python versions supplied out of the box were way too 
old to even get the build to start.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Patrick Ohly
On Fri, 2015-08-14 at 12:03 +0100, Burton, Ross wrote:
 
 On 14 August 2015 at 12:01, Paul Eggleton
 paul.eggle...@linux.intel.com wrote:
 Or install the buildtools perhaps.
 
 
 
 Yes, I had that in my head but someone failed to write it down.
 There's a fair chance that a host with a sufficiently old tar also
 needs other updates, and that's when buildtools is used (which
 includes tar).

The hosts we talk about (Debian 7, CentOS 6) work fine for normal usage
without having to install buildtools. Having to install them for a
situation that could also be dealt with by bitbake automatically seems
not very user friendly.

On IRC, Ross suggested that I should rewrite the regular tar recipe such
that it installs tar as opt-in binary and remove the (mostly obsolete)
tar-replacement-native). I'll send updated patches based on that.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Patrick Ohly
On Fri, 2015-08-14 at 15:56 +0100, Paul Eggleton wrote:
 On Friday 14 August 2015 16:52:08 Patrick Ohly wrote:
  On Fri, 2015-08-14 at 12:03 +0100, Burton, Ross wrote:
   On 14 August 2015 at 12:01, Paul Eggleton
   
   paul.eggle...@linux.intel.com wrote:
   Or install the buildtools perhaps.
   
   Yes, I had that in my head but someone failed to write it down.
   There's a fair chance that a host with a sufficiently old tar also
   needs other updates, and that's when buildtools is used (which
   includes tar).
  
  The hosts we talk about (Debian 7, CentOS 6) work fine for normal usage
  without having to install buildtools.
 
 Really? You must have added *something* on top of CentOS 6 because last I 
 checked its git, tar and python versions supplied out of the box were way too 
 old to even get the build to start.

I thought someone had said that CentOS is okay out-of-the-box, but I may
have misunderstood.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Paul Eggleton
On Friday 14 August 2015 17:38:38 Patrick Ohly wrote:
 On Fri, 2015-08-14 at 15:56 +0100, Paul Eggleton wrote:
  On Friday 14 August 2015 16:52:08 Patrick Ohly wrote:
   On Fri, 2015-08-14 at 12:03 +0100, Burton, Ross wrote:
On 14 August 2015 at 12:01, Paul Eggleton

paul.eggle...@linux.intel.com wrote:
Or install the buildtools perhaps.

Yes, I had that in my head but someone failed to write it down.
There's a fair chance that a host with a sufficiently old tar also
needs other updates, and that's when buildtools is used (which
includes tar).
   
   The hosts we talk about (Debian 7, CentOS 6) work fine for normal usage
   without having to install buildtools.
  
  Really? You must have added *something* on top of CentOS 6 because last I
  checked its git, tar and python versions supplied out of the box were way
  too old to even get the build to start.
 
 I thought someone had said that CentOS is okay out-of-the-box, but I may
 have misunderstood.

CentOS 7 might be though since it's quite a bit newer.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/3] tar-replacement-native: avoid race condition with host tar

2015-08-14 Thread Burton, Ross
On 11 August 2015 at 09:44, Patrick Ohly patrick.o...@intel.com wrote:

 Installing tar into the sysroot leads to race conditions
 (tasks which do not depend on tar-replacement-native may already
 call tar while it's installation is incomplete). Avoid those
 by installing only the tar binary under the name tar-native.


So the point of tar-replacement-native was that it's a *replacement* for
the host tar, so it needs the same name.  If it was to be installed as an
opt-in alternative there's a mechanism for doing that already (see
EXTRANATIVEPATH) but that would break the purpose of tar-replacement-native.

I think for this situation you should show how to enable xattrs for images,
with the caveat that if your host tar isn't suitable the user should
bitbake just tar-replacement-native *first*.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core