On Thu, 2008-09-18 at 17:41 +0200, Johannes Berg wrote: > On Thu, 2008-09-18 at 10:35 -0400, John W. Linville wrote: > > > > +/* 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 I agree. I always prefer to test for features, not for numbers. > 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... > > I don't know. Tell me what to do. I don't really like shipping the > header file either but it may be the lesser of two evils? I think the build system should use the included header by default. But it should be easy to enable compiling against the kernel sources. For instance, if the kernel path is defined in .config or there is an option on the command line. -- Regards, Pavel Roskin -- 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
