summaryrefslogtreecommitdiff
path: root/builtin/column.c
diff options
context:
space:
mode:
authorElijah Newren <newren@gmail.com>2019-10-01 18:17:27 (GMT)
committerJunio C Hamano <gitster@pobox.com>2019-10-02 05:59:29 (GMT)
commit8e4ec3376e9d73bd471336cc7c11b35f5bc5dc87 (patch)
tree10d45f8bffb53a03d8426d270a12839774d5ef58 /builtin/column.c
parent4615a8cb5b3a8d4959c30338925b1fa3b948ae52 (diff)
downloadgit-8e4ec3376e9d73bd471336cc7c11b35f5bc5dc87.zip
git-8e4ec3376e9d73bd471336cc7c11b35f5bc5dc87.tar.gz
git-8e4ec3376e9d73bd471336cc7c11b35f5bc5dc87.tar.bz2
merge-recursive: fix the diff3 common ancestor label for virtual commits
In commit 743474cbfa8b ("merge-recursive: provide a better label for diff3 common ancestor", 2019-08-17), the label for the common ancestor was changed from always being "merged common ancestors" to instead be based on the number of merge bases: >=2: "merged common ancestors" 1: <abbreviated commit hash> 0: "<empty tree>" Unfortunately, this did not take into account that when we have a single merge base, that merge base could be fake or constructed. In such cases, this resulted in a label of "00000000". Of course, the previous label of "merged common ancestors" was also misleading for this case. Since we have an API that is explicitly about creating fake merge base commits in merge_recursive_generic(), we should provide a better label when using that API with one merge base. So, when merge_recursive_generic() is called with one merge base, set the label to: "constructed merge base" Note that callers of merge_recursive_generic() include the builtin commands git-am (in combination with git apply --build-fake-ancestor), git-merge-recursive, and git-stash. Helped-by: Jeff King <peff@peff.net> Signed-off-by: Elijah Newren <newren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/column.c')
0 files changed, 0 insertions, 0 deletions