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

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

[Fathi Boudra]

* 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)
  - iproute2 (required for netns tests)
  - curl (curl command is required)
  - util-linux (ipcs command is required)

Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>
---
Changes in v3:
 * add iproute2, used by netns tests. Thanks to Tom Rini <trini@konsulko.com>

Changes in v2:
 * add missing backslash

 meta/recipes-extended/ltp/ltp_20170516.bb | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb b/meta/recipes-extended/ltp/ltp_20170516.bb
index f537d60c15..1d5f999dd0 100644
--- a/meta/recipes-extended/ltp/ltp_20170516.bb
+++ b/meta/recipes-extended/ltp/ltp_20170516.bb
@@ -95,7 +95,28 @@ 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 \
+    iproute2 \
+    ldd \
+    libaio \
+    logrotate \
+    perl \
+    python-core \
+    unzip \
+    util-linux \
+    which \
+"
 
 FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a /opt/ltp/testcases/data/nm01/lib.a"
 
@@ -106,4 +127,3 @@ INHIBIT_PACKAGE_STRIP = "1"
 INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
 # However, test_arch_stripped is already stripped, so...
 INSANE_SKIP_${PN} += "already-stripped"
-
-- 
2.13.0

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

[Tom Rini]

On Wed, Jul 05, 2017 at 02:13:11PM +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)
>   - iproute2 (required for netns tests)
>   - curl (curl command is required)
>   - util-linux (ipcs command is required)
> 
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom