git-codereview: work around git checkout-index bug in gofmt

If you run 'git gofmt' when files in the working directory differ from
modified files in the index and you are running the command not
from that working directory or a parent, then git checkout-index --temp
(used by git gofmt) will print mangled path names, confusing git gofmt.
The most common error this causes is:

	git-codereview: gofmt reported errors:
		stat : no such file or directory

although there are other possible ways it can fail.

I've reported the git bug, which will be fixed in Git 2.3.0.
Work around the bug by always invoking git checkout-index in the
repo root.

Fixes #9476.

Change-Id: Ifcbdfd4a966dd21518dd95ceacd81a1679048e3c
Reviewed-on: https://go-review.googlesource.com/2907
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
Reviewed-by: Andrew Gerrand <adg@golang.org>

1 files changed, 6 insertions, 2 deletions

diff --git a/git-codereview/gofmt.go b/git-codereview/gofmt.go
index 937fbd6..96b5e86 100644
--- a/git-codereview/gofmt.go
+++ b/git-codereview/gofmt.go
@@ -203,14 +203,18 @@ func runGofmt(flags int) (files []string, stderrText string) {
 			}
 			args := []string{"checkout-index", "--temp", "--"}
 			args = append(args, needTemp[:n]...)
-			for _, line := range nonBlankLines(cmdOutput("git", args...)) {
+			// Until Git 2.3.0, git checkout-index --temp is broken if not run in the repo root.
+			// Work around by running in the repo root.
+			// http://article.gmane.org/gmane.comp.version-control.git/261739
+			// https://github.com/git/git/commit/74c4de5
+			for _, line := range nonBlankLines(cmdOutputDir(repo, "git", args...)) {
 				i := strings.Index(line, "\t")
 				if i < 0 {
 					continue
 				}
 				temp, file := line[:i], line[i+1:]
 				temp = filepath.Join(repo, temp)
-				file = filepath.Join(pwd, file)
+				file = filepath.Join(repo, file)
 				tempToFile[temp] = file
 				fileToTemp[file] = temp
 			}