Re: linux-2.4.6-pre6: numerous dep_{bool,tristate} $CONFIG_ARCH_xxx bugs

Adam J. Richter (adam@yggdrasil.com)
Sat, 30 Jun 2001 06:32:36 -0700


>> Argh! I just accidentally sent and older version of my
>> patch. Here is the current version. Sorry about that.

>This just breaks stuff

>> +for var in $(cat arch/*/config.in |
>> + egrep -w -v '^[ ]*int' |
>> + tr ' $"' '\n\n\n' |
>> + egrep '^CONFIG_[A-Z0-9_]*$' |
>> + sort -u) ; do
>> + define_bool "$var" "n"
>> +done
>> +set -f
>>

>You've changed the entire semantics of dep_tristate by doing this

Please provide a real example.

As far as I can tell, the only change would be to make
dep_tristate behave the way it was expected to, as in
drives/sound/Config.in:

dep_tristate ' Netwinder WaveArtist' CONFIG_SOUND_WAVEARTIST $CONFIG_SOUND_OSS $CONFIG_ARCH_NETWINDER

The current, unintended, behavior is to allow this module to be
built all all non-ARM architectures (and probably bomb out), as well
as the intended architecture of ARM-Netwinder, and not any other ARM
platforms. The intended behavior, and the one that you would get with
either my change or Keith's is to only allow this module to be built
for ARM-Netwinder.

Adam J. Richter __ ______________ 4880 Stevens Creek Blvd, Suite 104
adam@yggdrasil.com \ / San Jose, California 95129-1034
+1 408 261-6630 | g g d r a s i l United States of America
fax +1 408 261-6631 "Free Software For The Rest Of Us."
-
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/