Skip to content

RFC all: use 'menuconfig' to group sub-options (branch yem/menuconfig)

Arnout Vandecappelle buildroot 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest27 of 84 patches 0 repliesNew

27 of 84 patches are in this archive. The cover letter announces 84.

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
8/84package/alsa-utils: use 'menuconfig' instead of 'config'+'menu'+0 -01 reply
10/84package/angularjs: simplify modules selection+0 -01 reply
12/84package/gd: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
14/84package/jquery: include external plugins from jquery's Config.in+0 -01 reply
14/84package/jquery: include external plugins from jquery's Config.in+0 -00 replies
14/84package/jquery: include external plugins from jquery's Config.in+0 -00 replies
16/84package/libdrm: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
17/84package/mono: use 'menuconfig' instead of 'config'+'menu'+0 -00 replies
17/84package/mono: use 'menuconfig' instead of 'config'+'menu'+0 -00 replies
18/84package/ncftp: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
20/84package/nodejs: use 'menuconfig' instead of 'config'+'menu'+0 -00 replies
22/84package/perl: use 'menuconfig' instead of 'config'+'menu'+0 -00 replies
22/84package/perl: use 'menuconfig' instead of 'config'+'menu'+0 -01 reply
24/84package/php: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
24/84package/php: use 'menuconfig' instad of 'config'+'menu'+0 -01 reply
27/84package/python{, 3}: move inclusion of external python modules+0 -00 replies
27/84package/python{, 3}: move inclusion of external python modules+0 -00 replies
29/84package/tcl: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
29/84package/tcl: use 'menuconfig' instad of 'config'+'menu'+0 -00 replies
31/84package/udpcast: use 'menuconfig' instad of 'config'+'menu'+0 -01 reply
32/84package/gstreamer: include plugins from gstreamer's real Config.in+0 -01 reply
32/84package/gstreamer: include plugins from gstreamer's real Config.in+0 -01 reply
33/84package/gstreamer: use 'menuconfig' instead of 'config'+'menu'+0 -01 reply
38/84docs/manual: add example about sub-options+0 -01 reply
38/84docs/manual: add example about sub-options+0 -01 reply
39/84docs/manual: use 'menuconfig' when there are more than 5 sub-options+0 -00 replies
46/84package/directfb: use 'menuconfig'+0 -01 reply

Reviews trailers per patch

15 of 27 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchArnout Vandecappelle (Essensium/Mind)
SeriesA-b A-b
8/84
10/84
12/84
14/84A-b
14/84A-b
14/84A-b
16/84
17/84A-b
17/84A-b
18/84
20/84
22/84A-b
22/84A-b
24/84A-b
24/84A-b
27/84A-b
27/84A-b
29/84A-b
29/84A-b
31/84
32/84A-b
32/84A-b
33/84
38/84
38/84
39/84
46/84

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [Buildroot] [PATCH 0/84 RFC] all: use 'menuconfig' to group sub-options (branch yem/menuconfig)
Quoted 10 lines · Yann E. MORIN

Hello All!

This RFC series is a proposal to cleanup the layout of the menuconfig UI. Although it looks to be pretty long (84 patches), it really is only three different changes :

- patches 1-6: re-order comments to fix indentation of sub-options; this can probably be applied without too much discussion, as it is really only a cleanup;

Quoted 8 lines

- patches 7-35: changes use of 'config'+'menu' into a 'menuconfig' (the Kconfig keywords) to automatically arrange the sub-options into sub-menus;

- patches 36-84: document that packages with 5 or more options should use sub-menus (with 'menuconfig') to store their sub-options (patches 36-39); it then changes a lot of packages accordingly (patches 40-84).

I reviewed and tested patches 7-39, you can add my

Acked-by Arnout Vandecappelle (Essensium/Mind) <[email protected]>

to the ones where I didn't have comments. I didn't bother adding Acks everywhere because it's anyway not going to be picked up by Patchwork.

Regards,
Arnout

Signature · 6 lines
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

Read the cover letter in its thread