Code

diff-index: careful when inspecting work tree items
authorJunio C Hamano <gitster@pobox.com>
Mon, 31 Mar 2008 00:29:48 +0000 (17:29 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 31 Mar 2008 05:22:09 +0000 (22:22 -0700)
commit948dd346fd6748f8c2c0ae488759cbbd05a09320
tree63ea6a54d1824350aa14b1353b65671ea186c21a
parent6301f303d436f226b705380ab97c7c0122374241
diff-index: careful when inspecting work tree items

Earlier, if you changed a staged path into a directory in the work tree,
we happily ran lstat(2) on it and found that it exists, and declared that
the user changed it to a gitlink.

This is wrong for two reasons:

 (1) It may be a directory, but it may not be a submodule, and in the
     latter case, the change we need to report is "the blob at the path
     has disappeared".  We need to check with resolve_gitlink_ref() to be
     consistent with what "git add" and "git update-index --add" does.

 (2) lstat(2) may have succeeded only because a leading component of the
     path was turned into a symbolic link that points at something that
     exists in the work tree.  In such a case, the path itself does not
     exist anymore, as far as the index is concerned.

This fixes these breakages in diff-index that the previous patch has
exposed.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff-lib.c
t/t2201-add-update-typechange.sh