[OE-core] [PATCH] perl: reorder tar arguments in do_install_ptest()
Aníbal Limón
anibal.limon at linux.intel.com
Mon May 30 21:06:05 UTC 2016
On 05/30/2016 03:39 PM, Renato Caldas wrote:
> Hi Aníbal,
>
> Sorry but I'm still a bit unfamiliar with the yocto workflow, should I
> resubmit exact same patch with the [krogoth] prefix added, or should I
> change the commit message in any way? It applies cleanly to both
> master and krogoth.
You need to send two patches one with [krogoth] prefix and another
without it for master.
Also is a good practice to cc the maintainer of the release [krogoth]
release, since the 2.1 release was a little time ago i don't know who is
the current maintainer sorry.
Cheers,
alimon
>
> Cheers,
> Renato
>
> 2016-05-30 21:11 GMT+01:00 Aníbal Limón <anibal.limon at linux.intel.com>:
>> Hi Renato,
>>
>> I tested the patch and works very well.
>> Could you sent another patch with [krogoth] prefix to be backported?
>>
>> Thanks a lot,
>> alimon
>>
>>
>> On 05/28/2016 06:53 AM, Renato Caldas wrote:
>>> On some distributions tar requires the FILE argument to be the last, and
>>> the existing order was causing the subsequent --exclude options to be dropped.
>>>
>>> Fixes [YOCTO #9673].
>>>
>>> Signed-off-by: Renato Caldas <rm.santos.caldas at gmail.com>
>> Acked-by: Aníbal Limón <anibal.limon at linux.intel.com>
>>> ---
>>> meta/recipes-devtools/perl/perl-ptest.inc | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-devtools/perl/perl-ptest.inc b/meta/recipes-devtools/perl/perl-ptest.inc
>>> index 948ea7c..66c5355 100644
>>> --- a/meta/recipes-devtools/perl/perl-ptest.inc
>>> +++ b/meta/recipes-devtools/perl/perl-ptest.inc
>>> @@ -7,8 +7,8 @@ do_install_ptest () {
>>> mkdir -p ${D}${PTEST_PATH}
>>> sed -e "s:\/opt:\/usr:" -i Porting/add-package.pl
>>> sed -e "s:\/local\/gnu\/:\/:" -i hints/cxux.sh
>>> - tar -cf - * --exclude \*.o --exclude libperl.so --exclude Makefile --exclude makefile --exclude hostperl \
>>> - --exclude miniperl --exclude generate_uudmap --exclude patches | ( cd ${D}${PTEST_PATH} && tar -xf - )
>>> + tar -cf - --exclude \*.o --exclude libperl.so --exclude Makefile --exclude makefile --exclude hostperl \
>>> + --exclude miniperl --exclude generate_uudmap --exclude patches * | ( cd ${D}${PTEST_PATH} && tar -xf - )
>>>
>>> sed -i -e "s,${D},,g" \
>>> -e "s,--sysroot=${STAGING_DIR_HOST},,g" \
>>>
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20160530/8c0ddd2d/attachment-0002.sig>
More information about the Openembedded-core
mailing list