rebase -i: more graceful handling of invalid commands
authorJan Krüger <jk@jk.gs>
Tue, 27 Oct 2009 14:58:14 +0000 (15:58 +0100)
committerJunio C Hamano <gitster@pobox.com>
Wed, 28 Oct 2009 06:12:44 +0000 (23:12 -0700)
commitf1be316ada93158507c315ee7948bb9e6007eb91
tree6b14f18ee0e6a9b0519db5e10493db3afc5509fa
parenta29aa47da79cfd0ef7ee0ef423e7e5a9a3cf07bd
rebase -i: more graceful handling of invalid commands

Currently, when there is an invalid command, the rest of the line is
still treated as if the command had been valid, i.e. rebase -i attempts
to produce a patch, using the next argument as a SHA1 name. If there is
no next argument or an invalid one, very confusing error messages
appear (the line was '.'; path to git-rebase-todo substituted):

Unknown command: .
fatal: ambiguous argument 'Please fix this in the file $somefile.':
unknown revision or path not in the working tree.
Use '--' to separate paths from revisions
fatal: Not a valid object name Please fix this in the file $somefile.
fatal: bad revision 'Please fix this in the file $somefile.'

Instead, verify the validity of the remaining line and error out earlier
if necessary.

Signed-off-by: Jan Krüger <jk@jk.gs>
Acked-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-rebase--interactive.sh