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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 18, 2006 at 01:46:27PM -0500, Robert P. J. Day wrote:
> On Mon, 18 Dec 2006, Adrian Bunk wrote:
> 
> > On Mon, Dec 18, 2006 at 11:41:59AM -0500, Robert P. J. Day wrote:
> > >
> > >   Remove the note in the documentation that suggests people can use
> > > "requires" for dependencies in Kconfig files.
> > >...
> >
> > Considering that noone uses it, what about the patch below to also
> > remove the implementation?
> 
> ... big snip ...
> 
> i have no problem knocking out of the parser anything related to
> "depends" or "requires."  in fact, i did note in earlier patch
> submissions that i was just cleaning the Kconfig files but i was
> leaving the parser alone, and someone else was welcome to take care of
> that.
> 
> if the kbuild folks are good with this, i certainly have no objection.
> 
> rday
> 
> 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).

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.

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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux