From cd1cd1bd3350c85aee8f496f63522422dcad73de Mon Sep 17 00:00:00 2001 From: John Mulligan Date: Mon, 10 Oct 2022 10:21:02 -0400 Subject: [PATCH] install-deps: switch to C.UTF-8 locale 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 --- install-deps.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/install-deps.sh b/install-deps.sh index 0703143d6cf2c..3762b3de03565 100755 --- a/install-deps.sh +++ b/install-deps.sh @@ -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) -- 2.39.5