[Buildroot] [PATCH] busybox: no need to disable clear and reset

Peter Korsgaard peter at korsgaard.com
Mon May 1 22:01:11 UTC 2017


>>>>> "Arnout" == Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be> writes:

 > Removing clear and reset from the busybox config when the ncurses tools
 > are enabled is not really needed.

 > Since commit 802bff9c42, the busybox install will not overwrite
 > existing programs. Therefore, the tools will be installed correctly
 > regardless of the order of the build:
 > - if busybox is built first, the clear and reset apps are installed,
 >   but they will be overwritten by ncurses;
 > - if ncurses is built first, it will install the clear and reset apps,
 >   and busybox will no longer install them.

 > We prefer not to modify the busybox configuration when not strictly
 > necessary, because it is surprising for the user that his configuration
 > is not applied. Clearly, it's not ideal that busybox is configured with
 > redundant apps, but if the user wants to shrink it, it's possible to
 > provide a custom config.

 > This partially reverts commit 33c72344a8686a136c1da6a056ed6c0945bbf8b7.

 > Cc: Matthew Weber <matthew.weber at rockwellcollins.com>
 > Cc: Danomi Manchego <danomimanchego123 at gmail.com>
 > Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>

Committed to 2017.02.x, thanks.

-- 
Bye, Peter Korsgaard


More information about the buildroot mailing list