Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies
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.ja...@gmail.com
--
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com
signature.asc
Description: Digital signature
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies
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.ja...@gmail.com
signature.asc
Description: Digital signature
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies
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
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies
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.mari...@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
> ---
> 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
>
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies
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
---
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
