* [PATCH 1/1] util-linux: Add missing ptest dependencies [not found] <cover.1558300553.git.just.another.mariano@gmail.com> @ 2019-05-19 21:21 ` Mariano López 2019-05-22 11:03 ` Martin Jansa 0 siblings, 1 reply; 5+ messages in thread From: Mariano López @ 2019-05-19 21:21 UTC (permalink / raw) To: openembedded-core There are some missing dependencies for the util-linux-ptest package that causes inconsistencies in the package tests run in different images. The kernel module in RRECOMMENDS is not build at this time, it needs more testing and check if the configuration change can be part of the yocto-kernel-cache repository. Signed-off-by: Mariano López <just.another.mariano@gmail.com> --- meta/recipes-core/util-linux/util-linux.inc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/meta/recipes-core/util-linux/util-linux.inc b/meta/recipes-core/util-linux/util-linux.inc index 34255a2dec..6dfbe0b683 100644 --- a/meta/recipes-core/util-linux/util-linux.inc +++ b/meta/recipes-core/util-linux/util-linux.inc @@ -142,7 +142,8 @@ RDEPENDS_${PN}_class-nativesdk = "" RPROVIDES_${PN}-dev = "${PN}-libblkid-dev ${PN}-libmount-dev ${PN}-libuuid-dev" RDEPENDS_${PN}-bash-completion += "${PN}-lsblk" -RDEPENDS_${PN}-ptest = "bash grep coreutils which btrfs-tools ${PN} sed" +RDEPENDS_${PN}-ptest = "bash bc btrfs-tools coreutils e2fsprogs grep iproute2 kmod mdadm ${PN} procps sed socat which xz" +RRECOMMENDS_${PN}-ptest = "kernel-module-scsi-debug" RDEPENDS_${PN}-swaponoff = "${PN}-swapon ${PN}-swapoff" ALLOW_EMPTY_${PN}-swaponoff = "1" -- 2.21.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] util-linux: Add missing ptest dependencies 2019-05-19 21:21 ` [PATCH 1/1] util-linux: Add missing ptest dependencies Mariano López @ 2019-05-22 11:03 ` Martin Jansa 2019-05-22 14:18 ` Adrian Bunk 0 siblings, 1 reply; 5+ messages in thread From: Martin Jansa @ 2019-05-22 11:03 UTC (permalink / raw) To: Mariano López; +Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 2096 bytes --] This pulls bc to many images with busybox, causing busybox postinst to fail with: update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid since /usr/bin/bc exists and is not a link I'll check which busybox defconfig entry enables BC and send patch to use u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb There already is: ALTERNATIVE_${PN} = "dc" ALTERNATIVE_PRIORITY = "100" but maybe dc is a typo. On Sun, May 19, 2019 at 11:21 PM Mariano López < just.another.mariano@gmail.com> wrote: > There are some missing dependencies for the util-linux-ptest package > that causes inconsistencies in the package tests run in different images. > > The kernel module in RRECOMMENDS is not build at this time, it needs > more testing and check if the configuration change can be part of the > yocto-kernel-cache repository. > > Signed-off-by: Mariano López <just.another.mariano@gmail.com> > --- > meta/recipes-core/util-linux/util-linux.inc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/meta/recipes-core/util-linux/util-linux.inc > b/meta/recipes-core/util-linux/util-linux.inc > index 34255a2dec..6dfbe0b683 100644 > --- a/meta/recipes-core/util-linux/util-linux.inc > +++ b/meta/recipes-core/util-linux/util-linux.inc > @@ -142,7 +142,8 @@ RDEPENDS_${PN}_class-nativesdk = "" > RPROVIDES_${PN}-dev = "${PN}-libblkid-dev ${PN}-libmount-dev > ${PN}-libuuid-dev" > > RDEPENDS_${PN}-bash-completion += "${PN}-lsblk" > -RDEPENDS_${PN}-ptest = "bash grep coreutils which btrfs-tools ${PN} sed" > +RDEPENDS_${PN}-ptest = "bash bc btrfs-tools coreutils e2fsprogs grep > iproute2 kmod mdadm ${PN} procps sed socat which xz" > +RRECOMMENDS_${PN}-ptest = "kernel-module-scsi-debug" > RDEPENDS_${PN}-swaponoff = "${PN}-swapon ${PN}-swapoff" > ALLOW_EMPTY_${PN}-swaponoff = "1" > > -- > 2.21.0 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core > [-- Attachment #2: Type: text/html, Size: 2944 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] util-linux: Add missing ptest dependencies 2019-05-22 11:03 ` Martin Jansa @ 2019-05-22 14:18 ` Adrian Bunk 2019-05-22 20:09 ` Martin Jansa 0 siblings, 1 reply; 5+ messages in thread From: Adrian Bunk @ 2019-05-22 14:18 UTC (permalink / raw) To: Martin Jansa; +Cc: Patches and discussions about the oe-core layer On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote: > This pulls bc to many images with busybox, causing busybox postinst to fail > with: > > update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid > since /usr/bin/bc exists and is not a link Are you enabling CONFIG_BC in your busybox config? It is not enabled in meta/recipes-core/busybox/busybox/defconfig > I'll check which busybox defconfig entry enables BC and send patch to use > u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb > > There already is: > ALTERNATIVE_${PN} = "dc" > ALTERNATIVE_PRIORITY = "100" > but maybe dc is a typo. bc and dc are two different binaries built from bc. busybox used to only have dc, bc was added 6 months ago. cu Adrian -- "Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] util-linux: Add missing ptest dependencies 2019-05-22 14:18 ` Adrian Bunk @ 2019-05-22 20:09 ` Martin Jansa 2019-05-22 20:48 ` Martin Jansa 0 siblings, 1 reply; 5+ messages in thread From: Martin Jansa @ 2019-05-22 20:09 UTC (permalink / raw) To: Adrian Bunk; +Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 1801 bytes --] On Wed, May 22, 2019 at 05:18:25PM +0300, Adrian Bunk wrote: > On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote: > > This pulls bc to many images with busybox, causing busybox postinst to fail > > with: > > > > update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid > > since /usr/bin/bc exists and is not a link > > Are you enabling CONFIG_BC in your busybox config? > It is not enabled in meta/recipes-core/busybox/busybox/defconfig It's not disabled in my defconfig like it is in meta/recipes-core/busybox/busybox/defconfig, maybe it got auto enabled. In log.do_configure I see: ... * * Miscellaneous Utilities * adjtimex (4.7 kb) (ADJTIMEX) [N/y/?] n bbconfig (9.7 kb) (BBCONFIG) [N/y/?] n bc (45 kb) (BC) [Y/n/?] (NEW) dc (36 kb) (DC) [Y/n/?] y Use bc code base for dc (larger, more features) (FEATURE_DC_BIG) [Y] (NEW) y Interactive mode (+4kb) (FEATURE_BC_INTERACTIVE) [Y/n/?] (NEW) Enable bc/dc long options (FEATURE_BC_LONG_OPTIONS) [Y/n] (NEW) beep (2.4 kb) (BEEP) [N/y/?] n chat (6.3 kb) (CHAT) [N/y/?] n conspy (10 kb) (CONSPY) [N/y/?] n ... > > I'll check which busybox defconfig entry enables BC and send patch to use > > u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb > > > > There already is: > > ALTERNATIVE_${PN} = "dc" > > ALTERNATIVE_PRIORITY = "100" > > but maybe dc is a typo. > > bc and dc are two different binaries built from bc. > > busybox used to only have dc, bc was added 6 months ago. Thanks I've never noticed dc before. I see both dc and bc in 1.06 version from meta-gplv2 which was released on 2000-11-15 bc/1.06-r3/image/usr/bin/bc As there is already u-a for dc, I will sent u-a for bc as well. -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 201 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] util-linux: Add missing ptest dependencies 2019-05-22 20:09 ` Martin Jansa @ 2019-05-22 20:48 ` Martin Jansa 0 siblings, 0 replies; 5+ messages in thread From: Martin Jansa @ 2019-05-22 20:48 UTC (permalink / raw) To: Adrian Bunk; +Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 2129 bytes --] On Wed, May 22, 2019 at 10:09:16PM +0200, Martin Jansa wrote: > On Wed, May 22, 2019 at 05:18:25PM +0300, Adrian Bunk wrote: > > On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote: > > > This pulls bc to many images with busybox, causing busybox postinst to fail > > > with: > > > > > > update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid > > > since /usr/bin/bc exists and is not a link > > > > Are you enabling CONFIG_BC in your busybox config? > > It is not enabled in meta/recipes-core/busybox/busybox/defconfig > > It's not disabled in my defconfig like it is in > meta/recipes-core/busybox/busybox/defconfig, maybe it got auto enabled. > > In log.do_configure I see: > ... > * > * Miscellaneous Utilities > * > adjtimex (4.7 kb) (ADJTIMEX) [N/y/?] n > bbconfig (9.7 kb) (BBCONFIG) [N/y/?] n > bc (45 kb) (BC) [Y/n/?] (NEW) dc (36 kb) (DC) [Y/n/?] y > Use bc code base for dc (larger, more features) (FEATURE_DC_BIG) [Y] (NEW) y > Interactive mode (+4kb) (FEATURE_BC_INTERACTIVE) [Y/n/?] (NEW) Enable bc/dc long options (FEATURE_BC_LONG_OPTIONS) [Y/n] (NEW) beep (2.4 kb) (BEEP) [N/y/?] n > chat (6.3 kb) (CHAT) [N/y/?] n > conspy (10 kb) (CONSPY) [N/y/?] n > ... > > > > I'll check which busybox defconfig entry enables BC and send patch to use > > > u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb > > > > > > There already is: > > > ALTERNATIVE_${PN} = "dc" > > > ALTERNATIVE_PRIORITY = "100" > > > but maybe dc is a typo. > > > > bc and dc are two different binaries built from bc. > > > > busybox used to only have dc, bc was added 6 months ago. > > Thanks I've never noticed dc before. > > I see both dc and bc in 1.06 version from meta-gplv2 which was released > on 2000-11-15 > > bc/1.06-r3/image/usr/bin/bc Ah you were talking about busybox having only dc until 6 months ago. Makes sense now, thanks. > As there is already u-a for dc, I will sent u-a for bc as well. > -- > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 201 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-05-22 20:48 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1558300553.git.just.another.mariano@gmail.com>
2019-05-19 21:21 ` [PATCH 1/1] util-linux: Add missing ptest dependencies Mariano López
2019-05-22 11:03 ` Martin Jansa
2019-05-22 14:18 ` Adrian Bunk
2019-05-22 20:09 ` Martin Jansa
2019-05-22 20:48 ` Martin Jansa
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.