lix-project/lix
20
55
Fork 31
Code Issues 345 Code Review (Gerrit) Projects 3 Releases Wiki Activity

git fetcher: set locale for rev-parse

Browse source
This commit is contained in:
Martin Schwaighofer 2022-02-07 20:36:39 +01:00
parent 9504445cab
commit 53523c0ab8
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/libfetchers/git.cc
View file

@ -220,11 +220,17 @@ struct GitInputScheme : InputScheme
if (!input.getRef() && !input.getRev() && isLocal) { if (!input.getRef() && !input.getRev() && isLocal) {
bool clean = false; bool clean = false;
auto env = getEnv();
// Set LC_ALL to C: because we rely on the error messages from git rev-parse to determine what went wrong
// that way unknown errors can lead to a failure instead of continuing through the wrong code path
env["LC_ALL"] = "C";
/* Check whether HEAD points to something that looks like a commit, /* Check whether HEAD points to something that looks like a commit,
since that is the refrence we want to use later on. */ since that is the refrence we want to use later on. */
auto result = runProgram(RunOptions { auto result = runProgram(RunOptions {
.program = "git", .program = "git",
.args = { "-C", actualUrl, "--git-dir=.git", "rev-parse", "--verify", "--no-revs", "HEAD^{commit}" }, .args = { "-C", actualUrl, "--git-dir=.git", "rev-parse", "--verify", "--no-revs", "HEAD^{commit}" },
.environment = env,
.mergeStderrToStdout = true .mergeStderrToStdout = true
}); });
auto exitCode = WEXITSTATUS(result.first); auto exitCode = WEXITSTATUS(result.first);