Skip to contentSkip to conversation

bitbake.conf: Add run-parts to HOSTTOOLS

PATCH In review André Draszik 3 participants 6 messages Aug 24, 2017 – Aug 25, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 bitbake.conf: Add run-parts to HOSTTOOLS

From: André Draszik <[email protected]>

ca-certificates runs a postinst task, update-ca-certificates, which ultimately wants to execute run-parts.

---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/conf/bitbake.conf +1 -1
index 334ba2361f..8011689118 100644
@@ -469,7 +469,7 @@ HOSTTOOLS += " \
     fgrep file find flock g++ gawk gcc getconf getopt git grep gunzip gzip \
     head hostname install ld ldd ln ls make makeinfo md5sum mkdir mknod \
     mktemp mv nm objcopy objdump od patch perl pod2man pr printf pwd python python2 \
-    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen sed sh sha256sum \
+    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen run-parts sed sh sha256sum \
     sleep sort split stat strings strip tail tar tee test touch tr true uname \
     uniq wc wget which xargs \
 "
Signature · 1 line
2.14.1
#1 lore raw
Quoted 22 lines · André Draszik

From: André Draszik <[email protected]>

ca-certificates runs a postinst task, update-ca-
certificates,
which ultimately wants to execute run-parts.

---
 meta/conf/bitbake.conf | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)
meta/conf/bitbake.conf +0 -0
index 334ba2361f..8011689118 100644
@@ -469,7 +469,7 @@ HOSTTOOLS += " \

     fgrep file find flock g++ gawk gcc getconf getopt git grep gunzip gzip \
     head hostname install ld ldd ln ls make makeinfo md5sum mkdir mknod \
     mktemp mv nm objcopy objdump od patch perl pod2man pr printf pwd python python2 \

Quoted 8 lines

-    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen sed sh sha256sum \
+    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen run- parts sed sh sha256sum \
     sleep sort split stat strings strip tail tar tee test touch tr true uname \
     uniq wc wget which xargs \
 "

Is run-parts guaranteed to be available on the host?

IMHO it would be better to ensure that run-parts is in recipe-sysroot- native when installing ca-certificates. PACKAGE_WRITE_DEPS can be set in ca-certificates.bb for that. I'm just not sure about what provides run-parts in OE.

Signature · 6 lines
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.
#2 lore raw
In reply to André Draszik
Quoted 21 lines · André Draszik

From: André Draszik <[email protected]>

ca-certificates runs a postinst task, update-ca-certificates, which ultimately wants to execute run-parts.

---
 meta/conf/bitbake.conf | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)
meta/conf/bitbake.conf +0 -0
index 334ba2361f..8011689118 100644
@@ -469,7 +469,7 @@ HOSTTOOLS += " \

     fgrep file find flock g++ gawk gcc getconf getopt git grep gunzip gzip \
     head hostname install ld ldd ln ls make makeinfo md5sum mkdir mknod \
     mktemp mv nm objcopy objdump od patch perl pod2man pr printf pwd python python2 \

Quoted 8 lines

-    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen sed sh sha256sum \
+    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen run- parts sed sh sha256sum \
     sleep sort split stat strings strip tail tar tee test touch tr true uname \
     uniq wc wget which xargs \
 "

We definitely should not be doing this, there is clearly a dependency missing somewhere else. ca-certificates should depend on whatever provides that...

Cheers,

Richard

#3 lore raw
Quoted 26 lines · Richard Purdie
Quoted 25 lines · André Draszik

From: André Draszik <[email protected]>

ca-certificates runs a postinst task, update-ca-certificates, which ultimately wants to execute run-parts.

---
 meta/conf/bitbake.conf | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)
meta/conf/bitbake.conf +0 -0
index 334ba2361f..8011689118 100644
@@ -469,7 +469,7 @@ HOSTTOOLS += " \

     fgrep file find flock g++ gawk gcc getconf getopt git grep gunzip gzip \
     head hostname install ld ldd ln ls make makeinfo md5sum mkdir mknod \
     mktemp mv nm objcopy objdump od patch perl pod2man pr printf pwd python python2 \
-    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen sed sh sha256sum \
+    python2.7 python3 ranlib readelf readlink rm rmdir rpcgen run- parts sed sh sha256sum \

Quoted 8 lines
Quoted 4 lines

     sleep sort split stat strings strip tail tar tee test touch tr true uname \
     uniq wc wget which xargs \
 "

We definitely should not be doing this, there is clearly a dependency missing somewhere else. ca-certificates should depend on whatever provides that...

In OE, run-parts is provided by debianutils, or by busybox, both of which don't have a native version in OE.

ca-certificates seems to always have relied on run-parts from the host environment. OE even carries a patch to make it work with run-parts from Fedora.

What would be the preferred way to resolve this so?

Cheers,
Andre'

#4 lore raw
Quoted 9 lines · André Draszik
Quoted 4 lines · Richard Purdie

We definitely should not be doing this, there is clearly a dependency
missing somewhere else. ca-certificates should depend on whatever provides that...

In OE, run-parts is provided by debianutils, or by busybox, both of which
don't have a native version in OE.

Quoted 7 lines

ca-certificates seems to always have relied on run-parts from the host
environment. OE even carries a patch to make it work with run-parts from
Fedora.

What would be the preferred way to resolve this so?

The patches I have in master-next which enable a native version of debianutils and then add it as a dependency from ca-certificates?

Cheers,

Richard

#5 lore raw
Quoted 2 lines · Richard Purdie

The patches I have in master-next which enable a native version of debianutils and then add it as a dependency from ca-certificates?

Cool! Hadn't seen those.

Cheers,
Andre'