diff options
| author | Tim Henigan <tim.henigan@gmail.com> | 2012-03-14 12:38:06 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2012-03-14 15:22:38 -0700 |
| commit | bfe392e3674770e8a0312d4945b0a5f2c9007eab (patch) | |
| tree | 69904b5eb0b3fddeec484b7a9c3d652182156c83 /contrib/diffall | |
| parent | 97549084f660c43fe4b6dc34bf60afef4948b432 (diff) | |
| download | git-bfe392e3674770e8a0312d4945b0a5f2c9007eab.tar.gz git-bfe392e3674770e8a0312d4945b0a5f2c9007eab.tar.xz | |
contrib/diffall: fix cleanup trap on Windows
Prior to this commit, the cleanup trap that removes the tmp dir
created by the script would fail on Windows. The error was silently
ignored by the script.
On Windows, a directory cannot be removed while it is the working
directory of the process (thanks to Johannes Sixt on the Git list
for this info [1]).
This commit eliminates the 'cd' into the tmp directory that caused
the error.
[1]: http://article.gmane.org/gmane.comp.version-control.git/193086
Signed-off-by: Tim Henigan <tim.henigan@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/diffall')
| -rwxr-xr-x | contrib/diffall/git-diffall | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/contrib/diffall/git-diffall b/contrib/diffall/git-diffall index 91a31c878..84f2b654d 100755 --- a/contrib/diffall/git-diffall +++ b/contrib/diffall/git-diffall @@ -49,7 +49,7 @@ cd "$cdup" || { tmp=$(perl -e 'use File::Temp qw(tempdir); $t=tempdir("/tmp/git-diffall.XXXXX") or exit(1); print $t') || exit 1 -trap 'rm -rf "$tmp" 2>/dev/null' EXIT +trap 'rm -rf "$tmp"' EXIT left= right= @@ -233,9 +233,8 @@ do fi done < "$tmp/filelist" -cd "$tmp" -LOCAL="$left_dir" -REMOTE="$right_dir" +LOCAL="$tmp/$left_dir" +REMOTE="$tmp/$right_dir" if test -n "$diff_tool" then |