[OE-core] [PATCH v4] coreutils: Move stdbuf into an own package coreutils-stdbuf
Randy MacLeod
randy.macleod at windriver.com
Wed Oct 30 20:36:58 UTC 2019
On 10/28/19 6:13 PM, Adrian Bunk wrote:
> This LD_PRELOAD trick is not really suitable for busybox,
> so can be the only part of coreutils needed.
Which LD_PRELOAD trick?
Ah, I think you mean:
The coreutils implementation of stdbuf uses LD_PRELOAD [1]
to dynamically load libstdbuf which calls setvbuf().
This can't be implemented simply in busybox since it produces
one or two statically linked executables. Therefore, stdbuf
should be packaged separately.
[1] ??
http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=blob;f=src/stdbuf.c;h=7eaadb45fef8c01783eeb5c27853fafbcef34f1c;hb=HEAD#l188
Could you send a v5 for plebs like me?
../Randy
>
> coreutils depends on the new package,
> so nothing changes when installing coreutils.
>
> Signed-off-by: Adrian Bunk <bunk at stusta.de>
> ---
> 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..57b2c1bdba 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_class-target += "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
>
--
# Randy MacLeod
# Wind River Linux
More information about the Openembedded-core
mailing list