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

Message ID 20170705111311.13943-1-fathi.boudra@linaro.org
State New
Headers show

Commit Message

Fathi Boudra July 5, 2017, 11:13 a.m.
* 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(-)

-- 
2.13.0

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

Comments

Tom Rini July 5, 2017, 12:29 p.m. | #1
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
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Patch

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"
-