[OpenWrt-Devel] [PATCH] scripts/metadata: Allow to select which profiles to build

Felix Fietkau nbd at openwrt.org
Mon Jan 11 15:20:28 EST 2016

On 2016-01-11 21:11, Daniel Dickinson wrote:
> On 11/01/16 08:35 AM, Felix Fietkau wrote:
>> I like the idea of allowing the user to select multiple profiles.
>> However, there also needs to be a clean and simple way to select a
>> single profile without going through the list and deselecting everything.
> Actually, while it would need to be extended to other platforms, on 
> ar71xx that is exactly what Minimal does.  In fact, assuming your on the 
> first run of menuconfig (i.e. the only time defaults work, due to the 
> way KConfig works; this is also a problem for ALL_KMODS and ALL (that 
> de-selecting all kmods and for de-selecting all packages), if you 
> deselect Default, all the items selected will become unselected.
I think that will not be clear to users at all, and it means you can't
conveniently switch from a config with all profiles to one with just one
profile enabled without re-making your changes.

> It is not possible with KConfig to have an *existing* .config and have 
> an option that causes deselection of selected items, that does not also 
> force deselection without the option to override (e.g. depends on 
> !MINIMAL won't work because it would force the item to off without 
> option to override).
> If you have some clever way to avoid this problem, I'd love to hear it, 
> because I don't know how to do it in KConfig.  I don't see how to allow 
> an *overridable* deselection of *all* items, unless perhaps one could 
> introduce a new submenu that is a choice section (I'd have to test to 
> see if that would actually work, or if duplicate symbols would be a 
> problem).
You could emit some extra code to have an option to select a single
profile, which will deselect all profiles except for one that is
selected by a choice value.

- Felix
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org

More information about the openwrt-devel mailing list