submodule: if $command was not matched, don't parse other args
authorRamkumar Ramachandra <artagnon@gmail.com>
Sat, 22 Sep 2012 11:27:59 +0000 (16:57 +0530)
committerJunio C Hamano <gitster@pobox.com>
Tue, 25 Sep 2012 18:31:48 +0000 (11:31 -0700)
commitaf9c9f97137dfb3f20a17d76520803c0312fd3e4
tree8ad5e10520b6cd36497ed07c2fc70dedefbc07e1
parentbafc478f1618534fcb85bedc0fa224bd2d462441
submodule: if $command was not matched, don't parse other args

"git submodule" command DWIMs the command line and assumes a
unspecified action word for 'status' action.  This is a UI mistake
that leads to a confusing behaviour.  A mistyped command name is
instead treated as a request for 'status' of the submodule with that
name, e.g.

    $ git submodule show
    error: pathspec 'show' did not match any file(s) known to git.
    Did you forget to 'git add'?

Stop DWIMming an unknown or mistyped subcommand name as pathspec
given to unspelled "status" subcommand.  "git submodule" without any
argument is still interpreted as "git submodule status", but its
value is questionable.

Adjust t7400 to match, and stop advertising the default subcommand
being 'status' which does not help much in practice, other than
promoting laziness and confusion.

Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Documentation/git-submodule.txt
git-submodule.sh
t/t7400-submodule-basic.sh