includes/Rts.h

@@ -29,12 +29,6 @@ extern "C" {
 #include <windows.h>
 #endif
 
-#if defined(ios_HOST_OS) || defined(darwin_HOST_OS)
-/* Inclusion of system headers usually requires _DARWIN_C_SOURCE on Mac OS X
- * because of some specific defines like MMAP_ANON, MMAP_ANONYMOUS. */
-#define _DARWIN_C_SOURCE 1
-#endif
-
 #if !defined(IN_STG_CODE)
 #define IN_STG_CODE 0
 #endif

rts/LinkerInternals.h

@@ -13,11 +13,6 @@
 #include "linker/M32Alloc.h"
 
 #if RTS_LINKER_USE_MMAP
-#if defined(ios_HOST_OS) || defined(darwin_HOST_OS)
-/* Inclusion of system headers usually requires _DARWIN_C_SOURCE on Mac OS X
- * because of some specific defines like MMAP_ANON, MMAP_ANONYMOUS. */
-#define _DARWIN_C_SOURCE 1
-#endif
 #include <sys/mman.h>
 #endif
 

rts/posix/Itimer.c

@@ -18,6 +18,27 @@
  */
 
 #include "PosixSource.h"
+
+/* We've defined _POSIX_SOURCE via "PosixSource.h", and yet still use
+   some non-POSIX features.  With _POSIX_SOURCE defined, visibility of
+   non-POSIX extension prototypes requires _DARWIN_C_SOURCE on Mac OS X and
+   __BSD_VISIBLE on FreeBSD and DragonflyBSD.  Otherwise, for example, code
+   using pthread_setname_np(3) and variants will not compile.  We must
+   therefore define the additional macros that expose non-POSIX APIs early,
+   before any of the relevant system headers are included via "Rts.h".
+
+   An alternative approach could be to write portable wrappers or stubs for all
+   the non-posix functions in a C-module that does not include "PosixSource.h",
+   and then use only POSIX features and the portable wrapper functions in all
+   other C-modules. */
+#include "ghcconfig.h"
+#if defined(freebsd_HOST_OS) || defined(dragonfly_HOST_OS)
+#define __BSD_VISIBLE   1
+#endif
+#if defined(darwin_HOST_OS)
+#define _DARWIN_C_SOURCE 1
+#endif
+
 #include "Rts.h"
 
 /*

rts/posix/OSThreads.c

@@ -9,14 +9,23 @@
 
 #include "PosixSource.h"
 
+/* We've defined _POSIX_SOURCE via "PosixSource.h", and yet still use
+   some non-POSIX features.  With _POSIX_SOURCE defined, visibility of
+   non-POSIX extension prototypes requires _DARWIN_C_SOURCE on Mac OS X and
+   __BSD_VISIBLE on FreeBSD and DragonflyBSD.  Otherwise, for example, code
+   using pthread_setname_np(3) and variants will not compile.  We must
+   therefore define the additional macros that expose non-POSIX APIs early,
+   before any of the relevant system headers are included via "Rts.h".
+
+   An alternative approach could be to write portable wrappers or stubs for all
+   the non-posix functions in a C-module that does not include "PosixSource.h",
+   and then use only POSIX features and the portable wrapper functions in all
+   other C-modules. */
+#include "ghcconfig.h"
 #if defined(freebsd_HOST_OS) || defined(dragonfly_HOST_OS)
-/* Inclusion of system headers usually requires __BSD_VISIBLE on FreeBSD and
- * DragonflyBSD, because of some specific types, like u_char, u_int, etc. */
 #define __BSD_VISIBLE   1
 #endif
 #if defined(darwin_HOST_OS)
-/* Inclusion of system headers usually requires _DARWIN_C_SOURCE on Mac OS X
- * because of some specific types like u_char, u_int, etc. */
 #define _DARWIN_C_SOURCE 1
 #endif