diff options
author | Elijah Newren <newren@gmail.com> | 2025-03-06 15:30:27 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2025-03-06 09:38:20 -0800 |
commit | 3adba40858036a5a44f550aaab5287ad135f5f87 (patch) | |
tree | 80c3d4188e6b10b71f8402e078791f1f928a31fb /builtin/count-objects.c | |
parent | 98a1a00d53018c7e664644d886466a820aa5e6d7 (diff) |
merge-ort: fix slightly overzealous assertion for rename-to-self
merge-ort has a number of sanity checks on the file it is processing in
process_renames(). One of these sanity checks was slightly overzealous
because it indirectly assumed that a renamed file always ended up at a
different path than where it started. That is normally an entirely fair
assumption, but directory rename detection can make things interesting.
As a quick refresher, if one side of history renames directory A/ -> B/,
and the other side of history adds new files to A/, then directory
rename detection notices and suggests moving those new files to B/. A
similar thing is done for paths renamed into A/, causing them to be
transitively renamed into B/. But, if the file originally came from B/,
then this can end up causing a file to be renamed back to itself.
It turns out the rest of the code following this assertion handled the
case fine; the assertion was just an extra sanity check, not a rigid
precondition. Therefore, simply adjust the assertion to pass under this
special case as well.
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/count-objects.c')
0 files changed, 0 insertions, 0 deletions