Code

checkout: fix bug with ambiguous refs
authorJunio C Hamano <gitster@pobox.com>
Tue, 11 Jan 2011 20:00:38 +0000 (15:00 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 11 Jan 2011 21:14:56 +0000 (13:14 -0800)
commit0cb6ad3c3d8e9c738686ef8dc6f173f725e509bc
treec218deb54009b1f59a959103a0390fc7d330f2a7
parent6e97ef31ce20ff93a4cc0e3c6ab5171fd6dda5f3
checkout: fix bug with ambiguous refs

The usual dwim_ref lookup prefers tags to branches. Because
checkout primarily works on branches, though, we switch that
behavior to prefer branches.

However, there was a bug in the implementation in which we
used lookup_commit_reference (which used the regular lookup
rules) to get the actual commit to checkout. Checking out an
ambiguous ref therefore ended up putting us in an extremely
broken state in which we wrote the branch ref into HEAD, but
actually checked out the tree for the tag.

This patch fixes the bug by always attempting to pull the
commit to be checked out from the branch-ified version of
the name we were given.

Patch by Junio, tests and commit message from Jeff King.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/checkout.c
t/t2019-checkout-amiguous-ref.sh [new file with mode: 0755]