[Buildroot] [PATCH 23/50 v2] package/busybox: invert dependency with netcat
Arnout Vandecappelle
arnout at mind.be
Sun Jul 8 11:00:34 UTC 2018
On 08-07-18 11:45, Yann E. MORIN wrote:
> Since netcat would be installed before busybox, we don't need to remove
> a pre-installed 'nc' anymore.
>
> If another netcat implementation (e.g. netcat-openbsd) is also enabled,
> there is no way to tell which would win in the end, especially when we
> add TLPB, in which case that will be forbidden.
>
> So, we don't need to remove anything anymore.
Well, we will have to add a dependency, and then we have to re-introduce this
thing. But maybe we'll have another solution (e.g. making them mutually
exclusive at the configure level) so let's remove it for now.
Regards,
Arnout
>
> Signed-off-by: Yann E. MORIN <yann.morin.1998 at free.fr>
> ---
> package/busybox/busybox.mk | 1 +
> package/netcat/netcat.mk | 13 -------------
> 2 files changed, 1 insertion(+), 13 deletions(-)
>
> diff --git a/package/busybox/busybox.mk b/package/busybox/busybox.mk
> index 003909806a..eeb9dc5bac 100644
> --- a/package/busybox/busybox.mk
> +++ b/package/busybox/busybox.mk
> @@ -22,6 +22,7 @@ BUSYBOX_LDFLAGS = \
>
> # Packages that provide commands that may also be busybox applets:
> BUSYBOX_DEPENDENCIES = \
> + $(if $(BR2_PACKAGE_NETCAT),netcat) \
> $(if $(BR2_PACKAGE_NETCAT_OPENSBSD),netcat-openbsd) \
> $(if $(BR2_PACKAGE_NMAP),nmap) \
> $(if $(BR2_PACKAGE_NTP),ntp) \
> diff --git a/package/netcat/netcat.mk b/package/netcat/netcat.mk
> index d8b3c930b9..eb7ddcac27 100644
> --- a/package/netcat/netcat.mk
> +++ b/package/netcat/netcat.mk
> @@ -9,17 +9,4 @@ NETCAT_SITE = http://downloads.sourceforge.net/project/netcat/netcat/$(NETCAT_VE
> NETCAT_LICENSE = GPL-2.0+
> NETCAT_LICENSE_FILES = COPYING
>
> -# Ensure Busybox gets built/installed before, so that this package
> -# overrides Busybox nc.
> -ifeq ($(BR2_PACKAGE_BUSYBOX),y)
> -NETCAT_DEPENDENCIES += busybox
> -endif
> -
> -# Netcat doesn't overwrite a pre-existing 'nc' (e.g. from busybox) so
> -# force-remove it.
> -define NETCAT_RMOVE_NC_LINK
> - rm -f $(TARGET_DIR)/usr/bin/nc
> -endef
> -NETCAT_PRE_INSTALL_TARGET_HOOKS += NETCAT_RMOVE_NC_LINK
> -
> $(eval $(autotools-package))
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list