Re: [PATCH] Remove reference to "depends" directive from Kconfig documentation.

[Adrian Bunk <bunk@stusta.de>]

On Mon, Dec 18, 2006 at 10:40:25AM -0500, Robert P. J. Day wrote:
> 
>   Remove from the documentation the notion of using "depends" rather
> than "depends on" in Kconfig files.
> 
> Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
> 
> ---
> 
>   given that there are only three Kconfig files left that still use
> "depends" rather than "depends on", there's no point encouraging
> anyone to still use it (although the parser itself still accepts both
> "depends" and "requires").
> 
> 
> diff --git a/Documentation/kbuild/kconfig-language.txt b/Documentation/kbuild/kconfig-language.txt
> index 536d5bf..658abb5 100644
> --- a/Documentation/kbuild/kconfig-language.txt
> +++ b/Documentation/kbuild/kconfig-language.txt
> @@ -77,7 +77,7 @@ applicable everywhere (see syntax).
>    Optionally, dependencies only for this default value can be added with
>    "if".
> 
> -- dependencies: "depends on"/"requires" <expr>
> +- dependencies: "depends on" <expr>
>    This defines a dependency for this menu entry. If multiple
>    dependencies are defined, they are connected with '&&'. Dependencies
>    are applied to all other options within this menu entry (which also

Your patch does something different:
It's not about "depends", it's about the unused alternative "requires".

Removing "requires" sounds reasonable (even for the implementation, not 
only the documentation), but that's different from what you were 
thinking about.

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

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/