Don't set LC_ALL=C during build as that breaks python apps
authorDaniel P. Berrange <berrange@redhat.com>
Tue, 25 Jul 2017 12:04:28 +0000 (13:04 +0100)
committerDaniel P. Berrange <berrange@redhat.com>
Tue, 25 Jul 2017 12:04:28 +0000 (13:04 +0100)
Setting LC_ALL=C breaks python apps doing I/O on UTF-8 source
files. In particular this broke glib-mkenums

  GEN      libvirt-gconfig-enum-types.h
Traceback (most recent call last):
  File "/usr/bin/glib-mkenums", line 669, in <module>
    process_file(fname)
  File "/usr/bin/glib-mkenums", line 406, in process_file
    line = curfile.readline()
  File "/usr/lib64/python3.6/encodings/ascii.py", line 26, in decode
    return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 849: ordinal not in range(128)

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

maint.mk

index 405c6d0..ef72b4f 100644 (file)
--- a/maint.mk
+++ b/maint.mk
@@ -117,8 +117,8 @@ news-check-lines-spec ?= 1,10
 news-check-regexp ?= '^\*.* $(VERSION_REGEXP) \($(today)\)'
 
 # Prevent programs like 'sort' from considering distinct strings to be equal.
-# Doing it here saves us from having to set LC_ALL elsewhere in this file.
-export LC_ALL = C
+# Doing it here saves us from having to set LC_COLLATE elsewhere in this file.
+export LC_COLLATE = C
 
 ## --------------- ##
 ## Sanity checks.  ##