From patchwork Wed Aug 3 09:36:56 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Srinivas KANDAGATLA X-Patchwork-Id: 1031012 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.4) with ESMTP id p73A7ebY032281 for ; Wed, 3 Aug 2011 10:07:40 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753866Ab1HCKH3 (ORCPT ); Wed, 3 Aug 2011 06:07:29 -0400 Received: from eu1sys200aog116.obsmtp.com ([207.126.144.141]:43598 "EHLO eu1sys200aog116.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753531Ab1HCKH0 (ORCPT ); Wed, 3 Aug 2011 06:07:26 -0400 Received: from beta.dmz-eu.st.com ([164.129.1.35]) (using TLSv1) by eu1sys200aob116.postini.com ([207.126.147.11]) with SMTP ID DSNKTjkd3Jxa7UyP0yUshKfS5CewqhvbF0U/@postini.com; Wed, 03 Aug 2011 10:07:26 UTC Received: from zeta.dmz-eu.st.com (zeta.dmz-eu.st.com [164.129.230.9]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 94E5EF0; Wed, 3 Aug 2011 09:37:57 +0000 (GMT) Received: from mail7.sgp.st.com (mail7.sgp.st.com [164.129.223.81]) by zeta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 58D0C1B27; Wed, 3 Aug 2011 09:37:57 +0000 (GMT) Received: from [10.65.51.191] (king.bri.st.com [10.65.51.191]) by mail7.sgp.st.com (MOS 4.1.8-GA) with ESMTP id AJP07301 (AUTH srinivak); Wed, 3 Aug 2011 11:37:56 +0200 Message-ID: <4E3916B8.10301@st.com> Date: Wed, 03 Aug 2011 10:36:56 +0100 From: Srinivas KANDAGATLA Reply-To: srinivas.kandagatla@st.com Organization: STMicroelectronics User-Agent: Thunderbird 2.0.0.24 (X11/20101027) MIME-Version: 1.0 To: Arnaud Lacombe Cc: linux-kbuild@vger.kernel.org, stuart.menefy@st.com Subject: Re: [PATCH kernel-3.0] kbuild: Fix help text not displayed in choice option. References: <1312311798-27851-1-git-send-email-srinivas.kandagatla@st.com> In-Reply-To: Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Wed, 03 Aug 2011 10:07:40 +0000 (UTC) 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 > wrote: >> From: Srinivas Kandagatla >> >> 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 >> Reviewed-by: Stuart Menefy >> > 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@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> Reviewed-by: Arnaud Lacombe From 7d1c644b41f246b897db40384618520787445328 Mon Sep 17 00:00:00 2001 From: Srinivas Kandagatla 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 Reviewed-by: Stuart Menefy --- 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