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

[Daniel Dickinson]

Actually I thought of a solution, but it starts to get ugly.  A choice 
submemenu with non-duplicate (i.e. TARGET_SINGLE_...) symbols that also 
select TARGET_SINGLE, AND have all the normal TARGET_... depend on 
!(TARGET_SYMBOL && !TARGET_SINGLE_..profile).

That would probably work, but it's rather ugly IMO.

Regards,

Daniel

On 11/01/16 08:35 AM, Felix Fietkau wrote:
> On 2016-01-11 06:16, openwrt at daniel.thecshore.com wrote:
>> From: Daniel Dickinson <openwrt at daniel.thecshore.com>
>>
>> Certain platforms have large numbers of possible images, and it can be
>> desirable to build neither all images nor only a single image,
>> therefore this patch makes selecting target profiles a menu instead of a
>> single choice, which allows the user to build a specific subset of all
>> possible images for a target.
>>
>> Signed-off-by: Daniel Dickinson <openwrt at daniel.thecshore.com>
>> ---
>>   scripts/metadata.pl | 32 +++++++++++++++++++++++++++++---
>>   1 file changed, 29 insertions(+), 3 deletions(-)
>>
>> diff --git a/scripts/metadata.pl b/scripts/metadata.pl
>> index 48b1b7a..4487d26 100755
>> --- a/scripts/metadata.pl
>> +++ b/scripts/metadata.pl
>> @@ -275,8 +275,7 @@ EOF
>>   print <<EOF;
>>   endchoice
>>
>> -choice
>> -	prompt "Target Profile"
>> +menu "Target Profile"
>>
>>   EOF
>>
>> @@ -288,8 +287,35 @@ EOF
>>   config TARGET_$target->{conf}_$profile->{id}
>>   	bool "$profile->{name}"
>>   	depends on TARGET_$target->{conf}
>> +EOF
>> +
>> +			if (not (($profile->{id} eq 'Default') || ($profile->{id} eq 'Minimal'))) {
>> +				print <<EOF;
>> +	default y if TARGET_$target->{conf}_Default
>> +	default n if TARGET_$target->{conf}_Minimal
>> +EOF
> 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.
> Also, I don't like hardcoded profile names in metadata.pl, a better
> approach would be to have a flag as part of a profile that indicates
> that it's not a device profile.
>
> - Felix
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel