diff mbox

[iptables,2/2,RFC] Remove Libc5 support code

Message ID 20150502195138.GB17994@euler
State Accepted
Delegated to: Florian Westphal
Headers show

Commit Message

Felix Janda May 2, 2015, 7:51 p.m. UTC
Current code makes the assumption that !defined(__GLIBC__) means libc5
which is very unlikely the case nowadays.

Fixes compile error because of conflict between kernel and musl headers.
---
If libc5 is considered still relevant, I could try to come up with an
autoconf test.
---
 include/libiptc/ipt_kernel_headers.h | 12 ------------
 1 file changed, 12 deletions(-)

Comments

Florian Westphal May 4, 2015, 10:48 a.m. UTC | #1
Felix Janda <felix.janda@posteo.de> wrote:
> Current code makes the assumption that !defined(__GLIBC__) means libc5
> which is very unlikely the case nowadays.
> 
> Fixes compile error because of conflict between kernel and musl headers.
> ---
> If libc5 is considered still relevant, I could try to come up with an
> autoconf test.

I'm all for removing libc5 support if this is whats preventing iptables
to be built with current non-glibc systems.

Pablo, Patrick, Eric, Jozsef - whats your take on this?
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Pablo Neira Ayuso May 4, 2015, 10:55 a.m. UTC | #2
On Mon, May 04, 2015 at 12:48:09PM +0200, Florian Westphal wrote:
> Felix Janda <felix.janda@posteo.de> wrote:
> > Current code makes the assumption that !defined(__GLIBC__) means libc5
> > which is very unlikely the case nowadays.
> > 
> > Fixes compile error because of conflict between kernel and musl headers.
> > ---
> > If libc5 is considered still relevant, I could try to come up with an
> > autoconf test.
> 
> I'm all for removing libc5 support if this is whats preventing iptables
> to be built with current non-glibc systems.
> 
> Pablo, Patrick, Eric, Jozsef - whats your take on this?

I would say, go take it.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jozsef Kadlecsik May 4, 2015, 11:34 a.m. UTC | #3
On Mon, 4 May 2015, Pablo Neira Ayuso wrote:

> On Mon, May 04, 2015 at 12:48:09PM +0200, Florian Westphal wrote:
> > Felix Janda <felix.janda@posteo.de> wrote:
> > > Current code makes the assumption that !defined(__GLIBC__) means libc5
> > > which is very unlikely the case nowadays.
> > > 
> > > Fixes compile error because of conflict between kernel and musl headers.
> > > ---
> > > If libc5 is considered still relevant, I could try to come up with an
> > > autoconf test.
> > 
> > I'm all for removing libc5 support if this is whats preventing iptables
> > to be built with current non-glibc systems.
> > 
> > Pablo, Patrick, Eric, Jozsef - whats your take on this?
> 
> I would say, go take it.

I second that.

Best regards,
Jozsef
-
E-mail  : kadlec@blackhole.kfki.hu, kadlecsik.jozsef@wigner.mta.hu
PGP key : http://www.kfki.hu/~kadlec/pgp_public_key.txt
Address : Wigner Research Centre for Physics, Hungarian Academy of Sciences
          H-1525 Budapest 114, POB. 49, Hungary
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Florian Westphal May 8, 2015, 2:04 p.m. UTC | #4
Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> wrote:
> > On Mon, May 04, 2015 at 12:48:09PM +0200, Florian Westphal wrote:
> > > Felix Janda <felix.janda@posteo.de> wrote:
> > > > Current code makes the assumption that !defined(__GLIBC__) means libc5
> > > > which is very unlikely the case nowadays.
> > > > 
> > > > Fixes compile error because of conflict between kernel and musl headers.
> > > > ---
> > > > If libc5 is considered still relevant, I could try to come up with an
> > > > autoconf test.
> > > 
> > > I'm all for removing libc5 support if this is whats preventing iptables
> > > to be built with current non-glibc systems.
> > > 
> > > Pablo, Patrick, Eric, Jozsef - whats your take on this?
> > 
> > I would say, go take it.
> 
> I second that.

I've applied this patch, thanks everyone.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/include/libiptc/ipt_kernel_headers.h b/include/libiptc/ipt_kernel_headers.h
index 18861fe..a5963e9 100644
--- a/include/libiptc/ipt_kernel_headers.h
+++ b/include/libiptc/ipt_kernel_headers.h
@@ -5,7 +5,6 @@ 
 
 #include <limits.h>
 
-#if defined(__GLIBC__) && __GLIBC__ == 2
 #include <netinet/ip.h>
 #include <netinet/in.h>
 #include <netinet/ip_icmp.h>
@@ -13,15 +12,4 @@ 
 #include <netinet/udp.h>
 #include <net/if.h>
 #include <sys/types.h>
-#else /* libc5 */
-#include <sys/socket.h>
-#include <linux/ip.h>
-#include <linux/in.h>
-#include <linux/if.h>
-#include <linux/icmp.h>
-#include <linux/tcp.h>
-#include <linux/udp.h>
-#include <linux/types.h>
-#include <linux/in6.h>
-#endif
 #endif