]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
install-deps: switch to C.UTF-8 locale 48425/head
authorJohn Mulligan <jmulligan@redhat.com>
Mon, 10 Oct 2022 14:21:02 +0000 (10:21 -0400)
committerJohn Mulligan <jmulligan@redhat.com>
Sun, 16 Oct 2022 15:50:50 +0000 (11:50 -0400)
The en_US.UTF-8 locale is not always going to present on all systems.
Specifically, setting en_US.UTF-8 causes numerous warnings when running
this script on a CentOS Stream 8 container. Rather than install
additional packages to suppress the warning, use the C.UTF-8 locale.

Additionally, try to update the comment explaining the line to something
based on the commit history rather than the confusing term "vulnerable".

Signed-off-by: John Mulligan <jmulligan@redhat.com>
install-deps.sh

index 0703143d6cf2c96cad0bfe01ed36e660e4617bf2..3762b3de03565828c988c8c3ac96cfccc3ad7a08 100755 (executable)
@@ -18,7 +18,9 @@ mkdir -p $DIR
 if test $(id -u) != 0 ; then
     SUDO=sudo
 fi
-export LC_ALL=en_US.UTF-8 # the following is vulnerable to i18n
+# enable UTF-8 encoding for programs like pip that expect to
+# print more than just ascii chars
+export LC_ALL=C.UTF-8
 
 ARCH=$(uname -m)