pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/14624


Change subject: configure.ac: Workaround some linux/if.h headers requiring 
external pre-include of sys/socket.h
......................................................................

configure.ac: Workaround some linux/if.h headers requiring external pre-include 
of sys/socket.h

Related: OS#3230
Change-Id: Iba869a75745cea01024fa3ce04917c02fa608a13
---
M configure.ac
1 file changed, 6 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ggsn refs/changes/24/14624/1

diff --git a/configure.ac b/configure.ac
index 5581c32..7b985b9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -74,8 +74,12 @@
 AC_HEADER_SYS_WAIT
 AC_CHECK_HEADERS([arpa/inet.h fcntl.h netdb.h netinet/in.h stdint.h stdlib.h 
string.h sys/ioctl.h sys/socket.h sys/time.h unistd.h])

-# Check for if header
-AC_CHECK_HEADERS([linux/if.h net/if.h])
+# Check for if header. Some versions of linux/if.h fail without sys/socket.h 
included beforehand:
+# see 
https://algorithmicallyrandom.blogspot.com/2012/07/error-on-including-include.html
+AC_CHECK_HEADERS([linux/if.h net/if.h], [], [], [#ifdef HAVE_SYS_SOCKET_H
+                                                # include <sys/socket.h>
+                                                # endif
+                                                ])

 # Check for tun header
 AC_CHECK_HEADERS([linux/if_tun.h net/if_tun.h])

--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/14624
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Iba869a75745cea01024fa3ce04917c02fa608a13
Gerrit-Change-Number: 14624
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange

Reply via email to