diff options
author | Mike Frysinger <vapier@chromium.org> | 2015-10-09 17:47:40 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-10-12 10:36:15 +0200 |
commit | 99dc70f023cf118f1e23125301d9fd38fb805f6d (patch) | |
tree | 64172c5ff01a3d356db10f907ff37553098b49e2 /src/conf.c | |
parent | 08c4b3b0adf1ab0c920dde5ffe5f914785686e54 (diff) | |
download | alsa-lib-99dc70f023cf118f1e23125301d9fd38fb805f6d.tar.gz |
fix _GNU_SOURCE handling & header inclusion
The configure script blindly adds -D_GNU_SOURCE to all build settings,
even on non-GNU systems. This isn't too much of a big deal (even if
it uses the wrong variable -- CFLAGS instead of CPPFLAGS), except that
the alsa-lib source itself determines whether to use GNU features when
this is defined (such as versionsort). So when we build on non-glibc
systems, we get build failures like:
src/ucm/parser.c:1268:18: error: 'versionsort' undeclared (first use in this function)
#define SORTFUNC versionsort
^
src/ucm/parser.c:1272:54: note: in expansion of macro 'SORTFUNC'
err = scandir(filename, &namelist, filename_filter, SORTFUNC);
^
The correct way to add these flags is to use the autoconf helper
AC_USE_SYSTEM_EXTENSIONS. Unfortunately, that triggers some more
bugs in the alsa build. This macro adds defines to config.h and
not directly to CPPFLAGS, so it relies on files correctly including
config.h before anything else. A number of alsa files do not do
this leading to build failures. The fix there is to shuffle the
includes around so that the local ones come first.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'src/conf.c')
-rw-r--r-- | src/conf.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -414,12 +414,12 @@ beginning:</P> */ +#include "local.h" #include <stdarg.h> #include <limits.h> #include <sys/stat.h> #include <dirent.h> #include <locale.h> -#include "local.h" #ifdef HAVE_LIBPTHREAD #include <pthread.h> #endif |