Hi Johannes, > > +/* This funcionality first appears "officially" in 2.6.28... */ > > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,28) > > This bites all developers of the tool. I'd use #ifdef > NL80211_ATTR_SUPPORTED_IFTYPES like Jouni did for a bunch of things but > I forgot to add > #define NL80211_ATTR_SUPPORTED_IFTYPES NL80211_ATTR_SUPPORTED_IFTYPES > to nl80211.h > > On the other hand, I'm with Pavel in that it sucks if you have a tool > which your distro compiled against 2.6.27 and then suddenly you want to > upgrade your kernel to 2.6.28 and the features don't work... if the numbers you are planning to use are stable, then please use the above approach to just add the missing defines to it. Like adding a compat.h file with #ifndef NL80211_ATTR_SUPPORTED_IFTYPES #define NL80211_ATTR_SUPPORTED_IFTYPES boo #endif That would make a distro complied binary work with an updates kernel and we don't end up with the issue the iwconfig current has when it has been compiled for an older kernel. However make sure to keep you numbers stable. Otherwise you break kernel and/or userspace assumptions. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
