Merge #19855: restore use of LC_ALL, not LC_MESSAGES

This commit is contained in:
R David Murray 2014-01-18 15:02:00 -05:00
commit bbf18bee3c
1 changed files with 4 additions and 3 deletions

View File

@ -321,9 +321,10 @@ def _find_mac(command, args, hw_identifiers, get_index):
return None return None
try: try:
# LC_MESSAGES to get English output, 2>/dev/null to # LC_ALL to ensure English output, 2>/dev/null to prevent output on
# prevent output on stderr # stderr (Note: we don't have an example where the words we search for
cmd = 'LC_MESSAGES=C %s %s 2>/dev/null' % (executable, args) # are actually localized, but in theory some system could do so.)
cmd = 'LC_ALL=C %s %s 2>/dev/null' % (executable, args)
with os.popen(cmd) as pipe: with os.popen(cmd) as pipe:
for line in pipe: for line in pipe:
words = line.lower().split() words = line.lower().split()