From 42db1733b9da53abdadeb650e9c6282110a66df8 Mon Sep 17 00:00:00 2001 From: Jason Ge Date: Fri, 29 Jul 2011 15:22:37 -0700 Subject: [PATCH] Specify date format in git blame Summary: right now 'arc blame' uses the user's blame.date setting, but it will only work when the format is iso. So it will fail for user who has customized it. Test Plan: run arc cover with 'blame.date' set to relative and verified that 'arc cover' still works Reviewed By: epriestley Reviewers: epriestley, codeblock CC: hwang, aran, epriestley Differential Revision: 745 --- src/repository/api/git/ArcanistGitAPI.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/repository/api/git/ArcanistGitAPI.php b/src/repository/api/git/ArcanistGitAPI.php index 13570a46..c6234cb1 100644 --- a/src/repository/api/git/ArcanistGitAPI.php +++ b/src/repository/api/git/ArcanistGitAPI.php @@ -317,7 +317,7 @@ class ArcanistGitAPI extends ArcanistRepositoryAPI { public function getBlame($path) { // TODO: 'git blame' supports --porcelain and we should probably use it. list($stdout) = execx( - '(cd %s; git blame -w -C %s -- %s)', + '(cd %s; git blame --date=iso -w -C %s -- %s)', $this->getPath(), $this->getRelativeCommit(), $path);