[PATCH v3 08/10] clone: refuse to clone if --branch points to bogus ref
- From
Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
- Date
- Jan 10, 2012, 09:57 UTC
- Message-ID
- <1326189427-20800-9-git-send-email-pclouds@gmail.com>
- In-Reply-To
- <1326023188-15559-1-git-send-email-pclouds@gmail.com>
It's possible that users make a typo in the branch name. Stop and let users recheck. Falling back to remote's HEAD is not documented any way.
Except when using remote helper, the pack has not been transferred at this stage yet so we don't waste much bandwidth.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> --- builtin/clone.c | 10 ++++------ t/t5706-clone-branch.sh | 8 ++------ 2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/builtin/clone.c b/builtin/clone.c index 960242f..f751997 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -766,12 +766,10 @@ int cmd_clone(int argc, const char **argv, const char *prefix) find_ref_by_name(mapped_refs, head.buf); strbuf_release(&head); - if (!our_head_points_at) { - warning(_("Remote branch %s not found in " - "upstream %s, using HEAD instead"), - option_branch, option_origin); - our_head_points_at = remote_head_points_at; - } + if (!our_head_points_at) + die(_("Remote branch %s not found in " + "upstream %s, using HEAD instead"), + option_branch, option_origin); } else our_head_points_at = remote_head_points_at; diff --git a/t/t5706-clone-branch.sh b/t/t5706-clone-branch.sh index f3f9a76..56be67e 100755 --- a/t/t5706-clone-branch.sh +++ b/t/t5706-clone-branch.sh @@ -57,12 +57,8 @@ test_expect_success 'clone -b does not munge remotes/origin/HEAD' ' ) ' -test_expect_success 'clone -b with bogus branch chooses HEAD' ' - git clone -b bogus parent clone-bogus && - (cd clone-bogus && - check_HEAD master && - check_file one - ) +test_expect_success 'clone -b with bogus branch' ' + test_must_fail git clone -b bogus parent clone-bogus ' test_done
-- 1.7.3.1.256.g2539c.dirty