All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/lxc: add patch to fix musl build
@ 2016-05-20  5:15 Jörg Krause
  2016-05-24 18:50 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Jörg Krause @ 2016-05-20  5:15 UTC (permalink / raw)
  To: buildroot

Add a patch to fix a build issue with the musl C library.

struct in6_addr is both defined in the C library header <netinet/in.h> and the
Linux kernel header <linux/in6.h>.

lxc_user_nic.c includes both <netinet/in.h> and <linux/if_bridge.h>. The later
one includes <linux/in6.h>.

This breaks build with the musl libc:
  error: redefinition of ?struct in6_addr?

As lxc_user_nic.c does not use any references from <linux/if_bridge.h> it is
safe to remove this header.

Fixes:
http://autobuild.buildroot.net/results/f32/f321823be6b477be7dc55393e563a3a61794265d/
http://autobuild.buildroot.net/results/8ab/8ab2dc8e0634fbfb582db94cdf94cee5712711f9/
http://autobuild.buildroot.net/results/e27/e27d637b1bb7278ae30d86a72a2d88432d805d2c/
.. and more.

Upstream status: Pending
https://github.com/lxc/lxc/pull/1029

Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
---
 .../0002-Fix-redefinition-of-struct-in6_addr.patch | 43 ++++++++++++++++++++++
 1 file changed, 43 insertions(+)
 create mode 100644 package/lxc/0002-Fix-redefinition-of-struct-in6_addr.patch

diff --git a/package/lxc/0002-Fix-redefinition-of-struct-in6_addr.patch b/package/lxc/0002-Fix-redefinition-of-struct-in6_addr.patch
new file mode 100644
index 0000000..b48ece3
--- /dev/null
+++ b/package/lxc/0002-Fix-redefinition-of-struct-in6_addr.patch
@@ -0,0 +1,43 @@
+From 245bba9aadf8e7aea487b6fbd851f86c75524552 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?J=C3=B6rg=20Krause?= <joerg.krause@embedded.rocks>
+Date: Thu, 19 May 2016 21:51:27 +0200
+Subject: [PATCH] Fix redefinition of struct in6_addr
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+struct in6_addr is both defined in the C library header <netinet/in.h>
+and the Linux kernel header <linux/in6.h>.
+
+lxc_user_nic.c includes both <netinet/in.h> and <linux/if_bridge.h>. The
+later one includes <linux/in6.h>.
+
+This breaks build with the musl libc:
+  error: redefinition of ?struct in6_addr?
+
+As lxc_user_nic.c does not use any references from <linux/if_bridge.h> it
+is safe to remove this header.
+
+Upstream status: Pending
+https://github.com/lxc/lxc/pull/1029
+
+Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
+---
+ src/lxc/lxc_user_nic.c | 1 -
+ 1 file changed, 1 deletion(-)
+
+diff --git a/src/lxc/lxc_user_nic.c b/src/lxc/lxc_user_nic.c
+index 87780ca..0cb38ba 100644
+--- a/src/lxc/lxc_user_nic.c
++++ b/src/lxc/lxc_user_nic.c
+@@ -41,7 +41,6 @@
+ #include <net/if.h>
+ #include <net/if_arp.h>
+ #include <netinet/in.h>
+-#include <linux/if_bridge.h>
+ #include <linux/netlink.h>
+ #include <linux/rtnetlink.h>
+ #include <linux/sockios.h>
+-- 
+2.8.2
+
-- 
2.8.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] package/lxc: add patch to fix musl build
  2016-05-20  5:15 [Buildroot] [PATCH] package/lxc: add patch to fix musl build Jörg Krause
@ 2016-05-24 18:50 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2016-05-24 18:50 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 20 May 2016 07:15:48 +0200, J?rg Krause wrote:
> Add a patch to fix a build issue with the musl C library.
> 
> struct in6_addr is both defined in the C library header <netinet/in.h> and the
> Linux kernel header <linux/in6.h>.
> 
> lxc_user_nic.c includes both <netinet/in.h> and <linux/if_bridge.h>. The later
> one includes <linux/in6.h>.
> 
> This breaks build with the musl libc:
>   error: redefinition of ?struct in6_addr?
> 
> As lxc_user_nic.c does not use any references from <linux/if_bridge.h> it is
> safe to remove this header.
> 
> Fixes:
> http://autobuild.buildroot.net/results/f32/f321823be6b477be7dc55393e563a3a61794265d/
> http://autobuild.buildroot.net/results/8ab/8ab2dc8e0634fbfb582db94cdf94cee5712711f9/
> http://autobuild.buildroot.net/results/e27/e27d637b1bb7278ae30d86a72a2d88432d805d2c/
> .. and more.
> 
> Upstream status: Pending
> https://github.com/lxc/lxc/pull/1029
> 
> Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
> ---
>  .../0002-Fix-redefinition-of-struct-in6_addr.patch | 43 ++++++++++++++++++++++
>  1 file changed, 43 insertions(+)
>  create mode 100644 package/lxc/0002-Fix-redefinition-of-struct-in6_addr.patch

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-05-24 18:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-20  5:15 [Buildroot] [PATCH] package/lxc: add patch to fix musl build Jörg Krause
2016-05-24 18:50 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.