[packages/rpm-build-tools] LANG=C is not enough to disable translations, add LC_ALL=C everywhere

baggins baggins at pld-linux.org
Tue Feb 23 10:55:28 CET 2021


commit ae0dd942e0b1f73604de4bb1a2adea3dccaf8b8b
Author: Jan Rękorajski <baggins at pld-linux.org>
Date:   Tue Feb 23 10:54:50 2021 +0100

    LANG=C is not enough to disable translations, add LC_ALL=C everywhere

 builder.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/builder.sh b/builder.sh
index e0f1da3..f94b0a3 100755
--- a/builder.sh
+++ b/builder.sh
@@ -479,7 +479,7 @@ Usage: builder [--all-branches] [-D|--debug] [-V|--version] [--short-version]  [
 is_rpmorg() {
 	local v
 
-	v=$(LANG=C rpm --version 2>&1)
+	v=$(LC_ALL=C LANG=C rpm --version 2>&1)
 	v=${v#RPM version } # rpm 4
 	v=${v#rpm \(RPM\) } # rpm 5
 
@@ -1896,7 +1896,7 @@ run_sub_builder() {
 # this requires following sudo rules:
 # - poldek --noask --caplookup -ug
 poldek_install() {
-	LANG=C $POLDEK_CMD --noask --caplookup --uniq -ug "$@"
+	LC_ALL=C LANG=C $POLDEK_CMD --noask --caplookup --uniq -ug "$@"
 }
 
 # install packages
@@ -2025,7 +2025,7 @@ _rpm_cnfl_check() {
 		DEPS=$(cat)
 	fi
 
-	LANG=C rpm -q --whatprovides $DEPS 2>/dev/null | awk '!/no package provides/ { print }'
+	LC_ALL=C LANG=C rpm -q --whatprovides $DEPS 2>/dev/null | awk '!/no package provides/ { print }'
 }
 
 # install deps via information from 'rpm-getdeps' or 'rpm --specsrpm'
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/rpm-build-tools.git/commitdiff/f9a4422fb8aaa78480bf9cfa7a514c4e7ab6a4bb



More information about the pld-cvs-commit mailing list