Paths

Table of Contentst

Differential D9922

Fix `arc` issues with non-English systems
AcceptedPublic
Actions

Authored by epriestley on Jul 13 2014, 6:43 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 19, 3:25 AM

	Unknown Object (File)
	Thu, Nov 14, 5:28 PM

	Unknown Object (File)
	Mon, Nov 11, 1:52 AM

	Unknown Object (File)
	Thu, Nov 7, 2:27 PM

	Unknown Object (File)
	Sun, Nov 3, 8:47 PM

	Unknown Object (File)
	Fri, Nov 1, 3:23 PM

	Unknown Object (File)
	Mon, Oct 28, 3:53 AM

	Unknown Object (File)
	Oct 22 2024, 5:25 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T5554: arc fails with svn/git/hg non english locales

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

Branch

lcall

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1684
Build 1685: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 23816.Jul 13 2014, 6:43 PM

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.

epriestley added a task: T5554: arc fails with svn/git/hg non english locales.

Herald added a subscriber: epriestley. · View Herald TranscriptJul 13 2014, 6:43 PM

Harbormaster completed remote builds in B1684: Diff 23816.Jul 13 2014, 6:44 PM

Try using LANG instead.
Consolidate LANG logic from svn commit.
Fix an issue with passthru commands.

Harbormaster completed remote builds in B1699: Diff 23839.Jul 15 2014, 4:39 PM

Don't set LANG for passthru commands.

Harbormaster completed remote builds in B1702: Diff 23844.Jul 15 2014, 9:11 PM

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

btrahan mentioned this in T5554: arc fails with svn/git/hg non english locales.Oct 17 2014, 4:57 PM

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

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistGitAPI.php

35 lines

ArcanistMercurialAPI.php

29 lines

ArcanistRepositoryAPI.php

53 lines

ArcanistSubversionAPI.php

9 lines

Diff 23816

src/repository/api/ArcanistGitAPI.php

Loading...

src/repository/api/ArcanistMercurialAPI.php

Loading...

src/repository/api/ArcanistRepositoryAPI.php

Loading...

src/repository/api/ArcanistSubversionAPI.php

Loading...