Re: 2.6.22-git* regression: Kconfig prompts without help text

[Adrian Bunk <bunk@stusta.de>]

On Wed, Jul 18, 2007 at 08:33:27PM +0200, Jan Engelhardt wrote:
> 
> On Jul 18 2007 11:04, Randy Dunlap wrote:
> >> > I have no idea where it came from.  (not me)
> >> 
> >> It was akpm:
> >> http://linux.bkbits.net:8080/linux-2.6/Documentation/CodingStyle?PAGE=diffs&REV=1.5
> >
> > I think that would only make sense if "---help---" is used (like you did)
> > instead of the plain "help" string.
> >
> > In the case of "---help---", it's OK with me not to be indented by 2 more
> > spaces.
> 
> ===
> 
> kconf also accepts "---help---" besides the usual "help" keyword.
> If it is used, I think we can omit the extra spaces, because the
> dashes are visible enough a sash.


Nope, the majority of "---help---" users don't omit the spaces - and it's 
more readable with them.


> Signed-off-by: Jan Engelhardt <jengelh@gmx.de>
> 
> ---
>  Documentation/CodingStyle |   13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> Index: linux-2.6.23/Documentation/CodingStyle
> ===================================================================
> --- linux-2.6.23.orig/Documentation/CodingStyle
> +++ linux-2.6.23/Documentation/CodingStyle
> @@ -511,17 +511,16 @@ remember: "indent" is not a fix for bad 
>  
>  For all of the Kconfig* configuration files throughout the source tree,
>  the indentation is somewhat different.  Lines under a "config" definition
> -are indented with one tab, while help text is indented an additional two
> -spaces.  Example:
> +are indented with one tab. Example:
>  
>  config AUDIT
>  	bool "Auditing support"
>  	depends on NET
> -	help
> -	  Enable auditing infrastructure that can be used with another
> -	  kernel subsystem, such as SELinux (which requires this for
> -	  logging of avc messages output).  Does not do system-call
> -	  auditing without CONFIG_AUDITSYSCALL.
> +	---help---
> +	Enable auditing infrastructure that can be used with another
> +	kernel subsystem, such as SELinux (which requires this for
> +	logging of avc messages output).  Does not do system-call
> +	auditing without CONFIG_AUDITSYSCALL.
>  
>  Features that might still be considered unstable should be defined as
>  dependent on "EXPERIMENTAL":

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed