git-svn: handle our top-level path being deleted and later re-added
Previously, git-svn would ignore cases where the path we're tracking is removed from the repository. This was to prevent heads with follow-parent from ending up with a tree full of empty revisions (and thus breaking rename detection). The previous behavior is fine until the path we're tracking is re-added later on, leading to the old files being merged in with the new files in the directory (because the old files were never marked as deleted) We will now only remove all the old files locally that were deleted remotely iff we detect the directory we're in is being created from scratch. Thanks for Marcus D. Hanwell for the bug report and Peter Baumann for the analysis. Signed-off-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
38a5b1d6ed
commit
12a6d752fb
14
git-svn.perl
14
git-svn.perl
@ -3045,6 +3045,20 @@ sub add_file {
|
||||
|
||||
sub add_directory {
|
||||
my ($self, $path, $cp_path, $cp_rev) = @_;
|
||||
my $gpath = $self->git_path($path);
|
||||
if ($gpath eq '') {
|
||||
my ($ls, $ctx) = command_output_pipe(qw/ls-tree
|
||||
-r --name-only -z/,
|
||||
$self->{c});
|
||||
local $/ = "\0";
|
||||
while (<$ls>) {
|
||||
chomp;
|
||||
$self->{gii}->remove($_);
|
||||
print "\tD\t$_\n" unless $::_q;
|
||||
}
|
||||
command_close_pipe($ls, $ctx);
|
||||
$self->{empty}->{$path} = 0;
|
||||
}
|
||||
my ($dir, $file) = ($path =~ m#^(.*?)/?([^/]+)$#);
|
||||
delete $self->{empty}->{$dir};
|
||||
$self->{empty}->{$path} = 1;
|
||||
|
39
t/t9103-git-svn-tracked-directory-removed.sh
Executable file
39
t/t9103-git-svn-tracked-directory-removed.sh
Executable file
@ -0,0 +1,39 @@
|
||||
#!/bin/sh
|
||||
#
|
||||
# Copyright (c) 2007 Eric Wong
|
||||
#
|
||||
|
||||
test_description='git-svn tracking removed top-level path'
|
||||
. ./lib-git-svn.sh
|
||||
|
||||
test_expect_success 'make history for tracking' '
|
||||
mkdir import &&
|
||||
mkdir import/trunk &&
|
||||
echo hello >> import/trunk/README &&
|
||||
svn import -m initial import $svnrepo &&
|
||||
rm -rf import &&
|
||||
svn co $svnrepo/trunk trunk &&
|
||||
echo bye bye >> trunk/README &&
|
||||
svn rm -m "gone" $svnrepo/trunk &&
|
||||
rm -rf trunk &&
|
||||
mkdir trunk &&
|
||||
echo "new" > trunk/FOLLOWME &&
|
||||
svn import -m "new trunk" trunk $svnrepo/trunk
|
||||
'
|
||||
|
||||
test_expect_success 'clone repo with git' '
|
||||
git svn clone -s $svnrepo x &&
|
||||
test -f x/FOLLOWME &&
|
||||
test ! -f x/README
|
||||
'
|
||||
|
||||
test_expect_success 'make sure r2 still has old file' '
|
||||
cd x &&
|
||||
test -n "$(git svn find-rev r1)" &&
|
||||
git reset --hard $(git svn find-rev r1) &&
|
||||
test -f README &&
|
||||
test ! -f FOLLOWME &&
|
||||
test x$(git svn find-rev r2) = x
|
||||
'
|
||||
|
||||
test_done
|
Loading…
Reference in New Issue
Block a user