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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20061218163522.GE10316@stusta.de>
Date:	Mon, 18 Dec 2006 17:35:22 +0100
From:	Adrian Bunk <bunk@...sta.de>
To:	"Robert P. J. Day" <rpjday@...dspring.com>
Cc:	Linux kernel mailing list <linux-kernel@...r.kernel.org>,
	trivial@...nel.org
Subject: Re: [PATCH] Remove reference to "depends" directive from Kconfig documentation.

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@...dspring.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/