Skip to content

Ensure consistent output from git command #181

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 14, 2014

Conversation

hashar
Copy link
Contributor

@hashar hashar commented Jul 25, 2014

The git command output can vary by language which would cause assertions
errors when parsing the output.

On POSIX system the language used by git can be adjusted by LC_MESSAGES.
The special language 'C' is guaranteed to be always available and is
whatever default the software has been written in (usually english, the
case for git).

Thus passing LC_MESSAGES to Popen will ensure we receive from git a
consistent output regardless of the user preference.

Addresses #153

The git command output can vary by language which would cause assertions
errors when parsing the output.

On POSIX system the language used by git can be adjusted by LC_MESSAGES.
The special language 'C' is guaranteed to be always available and is
whatever default the software has been written in (usually english, the
case for git).

Thus passing LC_MESSAGES to Popen will ensure we receive from git a
consistent output regardless of the user preference.

Addresses gitpython-developers#153
@coveralls
Copy link

Coverage Status

Coverage decreased (-12.21%) when pulling 3eb7265 on hashar:git-exec-const-output into 27c577d on gitpython-developers:0.3.

@Byron
Copy link
Member

Byron commented Nov 14, 2014

Thank you !

Byron added a commit that referenced this pull request Nov 14, 2014
Ensure consistent output from git command
@Byron Byron merged commit 9ea3dbd into gitpython-developers:0.3 Nov 14, 2014
@Byron Byron added this to the v0.3.2 milestone Nov 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants