Code

git-checkout: improve error messages, detect ambiguities.
authorPierre Habouzit <madcoder@debian.org>
Wed, 23 Jul 2008 10:15:33 +0000 (12:15 +0200)
committerJunio C Hamano <gitster@pobox.com>
Fri, 25 Jul 2008 06:24:00 +0000 (23:24 -0700)
commit859fdabaede6733c98b1ca8df2fabce000522bf9
treeffe48136cb1b49fbf1a0d802c952c9b5e88698d0
parent65360346309a550185363ef5b846a1202175a4e6
git-checkout: improve error messages, detect ambiguities.

The patch is twofold: it moves the option consistency checks just under
the parse_options call so that it doesn't get in the way of the tree
reference vs. pathspecs desambiguation.

The other part rewrites the way to understand arguments so that when
git-checkout fails it does with an understandable message. Compared to the
previous behavior we now have:

  - a better error message when doing:

        git checkout <blob reference> --

    now complains about the reference not pointing to a tree, instead of
    things like:

        error: pathspec <blob reference> did not match any file(s) known to git.
        error: pathspec '--' did not match any file(s) known to git.

  - a better error message when doing:

        git checkout <path> --

    It now complains about <path> not being a reference instead of the
    completely obscure:

        error: pathspec '--' did not match any file(s) known to git.

  - an error when -- wasn't used, and the first argument is ambiguous
    (i.e. can be interpreted as both ref and as path).

Signed-off-by: Pierre Habouzit <madcoder@debian.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-checkout.c
t/t2010-checkout-ambiguous.sh [new file with mode: 0755]