merge: fix incorrect merge message for ambiguous tag/branch
authorJeff King <peff@peff.net>
Sun, 9 Aug 2009 10:02:24 +0000 (06:02 -0400)
committerJunio C Hamano <gitster@pobox.com>
Sun, 9 Aug 2009 19:34:20 +0000 (12:34 -0700)
commit751c59746c7522982214528eb653dfb61d372257
tree7be0dd989eaef7dc37804527402533050ba72491
parentce0646184696ae6d87c0e243d058cf5df6459c4f
merge: fix incorrect merge message for ambiguous tag/branch

If we have both a tag and a branch named "foo", then calling
"git merge foo" will warn about the ambiguous ref, but merge
the tag.

When generating the commit message, though, we simply
checked whether "refs/heads/foo" existed, and if it did,
assumed it was a branch. This led to the statement "Merge
branch 'foo'" in the commit message, which is quite wrong.

Instead, we should use dwim_ref to find the actual ref used,
and describe it appropriately.

In addition to the test in t7608, we must also tweak the
expected output of t4202, which was accidentally triggering
this bug.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-merge.c
t/t4202-log.sh
t/t7608-merge-messages.sh