Hi Arnaud,
Thankyou for the comments,
Here is the new patch with comments included.
Thanks,
srini
Arnaud Lacombe wrote:
> Hi,
>
> On Tue, Aug 2, 2011 at 3:03 PM, Srinivas KANDAGATLA
> <srinivas.kandagatla@xxxxxx> wrote:
>> 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>
>>
> could you please provide a reduced testcase included in the changelog
> ? That will avoid people looking at the bug later on to figure out
> what caused the issue. The following snippet :
>
> choice
> prompt "Choice"
> ---help---
> HELP TEXT ...
>
> config A
> bool "A"
>
> config B
> bool "B"
>
> endchoice
>
> should be enough, as the problem is caused by "anonymous" choice.
>
>> ---
>>
>> 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.
>>
> that look good to me.
>
> Thanks,
> - Arnaud
>
> ps: do you consider Cc: for stable@ ?
>
>> 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
>>
>From 7d1c644b41f246b897db40384618520787445328 Mon Sep 17 00:00:00 2001
From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxx>
Date: Tue, 2 Aug 2011 18:49:52 +0100
Subject: [PATCH kernel-3.0] kbuild: Fix help text not displayed in choice option.
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@xxxxxx>
Reviewed-by: Stuart Menefy <stuart.menefy@xxxxxx>
---
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
