Merge: tests: fix locale issue on non-Debian linux
authorJean Privat <jean@pryen.org>
Fri, 6 Apr 2018 02:04:36 +0000 (22:04 -0400)
committerJean Privat <jean@pryen.org>
Fri, 6 Apr 2018 02:04:36 +0000 (22:04 -0400)
The C.UTF-8 locale is Debian-specific and causes issues on other
platforms.

To avoid setlocale error messages, the LC_ALL export will default on C
if C.UTF-8 is unavailable on the target platform.

Signed-off-by: Lucas Bajolet <lucas.bajolet@gmail.com>

Pull-Request: #2628

tests/tests.sh

index 46cb74f..9eda24c 100755 (executable)
 
 # Set lang do default to avoid failed tests because of locale
 export LANG=C.UTF-8
-export LC_ALL=C.UTF-8
+
+# Explore candidates for LC_ALL
+#
+# If C.UTF-8, the most agnostic UTF-8 locale is supported, we
+# use it, otherwise we default to the UTF-8 system locale, and
+# if unavailable, we default on C.
+#
+# Note that C however is not guaranteed to be UTF-8, and
+# some tests may fail when relying on Unicode semantics.
+locale_candidate=C.UTF-8
+if ! locale -a 2>/dev/null | grep -q C.UTF-8; then
+       locale_candidate="$(locale -a | grep -i utf8 | head -n1)"
+       if [ -z "$locale_candidate" ]; then
+               locale_candidate=C
+       fi
+fi
+export LC_ALL="$locale_candidate"
+
 if uname | grep Darwin 1>/dev/null 2>&1; then
        export LANG=en_US.UTF-8
        export LC_ALL=en_US.UTF-8