Message ID | 1445149965-15702-1-git-send-email-james.knight@rockwellcollins.com |
---|---|
State | Accepted |
Commit | 3c943477f6c5fad59c4dc3be1e4a0b113384c60a |
Headers | show |
James, All, On 2015-10-18 02:32 -0400, James Knight spake thusly: > For targets prepared with a usr-move, coreutils should not be attempting > to move binaries from `usr/bin` to `bin`. For the records, pleas see comments already made on the patch doing the same for sed: http://lists.busybox.net/pipermail/buildroot/2015-October/142230.html > Signed-off-by: James Knight <james.knight@rockwellcollins.com> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Regards, Yann E. MORIN. > --- > package/coreutils/coreutils.mk | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/package/coreutils/coreutils.mk b/package/coreutils/coreutils.mk > index bb2ef96..60455ac 100644 > --- a/package/coreutils/coreutils.mk > +++ b/package/coreutils/coreutils.mk > @@ -99,18 +99,24 @@ COREUTILS_CONF_OPTS += --with-openssl=yes > COREUTILS_DEPENDENCIES += openssl > endif > > -define COREUTILS_POST_INSTALL > +ifeq ($(BR2_ROOTFS_MERGED_USR),) > +define COREUTILS_CLEANUP_BIN > # some things go in root rather than usr > for f in $(COREUTILS_BIN_PROGS); do \ > mv -f $(TARGET_DIR)/usr/bin/$$f $(TARGET_DIR)/bin/$$f || exit 1; \ > done > +endef > +COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_CLEANUP_BIN > +endif > + > +define COREUTILS_CLEANUP > # link for archaic shells > ln -fs test $(TARGET_DIR)/usr/bin/[ > # gnu thinks chroot is in bin, debian thinks it's in sbin > mv -f $(TARGET_DIR)/usr/bin/chroot $(TARGET_DIR)/usr/sbin/chroot > endef > > -COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_POST_INSTALL > +COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_CLEANUP > > # If both coreutils and busybox are selected, the corresponding applets > # may need to be reinstated by the clean targets. > -- > 1.9.5.msysgit.1 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
>>>>> "James" == James Knight <james.d.knight@live.com> writes: > For targets prepared with a usr-move, coreutils should not be attempting > to move binaries from `usr/bin` to `bin`. > Signed-off-by: James Knight <james.knight@rockwellcollins.com> Committed, thanks.
diff --git a/package/coreutils/coreutils.mk b/package/coreutils/coreutils.mk index bb2ef96..60455ac 100644 --- a/package/coreutils/coreutils.mk +++ b/package/coreutils/coreutils.mk @@ -99,18 +99,24 @@ COREUTILS_CONF_OPTS += --with-openssl=yes COREUTILS_DEPENDENCIES += openssl endif -define COREUTILS_POST_INSTALL +ifeq ($(BR2_ROOTFS_MERGED_USR),) +define COREUTILS_CLEANUP_BIN # some things go in root rather than usr for f in $(COREUTILS_BIN_PROGS); do \ mv -f $(TARGET_DIR)/usr/bin/$$f $(TARGET_DIR)/bin/$$f || exit 1; \ done +endef +COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_CLEANUP_BIN +endif + +define COREUTILS_CLEANUP # link for archaic shells ln -fs test $(TARGET_DIR)/usr/bin/[ # gnu thinks chroot is in bin, debian thinks it's in sbin mv -f $(TARGET_DIR)/usr/bin/chroot $(TARGET_DIR)/usr/sbin/chroot endef -COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_POST_INSTALL +COREUTILS_POST_INSTALL_TARGET_HOOKS += COREUTILS_CLEANUP # If both coreutils and busybox are selected, the corresponding applets # may need to be reinstated by the clean targets.
For targets prepared with a usr-move, coreutils should not be attempting to move binaries from `usr/bin` to `bin`. Signed-off-by: James Knight <james.knight@rockwellcollins.com> --- package/coreutils/coreutils.mk | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)