Page MenuHomeDevCentral

Locale set to en_US.UTF-8
ClosedPublic

Authored by dereckson on Dec 1 2015, 08:46.
Referenced Files
F11718645: D101.id236.diff
Mon, Sep 15, 00:58
Unknown Object (File)
Fri, Sep 12, 16:45
Unknown Object (File)
Mon, Sep 8, 23:21
Unknown Object (File)
Sun, Sep 7, 14:22
Unknown Object (File)
Fri, Sep 5, 03:33
Unknown Object (File)
Tue, Sep 2, 10:12
Unknown Object (File)
Thu, Aug 28, 16:22
Unknown Object (File)
Wed, Aug 27, 09:05

Details

Summary

If a commit contains an UTF-8 character like → (right arrow),
Arcanist mangles it when using arc diff.

We install the locales package and fix the locale to en_US.UTF-8.

Reference: http://serverfault.com/a/689947/186660 by EirikW

Fixes T611.

Test Plan

Type an é in a shell launched in the container.

To launch it, arc shell or docker run --rm -it nasqueron/arcanist bash.

Diff Detail

Repository
rDARC nasqueron/arcanist Docker image
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to Locale set to en_US.UTF-8.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: Sandlayth.

Another test done: this diff has been created with an image built with this change, the from the commit were correctly printed and I were able to write an é.

I wonder if that shouldn't go to nasqueron/php-cli.

Sandlayth edited edge metadata.
This revision is now accepted and ready to land.Dec 2 2015, 09:24
This revision was automatically updated to reflect the committed changes.