Commit b5d609db authored by Matej Laitl's avatar Matej Laitl Committed by Sam Ravnborg

kconfig/menuconfig: distinguish between selected-by-another options and comments

menuconfig currently represents options implied by another option ('select'
directive in Kconfig) by prefixing them with '---'.  Unfortunately the same
notation is used for comments.  If the implied option is module capable,
user can still switch between Y and M, all without any feedback until she
visits option's help.  (try saying M to MAC80211 and then toggling
CFG80211)

This patch changes notation of selected-by-another items by introducing 2
new representations for implied options: {*} or {M} for options selected by
another modularized one, thus builtin or module capable, -*- or -M- for
options that cannot be at the moment changed by user.

The idea is to represent actual capability of the option by braces (dashes)
around and to always report actual state by * or M inside.
Signed-off-by: default avatarMatej Laitl <strohel@gmail.com>
Acked-by: default avatarRandy Dunlap <randy.dunlap@oracle.com>
Cc: Roman Zippel <zippel@linux-m68k.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarSam Ravnborg <sam@ravnborg.org>
parent a5bf3d89
...@@ -35,9 +35,13 @@ static const char mconf_readme[] = N_( ...@@ -35,9 +35,13 @@ static const char mconf_readme[] = N_(
"kernel parameters which are not really features, but must be\n" "kernel parameters which are not really features, but must be\n"
"entered in as decimal or hexadecimal numbers or possibly text.\n" "entered in as decimal or hexadecimal numbers or possibly text.\n"
"\n" "\n"
"Menu items beginning with [*], <M> or [ ] represent features\n" "Menu items beginning with following braces represent features that\n"
"configured to be built in, modularized or removed respectively.\n" " [ ] can be built in or removed\n"
"Pointed brackets <> represent module capable features.\n" " < > can be built in, modularized or removed\n"
" { } can be built in or modularized (selected by other feature)\n"
" - - are selected by other feature,\n"
"while *, M or whitespace inside braces means to build in, build as\n"
"a module or to exclude the feature respectively.\n"
"\n" "\n"
"To change any of these features, highlight it with the cursor\n" "To change any of these features, highlight it with the cursor\n"
"keys and press <Y> to build it in, <M> to make it a module or\n" "keys and press <Y> to build it in, <M> to make it a module or\n"
...@@ -569,7 +573,7 @@ static void build_conf(struct menu *menu) ...@@ -569,7 +573,7 @@ static void build_conf(struct menu *menu)
if (sym_is_changable(sym)) if (sym_is_changable(sym))
item_make("[%c]", val == no ? ' ' : '*'); item_make("[%c]", val == no ? ' ' : '*');
else else
item_make("---"); item_make("-%c-", val == no ? ' ' : '*');
item_set_tag('t'); item_set_tag('t');
item_set_data(menu); item_set_data(menu);
break; break;
...@@ -579,10 +583,13 @@ static void build_conf(struct menu *menu) ...@@ -579,10 +583,13 @@ static void build_conf(struct menu *menu)
case mod: ch = 'M'; break; case mod: ch = 'M'; break;
default: ch = ' '; break; default: ch = ' '; break;
} }
if (sym_is_changable(sym)) if (sym_is_changable(sym)) {
item_make("<%c>", ch); if (sym->rev_dep.tri == mod)
item_make("{%c}", ch);
else else
item_make("---"); item_make("<%c>", ch);
} else
item_make("-%c-", ch);
item_set_tag('t'); item_set_tag('t');
item_set_data(menu); item_set_data(menu);
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment