linux-kernel - Re: [2.6 patch] kconfig: remove the unused "requires" syntax

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Pine.LNX.4.64.0612181432140.5909@localhost.localdomain>
Date:	Mon, 18 Dec 2006 14:38:47 -0500 (EST)
From:	"Robert P. J. Day" <rpjday@...dspring.com>
To:	Adrian Bunk <bunk@...sta.de>
cc:	zippel@...ux-m68k.org,
	Linux kernel mailing list <linux-kernel@...r.kernel.org>,
	kbuild-devel@...ts.sourceforge.net
Subject: Re: [2.6 patch] kconfig: remove the unused "requires" syntax

On Mon, 18 Dec 2006, Adrian Bunk wrote:

> On Mon, Dec 18, 2006 at 01:46:27PM -0500, Robert P. J. Day wrote:

> > p.s.  i didn't look closely enough to see if your patch took out
> > support for both "depends" *and* "requires".  at this point,
> > neither of those are necessary anymore -- it's all "depends on"
> > except for three remaining Kconfig files.
>
> It takes out only "requires" (as the patch description says).

which makes perfect sense, of course.

> Whether to remove the plain "depends" (opposed to "depends on") is a
> different (and perhaps more controversial) question, but it should
> anyway not happen before the last usage is removed.

agreed on that last point.  my patch submission to change "depends" to
"depends on" globally *did* get applied recently:


commit 775ba7ad491a154f99871fe603f03366e84ae159
Merge: d940505... 18b36c7...
Author: Linus Torvalds <torvalds@...dy.osdl.org>
Date:   Tue Dec 12 18:51:51 2006 -0800

    Merge git://git.kernel.org/pub/scm/linux/kernel/git/bunk/trivial

    * git://git.kernel.org/pub/scm/linux/kernel/git/bunk/trivial:
      ...
      kconfig: Standardize "depends" -> "depends on" in Kconfig files
      ...

so it's clear that that transformation was approved.  now we just wait
for the patch to deal with the last three files to go through and
we're all set.

$ grep "depends" $(find . -name Kconfig) | grep -v "depends on"
./arch/arm/mm/Kconfig:  depends !MMU && CPU_CP15 && !CPU_ARM740T
./arch/arm/Kconfig:     depends CPU_XSCALE || CPU_XSC3
./arch/v850/Kconfig:      depends !V850E2_SIM85E2C


rday
-
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/