<div dir="ltr"><div><div>Felix,<br><br></div>Would it be unreasonable to have overridable defaults like suggested in <a href="http://metadata.pl" target="_blank">metadata.pl</a>? Convention over configuration and all that.<br><br></div>Eric<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 11, 2016 at 7:35 AM, Felix Fietkau <span dir="ltr"><<a href="mailto:nbd@openwrt.org" target="_blank">nbd@openwrt.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 2016-01-11 06:16, <a href="mailto:openwrt@daniel.thecshore.com">openwrt@daniel.thecshore.com</a> wrote:<br>
> From: Daniel Dickinson <<a href="mailto:openwrt@daniel.thecshore.com">openwrt@daniel.thecshore.com</a>><br>
><br>
> Certain platforms have large numbers of possible images, and it can be<br>
> desirable to build neither all images nor only a single image,<br>
> therefore this patch makes selecting target profiles a menu instead of a<br>
> single choice, which allows the user to build a specific subset of all<br>
> possible images for a target.<br>
><br>
> Signed-off-by: Daniel Dickinson <<a href="mailto:openwrt@daniel.thecshore.com">openwrt@daniel.thecshore.com</a>><br>
> ---<br>
>  scripts/<a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a> | 32 +++++++++++++++++++++++++++++---<br>
>  1 file changed, 29 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/scripts/<a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a> b/scripts/<a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a><br>
> index 48b1b7a..4487d26 100755<br>
> --- a/scripts/<a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a><br>
> +++ b/scripts/<a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a><br>
> @@ -275,8 +275,7 @@ EOF<br>
>  print <<EOF;<br>
>  endchoice<br>
><br>
> -choice<br>
> -     prompt "Target Profile"<br>
> +menu "Target Profile"<br>
><br>
>  EOF<br>
><br>
> @@ -288,8 +287,35 @@ EOF<br>
>  config TARGET_$target->{conf}_$profile->{id}<br>
>       bool "$profile->{name}"<br>
>       depends on TARGET_$target->{conf}<br>
> +EOF<br>
> +<br>
> +                     if (not (($profile->{id} eq 'Default') || ($profile->{id} eq 'Minimal'))) {<br>
> +                             print <<EOF;<br>
> +     default y if TARGET_$target->{conf}_Default<br>
> +     default n if TARGET_$target->{conf}_Minimal<br>
> +EOF<br>
</div></div>I like the idea of allowing the user to select multiple profiles.<br>
However, there also needs to be a clean and simple way to select a<br>
single profile without going through the list and deselecting everything.<br>
Also, I don't like hardcoded profile names in <a href="http://metadata.pl" rel="noreferrer" target="_blank">metadata.pl</a>, a better<br>
approach would be to have a flag as part of a profile that indicates<br>
that it's not a device profile.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Felix<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
openwrt-devel mailing list<br>
<a href="mailto:openwrt-devel@lists.openwrt.org">openwrt-devel@lists.openwrt.org</a><br>
<a href="https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel" rel="noreferrer" target="_blank">https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">Eric Schultz, Community Manager, prpl Foundation<br><a href="http://www.prplfoundation.org" target="_blank">http://www.prplfoundation.org</a><br><a href="mailto:eschultz@prplfoundation.org" target="_blank">eschultz@prplfoundation.org</a><br>cell: 920-539-0404<br>skype: ericschultzwi<br>@EricPrpl</div></div>
</div>