Re: [kbuild-devel] Re: Cross-referencing frenzy

Eric S. Raymond (esr@thyrsus.com)
Thu, 19 Apr 2001 09:36:13 -0400


Rogier Wolff <R.E.Wolff@BitWizard.nl>:
> I think it should be possible to do:
>
> /* to enable the special stuff, change the "undef" to "define",
> If you really want you can add this to Config.in so that you're presented
> with this choice when configuring your kernel. But it's not neccesary
> for the general public to always see this toggle. */
> #undef CONFIG_SX_SPECIALSTUFF
>
> #ifdef CONFIG_SX_SPECIALSTUFF
> ...
>
> #endif

Yes, I could write and test code to handle this in about twenty minutes.
And I was about to do it when I realized that it would be the wrong thing.

The right answer is that CONFIG_SX_SPECIALSTUFF *should* be flagged as
an error -- because it doesn't belong in the CONFIG_ namespace, which
by definition should be reserved for things the configurators control.

It should be called something else: perhaps ENABLE_SX_SPECIALSTUFF

-- 
		<a href="http://www.tuxedo.org/~esr/">Eric S. Raymond</a>

The IRS has become morally corrupted by the enormous power which we in Congress have unwisely entrusted to it. Too often it acts like a Gestapo preying upon defenseless citizens. -- Senator Edward V. Long - 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/