diff options
| author | Johan Herland <johan@herland.net> | 2011-11-25 00:09:47 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-11-29 00:38:46 (GMT) |
| commit | 183868578049f5a28f7166716a4b00768d148958 (patch) | |
| tree | e7b085ff706db7fa689d316adca3d90fb1e7e751 /t | |
| parent | 9ff5e21f0e2d0944535dbf4d35758a7e027ae0c6 (diff) | |
| download | git-183868578049f5a28f7166716a4b00768d148958.zip git-183868578049f5a28f7166716a4b00768d148958.tar.gz git-183868578049f5a28f7166716a4b00768d148958.tar.bz2 | |
fast-import: Fix incorrect fanout level when modifying existing notes refs
This fixes the bug uncovered by the tests added in the previous two patches.
When an existing notes ref was loaded into the fast-import machinery, the
num_notes counter associated with that ref remained == 0, even though the
true number of notes in the loaded ref was higher. This caused a fanout
level of 0 to be used, although the actual fanout of the tree could be > 0.
Manipulating the notes tree at an incorrect fanout level causes removals to
silently fail, and modifications of existing notes to instead produce an
additional note (leaving the old object in place at a different fanout level).
This patch fixes the bug by explicitly counting the number of notes in the
notes tree whenever it looks like the num_notes counter could be wrong (when
num_notes == 0). There may be false positives (i.e. triggering the counting
when the notes tree is truly empty), but in those cases, the counting should
not take long.
Signed-off-by: Johan Herland <johan@herland.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t9301-fast-import-notes.sh | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/t/t9301-fast-import-notes.sh b/t/t9301-fast-import-notes.sh index 57d85a6..83acf68 100755 --- a/t/t9301-fast-import-notes.sh +++ b/t/t9301-fast-import-notes.sh @@ -536,7 +536,7 @@ EXPECT_END j=$(($j + 1)) done -test_expect_failure 'change a few existing notes' ' +test_expect_success 'change a few existing notes' ' git fast-import <input && GIT_NOTES_REF=refs/notes/many_notes git log -n3 refs/heads/many_commits | @@ -545,7 +545,7 @@ test_expect_failure 'change a few existing notes' ' ' -test_expect_failure 'verify that changing notes respect existing fanout' ' +test_expect_success 'verify that changing notes respect existing fanout' ' # None of the entries in the top-level notes tree should be a full SHA1 git ls-tree --name-only refs/notes/many_notes | @@ -594,7 +594,7 @@ EXPECT_END i=$(($i - 1)) done -test_expect_failure 'remove lots of notes' ' +test_expect_success 'remove lots of notes' ' git fast-import <input && GIT_NOTES_REF=refs/notes/many_notes git log refs/heads/many_commits | @@ -603,7 +603,7 @@ test_expect_failure 'remove lots of notes' ' ' -test_expect_failure 'verify that removing notes trigger fanout consolidation' ' +test_expect_success 'verify that removing notes trigger fanout consolidation' ' # All entries in the top-level notes tree should be a full SHA1 git ls-tree --name-only -r refs/notes/many_notes | |