[MPlayer-dev-eng] [PATCH] Get rid of -D_GNU_SOURCE

Diego Biurrun diego at biurrun.de
Fri Dec 8 14:35:19 CET 2006


On Fri, Dec 08, 2006 at 10:15:30AM +0100, Reimar Doeffinger wrote:
> it seems to be unneeded, so this patch removes it. sdl-config once again
> is again a prime example why it is not a good idea to trust such scripts that
> create cflags/ldflags for you, given the amount of hacks we need
> currently to use it, not using it might even be the less complex
> approach...

I think I'm generally in favor of this, but ..

> --- libfaad2/Makefile	(revision 21535)
> +++ libfaad2/Makefile	(working copy)
> @@ -42,8 +42,6 @@
>  
> -CFLAGS  = -D_GNU_SOURCE
> -
> --- dvdread/Makefile	(revision 21535)
> +++ dvdread/Makefile	(working copy)
> @@ -11,7 +11,7 @@
>  
> -CFLAGS= -D__USE_UNIX98 -D_GNU_SOURCE -DSTDC_HEADERS \
> +CFLAGS= -D__USE_UNIX98 -DSTDC_HEADERS \
>  	-DHAVE_LIMITS_H -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64
>  
> --- libdvdcss/Makefile	(revision 21535)
> +++ libdvdcss/Makefile	(working copy)
> @@ -9,7 +9,7 @@
>  
> -CFLAGS= -D__USE_UNIX98 -D_GNU_SOURCE \
> +CFLAGS= -D__USE_UNIX98 \
>  	-DHAVE_LIMITS_H -DHAVE_ERRNO_H -DHAVE_INTTYPES_H -DHAVE_UNISTD_H \
>  	-D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -DVERSION=\"1.2.9\"

.. for external libraries I'm not so sure.  If they are compiled with
_GNU_SOURCE upstream then I think we should use the same configuration.

Diego



More information about the MPlayer-dev-eng mailing list