revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-20 13:52:40 +01:00
Code Issues Releases Wiki Activity

Use "en_US.UTF-8", not "C", as the LANG setting

Browse source 
Summary: `LANG=C` is smooshing UTF-8 in some cases. See IRC.

Test Plan: User confirmed this works.

Reviewers: btrahan, asherkin

Reviewed By: asherkin

CC: aran

Differential Revision: https://secure.phabricator.com/D7659
This commit is contained in:
epriestley 2013-11-26 08:50:36 -08:00
parent dd4a4a8c45
commit 209861500f
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/applications/repository/storage/PhabricatorRepository.php
View file

@ -294,10 +294,10 @@ final class PhabricatorRepository extends PhabricatorRepositoryDAO
private function getCommonCommandEnvironment() { private function getCommonCommandEnvironment() {
$env = array( $env = array(
// NOTE: Force the language to "C", which overrides locale settings. // NOTE: Force the language to "en_US.UTF-8", which overrides locale
// This makes stuff print in English instead of, e.g., French, so we can // settings. This makes stuff print in English instead of, e.g., French,
// parse the output of some commands, error messages, etc. // so we can parse the output of some commands, error messages, etc.
'LANG' => 'C', 'LANG' => 'en_US.UTF-8',
// Propagate PHABRICATOR_ENV explicitly. For discussion, see T4155. // Propagate PHABRICATOR_ENV explicitly. For discussion, see T4155.
'PHABRICATOR_ENV' => PhabricatorEnv::getSelectedEnvironmentName(), 'PHABRICATOR_ENV' => PhabricatorEnv::getSelectedEnvironmentName(),