[PATCH kernel-3.0] kbuild: Fix help text not displayed in choice option.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxx>

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.

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@xxxxxx>
Reviewed-by: Stuart Menefy <stuart.menefy@xxxxxx>
---

To see this bug in action, go to "PCI access mode" under PCI support, and hit the
help button. You will see no help text displayed. However in  arch/x86/Kconfig under 
"PCI access mode" there is help section.

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.

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


 scripts/kconfig/menu.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/scripts/kconfig/menu.c b/scripts/kconfig/menu.c
index d660086..beeb92e 100644
--- a/scripts/kconfig/menu.c
+++ b/scripts/kconfig/menu.c
@@ -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);
 	}
-- 
1.6.3.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux