[OE-core] [PATCH] ltp: add acl, attr, curl and util-linux runtime dependencies

[Fathi Boudra]

On 3 July 2017 at 11:59, Maxin B. John <maxin.john at intel.com> wrote:
> Hi,
>
> On Mon, Jul 03, 2017 at 11:03:31AM +0300, Fathi Boudra wrote:
>> * Use a list and re-order alphabetically to make it easier to read and
>>   update.
>> * Add missing runtime dependencies:
>>   - acl (getfacl/setfacl commands are required)
>>   - attr (getfattr/setfattr commands are required)
>>   - curl (curl command is required)
>>   - util-linux (ipcs command is required)
>>
>> Reported-by: Naresh Kamboju <naresh.kamboju at linaro.org>
>> Signed-off-by: Fathi Boudra <fathi.boudra at linaro.org>
>> ---
>>  meta/recipes-extended/ltp/ltp_20170516.bb | 23 +++++++++++++++++++++--
>>  1 file changed, 21 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb b/meta/recipes-extended/ltp/ltp_20170516.bb
>> index f537d60c15..564fdcf8c2 100644
>> --- a/meta/recipes-extended/ltp/ltp_20170516.bb
>> +++ b/meta/recipes-extended/ltp/ltp_20170516.bb
>> @@ -95,7 +95,27 @@ do_install(){
>>      cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
>>  }
>>
>> -RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect ldd unzip gzip cpio cronie logrotate which at"
>> +RDEPENDS_${PN} = "
>> +    acl \
>> +    at \
>> +    attr \
>> +    bash \
>> +    cpio \
>> +    cronie \
>> +    curl \
>> +    e2fsprogs-mke2fs \
>> +    expect \
>> +    gawk \
>> +    gzip \
>> +    ldd \
>> +    libaio \
>> +    logrotate \
>> +    perl \
>> +    python-core \
>> +    unzip \
>> +    util-linux \
>> +    which \
>> +"
>
> These changes look good. However, this patch is causing a parser error:
> ERROR: ParseError at /home/maxin/poky/meta/recipes-extended/ltp/ltp_20170516.bb:98: unparsed line: 'RDEPENDS_${PN} = "
>
> Please fix it.

fixed in v2: https://patchwork.openembedded.org/patch/141483/

>>  FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a /opt/ltp/testcases/data/nm01/lib.a"
>>
>> @@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"
>>  INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>>  # However, test_arch_stripped is already stripped, so...
>>  INSANE_SKIP_${PN} += "already-stripped"
>
> Best Regards,
> Maxin