[Buildroot] [PATCH] netcat: add forced dependence on BusyBox

[Arnout Vandecappelle]

On 31-03-18 12:34, Romain Naour wrote:
> Hi Carlos,
> 
> Le 12/10/2017 à 17:14, Carlos Santos a écrit :
>> It may be necessary if packages become built in parallel, leading to a
>> race condition on the creation of the "nc" link. Moreover, netcat is as
>> shy as BusyBox and would not override an existing file/link, so we must
>> remove $(TARGET_DIR)/usr/bin/nc in advance.
>>
>> Of course this still leaves a race conditon if other netcat competitors
>> but we must assume thet the user is a grown-up person who knows what is
>> doing.
> 
> It seems you are using a custom busybox config file because the netcat applet is
> disabled by default.
> 
> I'm agree, the dependency on busybox is missing but the TARGET_HOOKS doesn't
> seems necessary. I removed it and netcat package is able to override the busybox
> symlink.
> 
>  /usr/bin/install -c netcat output/target/usr/bin/netcat

 I've applied, after removing the unneeded removal of nc.

 If it turns out to be needed after all, please submit a follow-up patch.

 Regards,
 Arnout

> 
> Maybe we should check if other packages depending on
> BR2_PACKAGE_BUSYBOX_SHOW_OTHERS have a dependency on busybox.
[snip]

-- 
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