diff mbox

ipset: Don't include host headers

Message ID c69b0ea4caf640e9ca3c2357f6e701cab02fa689.1414449206.git.baruch@tkos.co.il
State Accepted
Headers show

Commit Message

Baruch Siach Oct. 27, 2014, 10:33 p.m. UTC
Add a patch removing /usr/local/include from the include path.

Fixes:
http://autobuild.buildroot.net/results/4f5/4f55c2415281c6204500efe28fe9e24c8ef73863/

Technically speaking this autobuild failure has already been fixes when
-Werror was removed in commit a923b44112aa3 (ipset: add patch to fix
--enable-debug build). But the warning in this case indicates a real problem
that needs to be fixed by itself.

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 ...Make_global.am-Don-t-include-host-headers.patch | 29 ++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch

Comments

Thomas Petazzoni Oct. 28, 2014, 5:57 p.m. UTC | #1
Dear Baruch Siach,

On Tue, 28 Oct 2014 00:33:26 +0200, Baruch Siach wrote:
> Add a patch removing /usr/local/include from the include path.
> 
> Fixes:
> http://autobuild.buildroot.net/results/4f5/4f55c2415281c6204500efe28fe9e24c8ef73863/
> 
> Technically speaking this autobuild failure has already been fixes when
> -Werror was removed in commit a923b44112aa3 (ipset: add patch to fix
> --enable-debug build). But the warning in this case indicates a real problem
> that needs to be fixed by itself.
> 
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  ...Make_global.am-Don-t-include-host-headers.patch | 29 ++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>  create mode 100644 package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch

Thanks, applied. Can you please submit the patch upstream?

Thomas
Baruch Siach Oct. 28, 2014, 7:38 p.m. UTC | #2
Hi Thomas,

On Tue, Oct 28, 2014 at 06:57:25PM +0100, Thomas Petazzoni wrote:
> On Tue, 28 Oct 2014 00:33:26 +0200, Baruch Siach wrote:
> > Add a patch removing /usr/local/include from the include path.
> > 
> > Fixes:
> > http://autobuild.buildroot.net/results/4f5/4f55c2415281c6204500efe28fe9e24c8ef73863/
> > 
> > Technically speaking this autobuild failure has already been fixes when
> > -Werror was removed in commit a923b44112aa3 (ipset: add patch to fix
> > --enable-debug build). But the warning in this case indicates a real problem
> > that needs to be fixed by itself.
> > 
> > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> > ---
> >  ...Make_global.am-Don-t-include-host-headers.patch | 29 ++++++++++++++++++++++
> >  1 file changed, 29 insertions(+)
> >  create mode 100644 package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch
> 
> Thanks, applied. Can you please submit the patch upstream?

Done (http://marc.info/?l=netfilter-devel&m=141452470614763&w=2).

baruch
diff mbox

Patch

diff --git a/package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch b/package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch
new file mode 100644
index 000000000000..f8809791d3ff
--- /dev/null
+++ b/package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch
@@ -0,0 +1,29 @@ 
+From 1f1b5ed5e8402214a6516a380247123e6dd75d2a Mon Sep 17 00:00:00 2001
+From: Baruch Siach <baruch@tkos.co.il>
+Date: Tue, 28 Oct 2014 00:19:25 +0200
+Subject: [PATCH] Make_global.am: Don't include host headers
+
+This is bad for cross compilation.
+
+Signed-off-by: Baruch Siach <baruch@tkos.co.il>
+---
+ Make_global.am | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+diff --git a/Make_global.am b/Make_global.am
+index 25c4d64715c7..11ea1c333186 100644
+--- a/Make_global.am
++++ b/Make_global.am
+@@ -71,8 +71,7 @@
+ #            curr:rev:age
+ LIBVERSION = 8:0:5
+ 
+-AM_CPPFLAGS = $(kinclude_CFLAGS) $(all_includes) -I$(top_srcdir)/include \
+-	-I/usr/local/include
++AM_CPPFLAGS = $(kinclude_CFLAGS) $(all_includes) -I$(top_srcdir)/include
+ 
+ AM_CFLAGS = -std=gnu99
+ 
+-- 
+2.1.1
+