Page MenuHomePhabricator

Fix `arc` issues with non-English systems
AcceptedPublic

Authored by epriestley on Jul 13 2014, 6:43 PM.
Tags
None
Referenced Files
F12847273: D9922.id.diff
Fri, Mar 29, 2:01 AM
Unknown Object (File)
Feb 27 2024, 12:30 PM
Unknown Object (File)
Feb 27 2024, 12:21 PM
Unknown Object (File)
Feb 9 2024, 11:00 AM
Unknown Object (File)
Feb 7 2024, 11:54 PM
Unknown Object (File)
Feb 4 2024, 4:41 AM
Unknown Object (File)
Jan 31 2024, 3:35 AM
Unknown Object (File)
Jan 16 2024, 10:26 PM
Subscribers

Details

Summary

Fixes T5554. I can't get any of this stuff running in non-English, but I'll get some people reporting this issue to try it out.

Test Plan

seems to work idk??

Diff Detail

Repository
rARC Arcanist
Branch
lcall
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1699
Build 1700: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Fix `arc` issues with non-English systems.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Try using LANG instead.
  • Consolidate LANG logic from svn commit.
  • Fix an issue with passthru commands.
  • Don't set LANG for passthru commands.
btrahan edited edge metadata.

haha, <3 the testplan

This revision is now accepted and ready to land.Jul 17 2014, 10:31 PM

This doesn't break anything for English users but I'm waiting on confirmation from a non-English user that it solves something. It's way cleaner overall regardless, but it would be nice to fix this issue too. :P

@epriestley this change works for us in de_DE locales. It would be nice if this was merged.