[PATCH v3] coreutils: Move stdbuf into an own package coreutils-stdbuf

[PATCH v3] coreutils: Move stdbuf into an own package coreutils-stdbuf

[Adrian Bunk]

This LD_PRELOAD trick is not really suitable for busybox,
so can be the only part of coreutils needed.

coreutils depends on the new package,
so nothing changes when installing coreutils.

Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
v3: coreutils-native fixed
---
 meta/recipes-core/coreutils/coreutils_8.31.bb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/coreutils/coreutils_8.31.bb b/meta/recipes-core/coreutils/coreutils_8.31.bb
index 4a74f619af..8b74ff1916 100644
--- a/meta/recipes-core/coreutils/coreutils_8.31.bb
+++ b/meta/recipes-core/coreutils/coreutils_8.31.bb
@@ -49,7 +49,7 @@ bindir_progs = "arch basename chcon cksum comm csplit cut dir dircolors dirname
                 env expand expr factor fmt fold groups head hostid id install \
                 join link logname md5sum mkfifo nl nohup nproc od paste pathchk \
                 pinky pr printf ptx readlink realpath runcon seq sha1sum sha224sum sha256sum \
-                sha384sum sha512sum shred shuf sort split stdbuf sum tac tail tee test timeout \
+                sha384sum sha512sum shred shuf sort split sum tac tail tee test timeout \
                 tr truncate tsort tty unexpand uniq unlink uptime users vdir wc who whoami yes"
 
 # hostname gets a special treatment and is not included in this
@@ -58,6 +58,10 @@ base_bindir_progs = "cat chgrp chmod chown cp date dd echo false hostname kill l
 
 sbindir_progs= "chroot"
 
+PACKAGE_BEFORE_PN_class-target += "coreutils-stdbuf"
+FILES_coreutils-stdbuf = "${bindir}/stdbuf ${libdir}/coreutils/libstdbuf.so"
+RDEPENDS_coreutils += "coreutils-stdbuf"
+
 # Let aclocal use the relative path for the m4 file rather than the
 # absolute since coreutils has a lot of m4 files, otherwise there might
 # be an "Argument list too long" error when it is built in a long/deep
-- 
2.17.1

Re: [PATCH v3] coreutils: Move stdbuf into an own package coreutils-stdbuf

[Richard Purdie]

On Sun, 2019-10-27 at 17:04 +0200, Adrian Bunk wrote:
> This LD_PRELOAD trick is not really suitable for busybox,
> so can be the only part of coreutils needed.
> 
> coreutils depends on the new package,
> so nothing changes when installing coreutils.
> 
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
> ---
> v3: coreutils-native fixed
> ---
>  meta/recipes-core/coreutils/coreutils_8.31.bb | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

This is better but there look to be nativesdk warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/23/builds/1406

Cheers,

Richard

Re: [PATCH v3] coreutils: Move stdbuf into an own package coreutils-stdbuf

[Richard Purdie]

On Sun, 2019-10-27 at 17:04 +0200, Adrian Bunk wrote:
> This LD_PRELOAD trick is not really suitable for busybox,
> so can be the only part of coreutils needed.
> 
> coreutils depends on the new package,
> so nothing changes when installing coreutils.
> 
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
> ---
> v3: coreutils-native fixed
> ---
>  meta/recipes-core/coreutils/coreutils_8.31.bb | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

This is better but there look to be nativesdk warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/23/builds/1406

Cheers,

Richard

Re: [PATCH v3] coreutils: Move stdbuf into an own package coreutils-stdbuf

[Adrian Bunk]

On Mon, Oct 28, 2019 at 01:44:47PM +0000, Richard Purdie wrote:
> On Sun, 2019-10-27 at 17:04 +0200, Adrian Bunk wrote:
> > This LD_PRELOAD trick is not really suitable for busybox,
> > so can be the only part of coreutils needed.
> > 
> > coreutils depends on the new package,
> > so nothing changes when installing coreutils.
> > 
> > Signed-off-by: Adrian Bunk <bunk@stusta.de>
> > ---
> > v3: coreutils-native fixed
> > ---
> >  meta/recipes-core/coreutils/coreutils_8.31.bb | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> This is better but there look to be nativesdk warnings:
> 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/23/builds/1406

I thought I was also testing nativesdk, but apparently not...

So the following works for native (no effect) but not for nativesdk:
  RDEPENDS_coreutils += "coreutils-stdbuf"

v4 (sic) sent.

> Cheers,
> 
> Richard

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