2010-11-18 02:49:16

by Thomas Chou

[permalink] [raw]
Subject: Re: [PATCH] gen_init_cpio: remove leading `/' from file names

Hi Andrew and Michal,

On 10/06/2010 10:51 PM, Mike Frysinger wrote:
> On Wed, Oct 6, 2010 at 03:13, Thomas Chou wrote:
>> When we extracted the generated cpio archive using "cpio -id" command,
>> it complained,
>>
>> cpio: Removing leading `/' from member names
>> var/run
>> cpio: Removing leading `/' from member names
>> var/lib
>> cpio: Removing leading `/' from member names
>> var/lib/misc
>>
>> It is worse with the latest "cpio" or "pax", which tries to overwrite
>> the host file system with the leading '/'.
>
> seems to work as intended on Blackfin systems, and i cant see any
> reason to require the leading slash, so:
> Acked-by: Mike Frysinger<[email protected]>
> -mike
>
>

Would you please review this patch? It is not clear as who should handle
this with the MAINTAINERS.

Best regards,
Thomas


2010-12-02 13:30:08

by Michal Marek

[permalink] [raw]
Subject: Re: [PATCH] gen_init_cpio: remove leading `/' from file names

On Thu, Nov 18, 2010 at 10:52:51AM +0800, Thomas Chou wrote:
> Hi Andrew and Michal,
>
> On 10/06/2010 10:51 PM, Mike Frysinger wrote:
> >On Wed, Oct 6, 2010 at 03:13, Thomas Chou wrote:
> >>When we extracted the generated cpio archive using "cpio -id" command,
> >>it complained,
> >>
> >>cpio: Removing leading `/' from member names
> >>var/run
> >>cpio: Removing leading `/' from member names
> >>var/lib
> >>cpio: Removing leading `/' from member names
> >>var/lib/misc
> >>
> >>It is worse with the latest "cpio" or "pax", which tries to overwrite
> >>the host file system with the leading '/'.
> >
> >seems to work as intended on Blackfin systems, and i cant see any
> >reason to require the leading slash, so:
> >Acked-by: Mike Frysinger<[email protected]>
> >-mike
> >
> >
>
> Would you please review this patch? It is not clear as who should
> handle this with the MAINTAINERS.

Applied to kbuild-2.6.git#kbuild, thanks a lot.

Michal