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 Latest3 of 84 patches 0 repliesNew

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

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
65/84package/gnuradio: split long omment+0 -00 replies
80/84boot/at91bootstrap3: use 'menuconfig'+0 -00 replies
82/84boot/grub: use 'menuconfig'+0 -00 replies

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchArnout Vandecappelle (Essensium/Mind)
SeriesA-b
65/84
80/84
82/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 23 lines · Yann E. MORIN

Arnout, All,

On 2015-12-31 01:23 +0100, Arnout Vandecappelle spake thusly:

Quoted 20 lines · Yann E. MORIN
Quoted 18 lines

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;

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

Quoted 6 lines
Quoted 3 lines

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

Thank you for the review. I know by experience that reviewing long series can be tedious, so: thanks!

You can now ack up to and including 58 (libiio).

Quoted 6 lines
Quoted 3 lines

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.

Yep, I sent the series as an RFC, just to "feel the temperature of the water".

For actual submission, perhaps squash a little here and there. Although in fact
they're not really squashable, except perhaps 41-84 but that would be a pretty big patch.

Regards,
Arnout

Quoted 6 lines

Thanks! :-)

Regards,
Yann E. MORIN.

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