Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error

[Linus Torvalds <torvalds@linux-foundation.org>]

On Tue, 6 Feb 2007, David Woodhouse wrote:
> 
> It's a bad example because it's not relevant to the 'select' question,
> and you're trying to use it as a straw man; assigning to me a belief I
> just don't have.

It *is* relevant.

Why are you ignoring the ATA example? Which is exactly the same thing? 
(And where we do the right thing - we just "select SCSI")

Why are you ignoring the USB example - which is exactly the same thing 
(and where we do the *wrong* thing: we "depend on SCSI" and then we have 
about 5 lines of warning both around USB and around SCSI and documentation 
just to say that USB needs it!)

> Perhaps I haven't spoken carefully enough. I mean to argue that being
> nice is good -- but that we shouldn't do so in a way which _costs_ those
> who use the system most.

It costs you nothing but arguing. 

> No, you really aren't paying attention. You CAN DO IT WITHOUT SELECT.

No, I was paying attention. You seem to just be obtuse on purpose.

WE WANT TO BE NICE.
 - the firewall example was not an example of 'select', but of the "we 
   want to be nice". But you simply DID NOT GET IT.
 - the USB and SATA examples are *also* examples of "we want to be nice", 
   and hell yeah, you need 'select' to do them. Claiming anything else is 
   just stupid.

So: are you stupid, or do you just refuse to even think about it?

> My point is not that we shouldn't be helpful. My point is that 'select'
> is the _wrong_ way to be 'helpful', because that's a PITA for other
> people.

It's a PITA just because you don't listen, and ignore everything I say.

In other words, the problem is *you*.

The problem is NOT "select".

People have even pointed out that you don't even have to create new tools. 
Use menuconfig and "?", which apparently already tells you what the 
dependencies are, and why you can't turn something off..

And yes, I just tried. I couldn't turn SCSI off in menuconfig, so I 
pressed '?', and at the end it does actually say:

	Selected by: ATA && BLOCK && (!M32R && !M68K || BROKEN) && (!SUN4 || BROKEN)

Not hugely readable, and I'm sure it could be improved a lot. 

And yes, I certainly think that it would be a good idea if there would be 
other tools that told you too.  The Kconfig engine *internally* already 
knows all this, of course, so all the hard work has already effectively 
been done, it's just not *shown*.

So you should just face it: the problem *really* isn't 'select'. 

		Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/