Code

rebase: use explicit "--" with checkout
authorJeff King <peff@peff.net>
Thu, 27 Jan 2011 00:26:59 +0000 (19:26 -0500)
committerJunio C Hamano <gitster@pobox.com>
Thu, 27 Jan 2011 01:26:58 +0000 (17:26 -0800)
commit3b21a438c99b38c772247201d30b7ee80b278463
tree2d34b23a453c099978a9a52d01b61e15c353b8b9
parentb312b4123b053898a583aa6e8da1b26ea828105a
rebase: use explicit "--" with checkout

In the case of a ref/pathname conflict, checkout will
already do the right thing and checkout the ref. However,
for a non-existant ref, this has two advantages:

  1. If a file with that pathname exists, rebase will
     refresh the file from the index and then rebase the
     current branch instead of producing an error.

  2. If no such file exists, the error message using an
     explicit "--" is better:

       # before
       $ git rebase -i origin bogus
       error: pathspec 'bogus' did not match any file(s) known to git.
       Could not checkout bogus

       # after
       $ git rebase -i origin bogus
       fatal: invalid reference: bogus
       Could not checkout bogus

The problems seem to be trigger-able only through "git
rebase -i", as regular git-rebase checks the validity of the
branch parameter as a ref very early on. However, it doesn't
hurt to be defensive.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-rebase--interactive.sh
git-rebase.sh