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, open...@daniel.thecshore.com wrote:
From: Daniel Dickinson <open...@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 <open...@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@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to