fast-import: prevent producing bad delta
authorDmitry Ivankov <divanorama@gmail.com>
Sun, 14 Aug 2011 18:32:24 +0000 (00:32 +0600)
committerJunio C Hamano <gitster@pobox.com>
Sun, 14 Aug 2011 21:40:01 +0000 (14:40 -0700)
commit8fb3ad76b1782a5439343b9601f73a9287a245cf
tree20be039763bddee81ceec02e8c6cf6ed87d63c21
parent9a0edb79f2f00c79b9dced22f67d226f6bb9c741
fast-import: prevent producing bad delta

To produce deltas for tree objects fast-import tracks two versions
of tree's entries - base and current one. Base version stands both
for a delta base of this tree, and for a entry inside a delta base
of a parent tree. So care should be taken to keep it in sync.

tree_content_set cuts away a whole subtree and replaces it with a
new one (or NULL for lazy load of a tree with known sha1). It
keeps a base sha1 for this subtree (needed for parent tree). And
here is the problem, 'subtree' tree root doesn't have the implied
base version entries.

Adjusting the subtree to include them would mean a deep rewrite of
subtree. Invalidating the subtree base version would mean recursive
invalidation of parents' base versions. So just mark this tree as
do-not-delta me. Abuse setuid bit for this purpose.

tree_content_replace is the same as tree_content_set except that is
is used to replace the root, so just clearing base sha1 here (instead
of setting the bit) is fine.

[di: log message]

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Dmitry Ivankov <divanorama@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
fast-import.c
t/t9300-fast-import.sh