[Buildroot] [PATCH 0/8 v3] core: add support for package help (branch yem/help)

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Jun 7 20:18:44 UTC 2016


Hello,

On Sat,  4 Jun 2016 18:30:49 +0200, Yann E. MORIN wrote:

> Yann E. MORIN (8):
>       core/pkg-generic: add support for package-defined help
>       core: name the package before its help text
>       core/pkg-utils: add a macro to pretty-print a help entry
>       package/busybox: use the generic help rules
>       linux: use the generic help rules
>       package/uclibc: use the generic help rules
>       boot/barebox: use the generic help rules
>       core: use the print-help macro to print the main help

This last commit made me realize that the helper is a bad idea:

 1/ It makes the code longer in number of lines.

 2/ It makes it harder to read.

So, as discussed on IRC, I've reworked your patches to drop the helper
that pretty-print the help text, and simply used the @echo calls in the
various packages.

So essentially, I've dropped patches 3/8 and 8/8, and I've reworked
patches 4, 5, 6 and 7 to not use the helper.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list