office-gobmx/sal
Stephan Bergmann 56bc0b1a37 Don't set locale env vars on macOS
macosx_getLocale (via getProcessLocale, both sal/osl/unx/osxlocale.cxx) obtains
from the system a locale string like "en-DE".  (Whether that is even a sane and
recommended way to obtain the system locale on macOS I don't know; but lets
leave it at that for now.)

However, setting a locale env var (LANG, LC_ALL) to such a value causes a
constructor call std::locale("") to throw a std::runtime_error
"collate_byname<char>::collate_byname failed to construct for ", at least on
macOS 10.14.4.  And libdivvun (which might be bundled with a LO extension) is
known to be hit by that, see <https://github.com/divvun/libdivvun/issues/28>
"locale("") gives 'collate_byname<char>::collate_byname failed to construct
for ' on LO on mac".

The code setting LC_ALL/LC_CTYPE/LANG was there ever since
8737d1831b "INTEGRATION:
CWS geordi2q14: #111934#: merge CWS ooo111fix2", but for unclear reasons.  Lets
assume that it had no purpose (any longer).

Change-Id: I0b519ad567a713d61f662aa984791db1a91c708c
Reviewed-on: https://gerrit.libreoffice.org/70918
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-04-18 10:51:25 +02:00
..
android android: Make the tiled rendering from the loolwsd actually work. 2019-04-13 08:13:06 +02:00
cppunittester
inc
osl Don't set locale env vars on macOS 2019-04-18 10:51:25 +02:00
qa loplugin:sequentialassign in sal 2019-04-13 21:14:34 +02:00
rtl loplugin:sequentialassign in sal 2019-04-13 21:14:34 +02:00
test
textenc
util
CppunitTest_Module_DLL.mk
CppunitTest_sal_comtools.mk
CppunitTest_sal_osl.mk
CppunitTest_sal_osl_security.mk
CppunitTest_sal_rtl.mk
CppunitTest_sal_types.mk
Executable_cppunittester.mk
Executable_osl_process_child.mk
Library_lo-bootstrap.mk
Library_sal.mk
Library_sal_textenc.mk
Makefile
Module_sal.mk
README

System abstraction layer; rtl, osl and sal

rtl:
Platform independent strings

osl:
platform specific stuff, threads, dynamic loading, process, ipc, etc

Exports only C API and some inline-methods (only C++ API).