feature test macros: make _GNU_SOURCE enable everything

previously, a few BSD features were enabled only by _BSD_SOURCE, not
by _GNU_SOURCE. since _BSD_SOURCE is default in the absence of other
feature test macros, this made adding _GNU_SOURCE to a project not a
purely additive feature test macro; it actually caused some features
to be suppressed.

most of the changes made by this patch actually bring musl in closer
alignment with the glibc behavior for _GNU_SOURCE. the only exceptions
are the added visibility of functions like strlcpy which were BSD-only
due to being disliked/rejected by glibc maintainers. here, I feel the
consistency of having _GNU_SOURCE mean "everything", and especially
the property of it being purely additive, are more valuable than
hiding functions which glibc does not have.

diff --git a/include/dirent.h b/include/dirent.h
index 726067f6afa49f4c856e6647e346b2a011c134fa..d445f80a26b7f112528f93fa8637f966f73d9b39 100644 (file)
--- a/include/dirent.h
+++ b/include/dirent.h
@@ -9,7 +9,7 @@ extern "C" {
 
 #define __NEED_ino_t
 #define __NEED_off_t
-#ifdef _BSD_SOURCE
+#if defined(_BSD_SOURCE) || defined(_GNU_SOURCE)
 #define __NEED_size_t
 #endif
 
@@ -53,16 +53,13 @@ int scandir(const char *, struct dirent ***, int (*)(const struct dirent *), int
 #define DT_WHT 14
 #define IFTODT(x) ((x)>>12 & 017)
 #define DTTOIF(x) ((x)<<12)
+int getdents(int, struct dirent *, size_t);
 #endif
 
 #ifdef _GNU_SOURCE
 int versionsort(const struct dirent **, const struct dirent **);
 #endif
 
-#ifdef _BSD_SOURCE
-int getdents(int, struct dirent *, size_t);
-#endif
-
 #if defined(_LARGEFILE64_SOURCE) || defined(_GNU_SOURCE)
 #define dirent64 dirent
 #define readdir64 readdir
@@ -72,10 +69,8 @@ int getdents(int, struct dirent *, size_t);
 #define versionsort64 versionsort
 #define off64_t off_t
 #define ino64_t ino_t
-#ifdef _BSD_SOURCE
 #define getdents64 getdents
 #endif
-#endif
 
 #ifdef __cplusplus
 }

diff --git a/include/signal.h b/include/signal.h
index 89080ab6acda164314c9736e54a968bf52ff5bd6..b24f43a5c57c721f40631192872a7f4b2d445ac9 100644 (file)
--- a/include/signal.h
+++ b/include/signal.h
@@ -212,7 +212,7 @@ void (*sigset(int, void (*)(int)))(int);
 #define NSIG _NSIG
 #endif
 
-#ifdef _BSD_SOURCE
+#if defined(_BSD_SOURCE) || defined(_GNU_SOURCE)
 typedef void (*sig_t)(int);
 #endif
 

diff --git a/include/stdio.h b/include/stdio.h
index 9a2093763505dc9bbd6444dd86229a629ca6b8a1..6e8e64544d97d6e0747aa3adb3125b89aa69a976 100644 (file)
--- a/include/stdio.h
+++ b/include/stdio.h
@@ -174,9 +174,6 @@ int ferror_unlocked(FILE *);
 int fileno_unlocked(FILE *);
 int getw(FILE *);
 int putw(int, FILE *);
-#endif
-
-#ifdef _BSD_SOURCE
 char *fgetln(FILE *, size_t *);
 #endif
 

diff --git a/include/string.h b/include/string.h
index 33f013782a4a5fcdf7ec81c6463461352e01e7b4..c2f8eb558c502a5b36b6dd896c89c4a6b2f4b67e 100644 (file)
--- a/include/string.h
+++ b/include/string.h
@@ -81,9 +81,6 @@ void *memccpy (void *__restrict, const void *__restrict, int, size_t);
 
 #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE)
 char *strsep(char **, const char *);
-#endif
-
-#ifdef _BSD_SOURCE
 size_t strlcat (char *, const char *, size_t);
 size_t strlcpy (char *, const char *, size_t);
 #endif

diff --git a/include/sys/stat.h b/include/sys/stat.h
index c63c6b8b6f72dcd651de61dba650837ac5def922..c6abab5a0c01f58746def40c29cc32cfe976c27e 100644 (file)
--- a/include/sys/stat.h
+++ b/include/sys/stat.h
@@ -88,11 +88,8 @@ int mkfifoat(int, const char *, mode_t);
 int futimens(int, const struct timespec [2]);
 int utimensat(int, const char *, const struct timespec [2], int);
 
-#ifdef _BSD_SOURCE
-int lchmod(const char *, mode_t);
-#endif
-
 #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE)
+int lchmod(const char *, mode_t);
 #define S_IREAD S_IRUSR
 #define S_IWRITE S_IWUSR
 #define S_IEXEC S_IXUSR