cURL / Mailing Lists / curl-library / Single Mail

curl-library

curl.h: <sys/select.h> for OpenBSD

From: Christian Weisgerber <naddy_at_mips.inka.de>
Date: Wed, 4 Dec 2013 16:45:50 +0100

curl.h should also include <sys/select.h> on OpenBSD to reliably
pull in select(). Typically, including <sys/time.h> will be enough,
but not if strict standards-compliance is requested (e.g. by defining
_XOPEN_SOURCE).

I think the FreeBSD version check is also dubious. It's true that
FreeBSD 8.0+ pulls in <sys/select.h> from <sys/time.h>, so explicitly
including select.h isn't _strictly_ necessary, but again, this means
relying on a compatibility fallback. <sys/select.h> is the documented
way to get select() on FreeBSD.

--- include/curl/curl.h.orig Fri Sep 27 10:57:00 2013
+++ include/curl/curl.h Wed Dec 4 14:22:58 2013
@@ -69,7 +69,7 @@
    require it! */
 #if defined(_AIX) || defined(__NOVELL_LIBC__) || defined(__NetBSD__) || \
     defined(__minix) || defined(__SYMBIAN32__) || defined(__INTEGRITY) || \
- defined(ANDROID) || defined(__ANDROID__) || \
+ defined(ANDROID) || defined(__ANDROID__) || defined(__OpenBSD__) || \
    (defined(__FreeBSD_version) && (__FreeBSD_version < 800000))
 #include <sys/select.h>
 #endif

-- 
Christian "naddy" Weisgerber                          naddy_at_mips.inka.de
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2013-12-04