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 <[email protected]>
---
  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
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to