Browse Source

Fix check_file_directory_conflict().

When replacing an existing file A with a directory A that has a
file A/B in it in the index, 'update-index --replace --add A/B'
did not properly remove the file to make room for the new
directory.

There was a trivial logic error, most likely a cut & paste one,
dating back to quite early days of git.

Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Junio C Hamano 18 years ago
parent
commit
81a361be3b
  1. 2
      read-cache.c
  2. 9
      t/t0000-basic.sh

2
read-cache.c

@ -517,7 +517,7 @@ static int has_dir_name(const struct cache_entry *ce, int pos, int ok_to_replace @@ -517,7 +517,7 @@ static int has_dir_name(const struct cache_entry *ce, int pos, int ok_to_replace
pos = cache_name_pos(name, ntohs(create_ce_flags(len, stage)));
if (pos >= 0) {
retval = -1;
if (ok_to_replace)
if (!ok_to_replace)
break;
remove_cache_entry_at(pos);
continue;

9
t/t0000-basic.sh

@ -272,4 +272,13 @@ test_expect_success \ @@ -272,4 +272,13 @@ test_expect_success \
wc -l) &&
test $numparent = 1'

test_expect_success 'update-index D/F conflict' '
mv path0 tmp &&
mv path2 path0 &&
mv tmp path2 &&
git update-index --add --replace path2 path0/file2 &&
numpath0=$(git ls-files path0 | wc -l) &&
test $numpath0 = 1
'

test_done

Loading…
Cancel
Save