diff options
author | Jonathan Nieder <jrnieder@gmail.com> | 2009-10-29 11:10:30 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2009-10-31 01:18:11 +0300 |
commit | d52dc4b10b2f78dc24ea05e88ddc25ee0f46491e (patch) | |
tree | 7e2957739b1ea3df787185f13fcbd3ebf15054c2 | |
parent | ebc9d420566de32dbadada5a4f700db2d974853c (diff) |
clone: detect extra arguments
If git clone is given more than two non-option arguments, it
silently throws away all but the first one. Complain instead.
Discovered by comparing the new builtin clone to the old
git-clone.sh.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | builtin-clone.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/builtin-clone.c b/builtin-clone.c index 0a4f6124f5..caf3025031 100644 --- a/builtin-clone.c +++ b/builtin-clone.c @@ -379,8 +379,13 @@ int cmd_clone(int argc, const char **argv, const char *prefix) argc = parse_options(argc, argv, prefix, builtin_clone_options, builtin_clone_usage, 0); + if (argc > 2) + usage_msg_opt("Too many arguments.", + builtin_clone_usage, builtin_clone_options); + if (argc == 0) - die("You must specify a repository to clone."); + usage_msg_opt("You must specify a repository to clone.", + builtin_clone_usage, builtin_clone_options); if (option_mirror) option_bare = 1; |