[I wrote] > > I guess I wasn't quite clear: my current implementation is both > > visibility + value, not visibility only (like current if [ ]) or > > value only (like an earlier discussion of dep_if).
[Greg Banks] > Aha. I think you're going to be arguing uphill to get it in. Could be. Perhaps none of this will see the light of day. But, while I've designed it to replace the current syntax, it can coexist with it for as long as necessary. > > and easy to work around if not wanted. > > Not so sure about that. My Menuconfig patch still has one bug, which I'm still thinking about the best way to fix, so I can't properly test this stuff yet - but with the Config.in files I've dealt with so far, I haven't found it hard to work around this semantic difference. There are lots of instances of things like if [ "$CONFIG_FOO" = "y" -o "$CONFIG_FOO" = "m" ]; then dep_tristate 'Bar' CONFIG_BAR $CONFIG_FOO ... fi which can be replaced by dep_if CONFIG_FOO tristate 'Bar' CONFIG_BAR ... dep_fi but few, if any, instances of if [ "$CONFIG_FOO" = "y" -o "$CONFIG_FOO" = "m" ]; then tristate 'Bar' CONFIG_BAR fi which would require the more complex dep_if CONFIG_FOO=y or CONFIG_FOO=m tristate 'Bar' CONFIG_BAR dep_fi (Not to mention the possibility that such instances are buggy and should have been a dep_tristate in the first place.) > > My primitives are all set to *not* ignore empty dependency values, > > if and when we can verify that the corpus is ready for that > > change. The mechanism is also there (come The Day) to change > > Oh no, the dreaded flag day. Heh. No, a flag day is where BIGNUM things have to be patched at once because there's no migration path. In this case there *is* a migration path, so the only things that break on The Day are things the gcml2 checker (and a grep through the makefiles for suspicious things like 'ifdef') has warned us about but we didn't bother to fix. Peter ------------------------------------------------------- This sf.net email is sponsored by: OSDN - Tired of that same old cell phone? Get a new here for FREE! https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390 _______________________________________________ kbuild-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/kbuild-devel