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

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

[Robert P. J. Day]

  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

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

[Adrian Bunk]

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

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

[Robert P. J. Day]

On Mon, 18 Dec 2006, Adrian Bunk wrote:

> 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.

argh, sorry, brain glitch there.  i'll resubmit with the right
wording.

rday