kbuild: Fix help text not displayed in choice option.
authorSrinivas Kandagatla <srinivas.kandagatla@st.com>
Tue, 2 Aug 2011 17:49:52 +0000 (18:49 +0100)
committerMichal Marek <mmarek@suse.cz>
Mon, 8 Aug 2011 14:01:54 +0000 (16:01 +0200)
Help text under choice menu is never displayed because it does not have
symbol name associated with it, however many kconfigs have help text
under choice, assuming that it will be displayed when user selects help.
for example in Kconfig if we have:
choice
        prompt "Choice"
        ---help---
           HELP TEXT ...

config A
        bool "A"

config B
        bool "B"

endchoice

Without this patch "HELP TEXT" is not displayed when user selects help
option when "Choice" is highlighted from menuconfig or xconfig or
gconfig.

This patch changes the logic in menu_get_ext_help to display help for
cases which dont have symbol names like choice.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@st.com>
Reviewed-by: Stuart Menefy <stuart.menefy@st.com>
Reviewed-by: Arnaud Lacombe <lacombar@gmail.com>
Cc: stable@kernel.org
Signed-off-by: Michal Marek <mmarek@suse.cz>
scripts/kconfig/menu.c

index d66008639a43f46e60235dee932558a6c4d00223..beeb92e75f8f01208ee3065d5d74dc53a732aa50 100644 (file)
@@ -597,11 +597,10 @@ void menu_get_ext_help(struct menu *menu, struct gstr *help)
        struct symbol *sym = menu->sym;
 
        if (menu_has_help(menu)) {
-               if (sym->name) {
+               if (sym->name)
                        str_printf(help, "%s%s:\n\n", CONFIG_, sym->name);
-                       str_append(help, _(menu_get_help(menu)));
-                       str_append(help, "\n");
-               }
+               str_append(help, _(menu_get_help(menu)));
+               str_append(help, "\n");
        } else {
                str_append(help, nohelp_text);
        }