[PATCH 1/6] Disable IPv6 support for Haiku

Subsystems: kernel build + files below scripts/ (unless maintained elsewhere), the rest

DORMANTno replies REVIEWED: 2 (0M)

2 review trailers.

4 messages, 2 authors, 2016-06-15 · open the first message on its own page

[PATCH 1/6] Disable IPv6 support for Haiku

From: Andreas Färber <hidden>
Date: 2016-06-15 22:45:10

Signed-off-by: Andreas Faerber <redacted>
Acked-by: Ingo Weinhold <redacted>
Acked-by: Scott McCreary <redacted>
---
  Makefile |    3 +++
  1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/Makefile b/Makefile
index 53ab4b5..b22dbcf 100644
--- a/Makefile
+++ b/Makefile
@@ -728,6 +728,9 @@ ifeq ($(uname_S),HP-UX)
  	NO_HSTRERROR = YesPlease
  	NO_SYS_SELECT_H = YesPlease
  endif
+ifeq ($(uname_S),Haiku)
+	NO_IPV6 = YesPlease
+endif
  ifneq (,$(findstring MINGW,$(uname_S)))
  	NO_MMAP = YesPlease
  	NO_PREAD = YesPlease
-- 
1.6.0.rc3.32.g8aaa

Re: [PATCH 1/6] Disable IPv6 support for Haiku

From: Alex Riesen <hidden>
Date: 2016-06-15 22:45:10

Andreas Färber, Sun, Aug 17, 2008 10:56:56 +0200:
+ifeq ($(uname_S),Haiku)
+	NO_IPV6 = YesPlease
+endif
Does Haiku have no IPv6 support at all, or it is just the getaddrinfo
and the like functions which are missing?

Re: [PATCH 1/6] Disable IPv6 support for Haiku

From: Andreas Färber <hidden>
Date: 2016-06-15 22:45:10

Am 17.08.2008 um 22:33 schrieb Alex Riesen:
Andreas Färber, Sun, Aug 17, 2008 10:56:56 +0200:
quoted
+ifeq ($(uname_S),Haiku)
+	NO_IPV6 = YesPlease
+endif
Does Haiku have no IPv6 support at all, or it is just the getaddrinfo
and the like functions which are missing?
Haiku does not yet have complete IPv6 support afaik (would've been a  
GSoC project), it does have getaddrinfo though. Not setting NO_IPV6  
currently leads to compilation errors (redefinition of struct  
sockaddr_in), I assume that should be fixed on their side of things.

In the current series of patches I had not included a check for  
libnetwork in configure.ac, so that getaddrinfo does not get detected.  
Would you prefer to have it already detected, but to override it via  
the above Makefile line? (That works as well.) Or to have it detected  
but have the user manually pass NO_IPV6=YesPlease to avoid a future  
revert of that line?

Andreas

Re: [PATCH 1/6] Disable IPv6 support for Haiku

From: Alex Riesen <hidden>
Date: 2016-06-15 22:45:10

Andreas Färber, Mon, Aug 18, 2008 19:23:35 +0200:
Am 17.08.2008 um 22:33 schrieb Alex Riesen:
quoted
Andreas Färber, Sun, Aug 17, 2008 10:56:56 +0200:
quoted
+ifeq ($(uname_S),Haiku)
+	NO_IPV6 = YesPlease
+endif
Does Haiku have no IPv6 support at all, or it is just the getaddrinfo
and the like functions which are missing?
Haiku does not yet have complete IPv6 support afaik (would've been a  
GSoC project), it does have getaddrinfo though. Not setting NO_IPV6  
currently leads to compilation errors (redefinition of struct  
sockaddr_in), I assume that should be fixed on their side of things.
Ah, that's what it was...
In the current series of patches I had not included a check for  
libnetwork in configure.ac, so that getaddrinfo does not get detected.  
Would you prefer to have it already detected, but to override it via the 
above Makefile line? (That works as well.) Or to have it detected but 
have the user manually pass NO_IPV6=YesPlease to avoid a future revert of 
that line?
I think your patch as-is will be preferred (it just works).
It is not a big deal to change it later.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help