[vlc-devel] [PATCH] contrib: live555: prevent inclusion of xlocale.h on linux

Filip Roséen filip at atch.se
Mon Jul 16 05:29:06 CEST 2018


https://sourceware.org/glibc/wiki/Release/2.26#Removal_of_.27xlocale.h.27

Also, as stated in the above notes, xlocale.h was never meant to be
included, so live555 is in the wrong for even trying.

--

I do not know of any LINUX system where xlocale.h is reliably around,
nor do I know of a system where <locale.h> would "not include"
relevant contents from the stated <xlocale.h>.

contrib/.../live555/liveMedia/include/Locale.hh:47:
        #include <xlocale.h> // because, on some systems, <locale.h>
                                doesn't include <xlocale.h>; this makes
                                sure that we get both
---
 contrib/src/live555/rules.mak | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/contrib/src/live555/rules.mak b/contrib/src/live555/rules.mak
index 3093dedec6..fa1868bb15 100644
--- a/contrib/src/live555/rules.mak
+++ b/contrib/src/live555/rules.mak
@@ -19,9 +19,12 @@ $(TARBALLS)/$(LIVE555_FILE):
 
 .sum-live555: $(LIVE555_FILE)
 
+LIVE_EXTRA_CFLAGS := $(EXTRA_CFLAGS) -fexceptions $(CFLAGS)
+
 LIVE_TARGET = $(error live555 target not defined!)
 ifdef HAVE_LINUX
 LIVE_TARGET := linux
+LIVE_EXTRA_CFLAGS += -DXLOCALE_NOT_USED
 endif
 ifdef HAVE_WIN32
 LIVE_TARGET := mingw
@@ -41,8 +44,6 @@ LIVE_TARGET := solaris-32bit
 endif
 endif
 
-LIVE_EXTRA_CFLAGS := $(EXTRA_CFLAGS) -fexceptions $(CFLAGS)
-
 live555: $(LIVE555_FILE) .sum-live555
 	rm -Rf live && $(UNPACK)
 
-- 
2.18.0



More information about the vlc-devel mailing list