Skip to contentSkip to conversation

ipset: Don't include host headers

PATCH New Baruch Siach 2 participants 3 messages Oct 27, 2014 – Oct 28, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ipset: Don't include host headers

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.

---
 ...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
package/ipset/0002-Make_global.am-Don-t-include-host-headers.patch new file +29 -0
new file mode 100644
index 000000000000..f8809791d3ff
@@ -0,0 +1,29 @@
+From 1f1b5ed5e8402214a6516a380247123e6dd75d2a Mon Sep 17 00:00:00 2001
+From: Baruch Siach <[email protected]>
+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 <[email protected]>
+---
+ 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
+
Signature · 1 line
2.1.1
#1 lore raw

PATCH 1/1 ipset: Don't include host headers

Dear Baruch Siach,

Quoted 10 lines · Baruch Siach

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.

Quoted 5 lines
---
 ...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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 ipset: Don't include host headers

Hi Thomas,

Quoted 11 lines · Thomas Petazzoni
Quoted 10 lines · Baruch Siach

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.

Quoted 7 lines
Quoted 5 lines
---
 ...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

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -