diff options
author | Jonathan Tan <jonathantanmy@google.com> | 2017-12-14 13:44:44 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-12-14 14:28:02 -0800 |
commit | 245abe34ac0622b349470b89d4b76a082a133cab (patch) | |
tree | 0d88e6d71ad63ee26d1c94f889765a2f9cdadd1d /builtin | |
parent | 170078693fcccd614406e1c3c9fcbea8aaba0c13 (diff) | |
download | git-245abe34ac0622b349470b89d4b76a082a133cab.tar.gz |
clone, fetch: remove redundant transport check
Prior to commit a2d725b7bdf7 ("Use an external program to implement
fetching with curl", 2009-08-05), if Git was compiled with NO_CURL, the
get_refs_list and fetch methods in struct transport might not be
populated, hence the checks in clone and fetch. After that commit, all
transports populate get_refs_list and fetch, making the checks in clone
and fetch redundant. Remove those checks.
Signed-off-by: Jonathan Tan <jonathantanmy@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/clone.c | 3 | ||||
-rw-r--r-- | builtin/fetch.c | 3 |
2 files changed, 0 insertions, 6 deletions
diff --git a/builtin/clone.c b/builtin/clone.c index dbddd98f80..979af0383d 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -1083,9 +1083,6 @@ int cmd_clone(int argc, const char **argv, const char *prefix) warning(_("--local is ignored")); transport->cloning = 1; - if (!transport->get_refs_list || (!is_local && !transport->fetch)) - die(_("Don't know how to clone %s"), transport->url); - transport_set_option(transport, TRANS_OPT_KEEP, "yes"); if (option_depth) diff --git a/builtin/fetch.c b/builtin/fetch.c index 225c734924..09eb1fc171 100644 --- a/builtin/fetch.c +++ b/builtin/fetch.c @@ -1094,9 +1094,6 @@ static int do_fetch(struct transport *transport, tags = TAGS_UNSET; } - if (!transport->get_refs_list || !transport->fetch) - die(_("Don't know how to fetch from %s"), transport->url); - /* if not appending, truncate FETCH_HEAD */ if (!append && !dry_run) { retcode = truncate_fetch_head(); |