Re: [PATCH v2] kconfig: check for select dependency errors on config load

Sam Ravnborg
2009-02-21T21:49:16Z

On Sat, Feb 21, 2009 at 03:46:29PM -0500, Steven Rostedt wrote:
>
> On Sat, 21 Feb 2009, Steven Rostedt wrote:
> > On Fri, 20 Feb 2009, Randy Dunlap wrote:
> > > EMBEDDED is misnamed. It means "those who think that they know enough
> > > to use all of the power of kconfig."
> > > Some people spell that EXPERT etc.
> > > Or it means "let me shoot myself in the foot."
>
> >
> > Anyway, to avoid having these spit out all the time, I'll see if I can
> > make it not warn if the dependency was on EMBEDDED/EXPERT. I'll try to
> > work on it when I have time.
>
> That was easier than I expected. Here's an updated version of the patch.
>
> It now finds the symbol EMBEDDED, saves its state. Updates its state to
> 'yes', runs the tests, resets its state back to what it was.

We do not want to have such hardcoded information
about the Kconfig structure in the backend.
Rather we should fix the configuration once and for all so EMBEDDED
get split into the sensible options is actually is.

Options that come to my mind is:
OPTIMIZE_FOR_TEXT_SIZE
OPTIMIZE_FOR_DATA_SIZE
KERNEL_EXPERT

And we should them make their use of select conform
like the rest of the kernel configuration.

So the first version of your patch is IMO better - but
it could use some comments so others better follows what is
going on. The current level of comments in the kconfig
source base is not an example to follw

Sam