undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'jk/difftool-in-subdir' into maint

Browse Source 
"git difftool <paths>..." started in a subdirectory failed to
interpret the paths relative to that directory, which has been
fixed.

* jk/difftool-in-subdir:
  difftool: use Git::* functions instead of passing around state
  difftool: avoid $GIT_DIR and $GIT_WORK_TREE
  difftool: fix argument handling in subdirs
This commit is contained in:
Junio C Hamano 2016-08-12 09:16:57 -07:00
commit 9b601eafd1
2 changed files with 41 additions and 59 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

86
git-difftool.perl
View File

@ -37,14 +37,6 @@ USAGE
exit($exitcode); exit($exitcode);
} }
sub find_worktree
{
# Git->repository->wc_path() does not honor changes to the working
# tree location made by $ENV{GIT_WORK_TREE} or the 'core.worktree'
# config variable.
return Git::command_oneline('rev-parse', '--show-toplevel');
}
sub print_tool_help sub print_tool_help
{ {
# See the comment at the bottom of file_diff() for the reason behind # See the comment at the bottom of file_diff() for the reason behind
@ -67,14 +59,14 @@ sub exit_cleanup
sub use_wt_file sub use_wt_file
{ {
my ($repo, $workdir, $file, $sha1) = @_; my ($workdir, $file, $sha1) = @_;
my $null_sha1 = '0' x 40; my $null_sha1 = '0' x 40;
if (-l "$workdir/$file" || ! -e _) { if (-l "$workdir/$file" || ! -e _) {
return (0, $null_sha1); return (0, $null_sha1);
} }
my $wt_sha1 = $repo->command_oneline('hash-object', "$workdir/$file"); my $wt_sha1 = Git::command_oneline('hash-object', "$workdir/$file");
my $use = ($sha1 eq $null_sha1) || ($sha1 eq $wt_sha1); my $use = ($sha1 eq $null_sha1) || ($sha1 eq $wt_sha1);
return ($use, $wt_sha1); return ($use, $wt_sha1);
} }
@ -83,20 +75,17 @@ sub changed_files
{ {
my ($repo_path, $index, $worktree) = @_; my ($repo_path, $index, $worktree) = @_;
$ENV{GIT_INDEX_FILE} = $index; $ENV{GIT_INDEX_FILE} = $index;
$ENV{GIT_WORK_TREE} = $worktree;
my $must_unset_git_dir = 0;
if (not defined($ENV{GIT_DIR})) {
$must_unset_git_dir = 1;
$ENV{GIT_DIR} = $repo_path;
}
my @refreshargs = qw/update-index --really-refresh -q --unmerged/; my @gitargs = ('--git-dir', $repo_path, '--work-tree', $worktree);
my @gitargs = qw/diff-files --name-only -z/; my @refreshargs = (
@gitargs, 'update-index',
'--really-refresh', '-q', '--unmerged');
try { try {
Git::command_oneline(@refreshargs); Git::command_oneline(@refreshargs);
} catch Git::Error::Command with {}; } catch Git::Error::Command with {};
my $line = Git::command_oneline(@gitargs); my @diffargs = (@gitargs, 'diff-files', '--name-only', '-z');
my $line = Git::command_oneline(@diffargs);
my @files; my @files;
if (defined $line) { if (defined $line) {
@files = split('\0', $line); @files = split('\0', $line);
@ -105,26 +94,15 @@ sub changed_files
} }
delete($ENV{GIT_INDEX_FILE}); delete($ENV{GIT_INDEX_FILE});
delete($ENV{GIT_WORK_TREE});
delete($ENV{GIT_DIR}) if ($must_unset_git_dir);
return map { $_ => 1 } @files; return map { $_ => 1 } @files;
} }
sub setup_dir_diff sub setup_dir_diff
{ {
my ($repo, $workdir, $symlinks) = @_; my ($workdir, $symlinks) = @_;
# Run the diff; exit immediately if no diff found
# 'Repository' and 'WorkingCopy' must be explicitly set to insure that
# if $GIT_DIR and $GIT_WORK_TREE are set in ENV, they are actually used
# by Git->repository->command*.
my $repo_path = $repo->repo_path();
my %repo_args = (Repository => $repo_path, WorkingCopy => $workdir);
my $diffrepo = Git->repository(%repo_args);
my @gitargs = ('diff', '--raw', '--no-abbrev', '-z', @ARGV); my @gitargs = ('diff', '--raw', '--no-abbrev', '-z', @ARGV);
my $diffrtn = $diffrepo->command_oneline(@gitargs); my $diffrtn = Git::command_oneline(@gitargs);
exit(0) unless defined($diffrtn); exit(0) unless defined($diffrtn);
# Build index info for left and right sides of the diff # Build index info for left and right sides of the diff
@ -176,12 +154,12 @@ EOF
if ($lmode eq $symlink_mode) { if ($lmode eq $symlink_mode) {
$symlink{$src_path}{left} = $symlink{$src_path}{left} =
$diffrepo->command_oneline('show', "$lsha1"); Git::command_oneline('show', $lsha1);
} }
if ($rmode eq $symlink_mode) { if ($rmode eq $symlink_mode) {
$symlink{$dst_path}{right} = $symlink{$dst_path}{right} =
$diffrepo->command_oneline('show', "$rsha1"); Git::command_oneline('show', $rsha1);
} }
if ($lmode ne $null_mode and $status !~ /^C/) { if ($lmode ne $null_mode and $status !~ /^C/) {
@ -193,8 +171,8 @@ EOF
if ($working_tree_dups{$dst_path}++) { if ($working_tree_dups{$dst_path}++) {
next; next;
} }
my ($use, $wt_sha1) = use_wt_file($repo, $workdir, my ($use, $wt_sha1) =
$dst_path, $rsha1); use_wt_file($workdir, $dst_path, $rsha1);
if ($use) { if ($use) {
push @working_tree, $dst_path; push @working_tree, $dst_path;
$wtindex .= "$rmode $wt_sha1\t$dst_path\0"; $wtindex .= "$rmode $wt_sha1\t$dst_path\0";
@ -211,44 +189,34 @@ EOF
mkpath($ldir) or exit_cleanup($tmpdir, 1); mkpath($ldir) or exit_cleanup($tmpdir, 1);
mkpath($rdir) or exit_cleanup($tmpdir, 1); mkpath($rdir) or exit_cleanup($tmpdir, 1);
# If $GIT_DIR is not set prior to calling 'git update-index' and
# 'git checkout-index', then those commands will fail if difftool
# is called from a directory other than the repo root.
my $must_unset_git_dir = 0;
if (not defined($ENV{GIT_DIR})) {
$must_unset_git_dir = 1;
$ENV{GIT_DIR} = $repo_path;
}
# Populate the left and right directories based on each index file # Populate the left and right directories based on each index file
my ($inpipe, $ctx); my ($inpipe, $ctx);
$ENV{GIT_INDEX_FILE} = "$tmpdir/lindex"; $ENV{GIT_INDEX_FILE} = "$tmpdir/lindex";
($inpipe, $ctx) = ($inpipe, $ctx) =
$repo->command_input_pipe(qw(update-index -z --index-info)); Git::command_input_pipe('update-index', '-z', '--index-info');
print($inpipe $lindex); print($inpipe $lindex);
$repo->command_close_pipe($inpipe, $ctx); Git::command_close_pipe($inpipe, $ctx);
my $rc = system('git', 'checkout-index', '--all', "--prefix=$ldir/"); my $rc = system('git', 'checkout-index', '--all', "--prefix=$ldir/");
exit_cleanup($tmpdir, $rc) if $rc != 0; exit_cleanup($tmpdir, $rc) if $rc != 0;
$ENV{GIT_INDEX_FILE} = "$tmpdir/rindex"; $ENV{GIT_INDEX_FILE} = "$tmpdir/rindex";
($inpipe, $ctx) = ($inpipe, $ctx) =
$repo->command_input_pipe(qw(update-index -z --index-info)); Git::command_input_pipe('update-index', '-z', '--index-info');
print($inpipe $rindex); print($inpipe $rindex);
$repo->command_close_pipe($inpipe, $ctx); Git::command_close_pipe($inpipe, $ctx);
$rc = system('git', 'checkout-index', '--all', "--prefix=$rdir/"); $rc = system('git', 'checkout-index', '--all', "--prefix=$rdir/");
exit_cleanup($tmpdir, $rc) if $rc != 0; exit_cleanup($tmpdir, $rc) if $rc != 0;
$ENV{GIT_INDEX_FILE} = "$tmpdir/wtindex"; $ENV{GIT_INDEX_FILE} = "$tmpdir/wtindex";
($inpipe, $ctx) = ($inpipe, $ctx) =
$repo->command_input_pipe(qw(update-index --info-only -z --index-info)); Git::command_input_pipe('update-index', '--info-only', '-z', '--index-info');
print($inpipe $wtindex); print($inpipe $wtindex);
$repo->command_close_pipe($inpipe, $ctx); Git::command_close_pipe($inpipe, $ctx);
# If $GIT_DIR was explicitly set just for the update/checkout # If $GIT_DIR was explicitly set just for the update/checkout
# commands, then it should be unset before continuing. # commands, then it should be unset before continuing.
delete($ENV{GIT_DIR}) if ($must_unset_git_dir);
delete($ENV{GIT_INDEX_FILE}); delete($ENV{GIT_INDEX_FILE});
# Changes in the working tree need special treatment since they are # Changes in the working tree need special treatment since they are
@ -415,9 +383,9 @@ sub dir_diff
my $rc; my $rc;
my $error = 0; my $error = 0;
my $repo = Git->repository(); my $repo = Git->repository();
my $workdir = find_worktree(); my $repo_path = $repo->repo_path();
my ($a, $b, $tmpdir, @worktree) = my $workdir = $repo->wc_path();
setup_dir_diff($repo, $workdir, $symlinks); my ($a, $b, $tmpdir, @worktree) = setup_dir_diff($workdir, $symlinks);
if (defined($extcmd)) { if (defined($extcmd)) {
$rc = system($extcmd, $a, $b); $rc = system($extcmd, $a, $b);
@ -443,10 +411,10 @@ sub dir_diff
next if ! -f "$b/$file"; next if ! -f "$b/$file";
if (!$indices_loaded) { if (!$indices_loaded) {
%wt_modified = changed_files($repo->repo_path(), %wt_modified = changed_files(
"$tmpdir/wtindex", "$workdir"); $repo_path, "$tmpdir/wtindex", $workdir);
%tmp_modified = changed_files($repo->repo_path(), %tmp_modified = changed_files(
"$tmpdir/wtindex", "$b"); $repo_path, "$tmpdir/wtindex", $b);
$indices_loaded = 1; $indices_loaded = 1;
} }

14
t/t7800-difftool.sh
View File

@ -412,6 +412,20 @@ run_dir_diff_test 'difftool --dir-diff from subdirectory' '
) )
' '
run_dir_diff_test 'difftool --dir-diff from subdirectory with GIT_DIR set' '
(
GIT_DIR=$(pwd)/.git &&
export GIT_DIR &&
GIT_WORK_TREE=$(pwd) &&
export GIT_WORK_TREE &&
cd sub &&
git difftool --dir-diff $symlinks --extcmd ls \
branch -- sub >output &&
grep sub output &&
! grep file output
)
'
run_dir_diff_test 'difftool --dir-diff when worktree file is missing' ' run_dir_diff_test 'difftool --dir-diff when worktree file is missing' '
test_when_finished git reset --hard && test_when_finished git reset --hard &&
rm file2 && rm file2 &&