Hi Florian, On Thu, Apr 25, 2013 at 12:43:28PM +0200, Florian Westphal wrote: > This is a partial revert of a0c885ae5a79457aa592cb70c27a7dee619762a4 > > Specifically, it removes the header linux/netfilter/nfnetlink_queue.h > added in that commit. The idea for caching that file header is to make sure libnetfilter_queue compiles out of the box, without external linux kernel headers installed in the system. The idea is to make sure the libraries compilation does not break in case there's an old library header installed in the system. > 1), there is already a /usr/include/linux/netfilter/nfnetlink_queue.h, > which is part of the linux kernel API As said, that file may not be available or may be stale. > 2), we already have > include/libnetfilter_queue/linux_nfnetlink_queue.h That was added initially for the old libnetfilter_queue API, I'd like to get rid of it. It's currently being installed and it results in (very likely) two duplicated headers in the system (the one from the linux kernel headers and this one from libnetfilter_queue). Once the old API is deprecated (we should do that anytime soon), we can get finally rid of linux_nfnetlink_queue.h > which contains the same definitions/structures/macros, so it makes > little sense to have two headers in libnetfilter_queue that share > almost their entire content. > > [ worse, the nfnetlink_queue.h header reverted here actually is > incompatible with mainline kernels, since a few defines have the > wrong value ... ] Then, please refresh it. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html