Skip to content

Commit

Permalink
clone: do not add alternate references to extra_refs
Browse files Browse the repository at this point in the history
Alternate references are directly (and now, correctly) handled by
fetch-pack, so there is no need to inform fetch-pack about them via
the extra_refs back channel.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Michael Haggerty authored and Junio C Hamano committed Feb 13, 2012
1 parent f257659 commit a1287f7
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions builtin/clone.c
Original file line number Diff line number Diff line change
Expand Up @@ -232,9 +232,6 @@ static int add_one_reference(struct string_list_item *item, void *cb_data)
{
char *ref_git;
struct strbuf alternate = STRBUF_INIT;
struct remote *remote;
struct transport *transport;
const struct ref *extra;

/* Beware: real_path() and mkpath() return static buffer */
ref_git = xstrdup(real_path(item->string));
Expand All @@ -249,14 +246,6 @@ static int add_one_reference(struct string_list_item *item, void *cb_data)
strbuf_addf(&alternate, "%s/objects", ref_git);
add_to_alternates_file(alternate.buf);
strbuf_release(&alternate);

remote = remote_get(ref_git);
transport = transport_get(remote, ref_git);
for (extra = transport_get_remote_refs(transport); extra;
extra = extra->next)
add_extra_ref(extra->name, extra->old_sha1, 0);

transport_disconnect(transport);
free(ref_git);
return 0;
}
Expand Down Expand Up @@ -500,7 +489,6 @@ static void update_remote_refs(const struct ref *refs,
const char *msg)
{
if (refs) {
clear_extra_refs();
write_remote_refs(mapped_refs);
if (option_single_branch)
write_followtags(refs, msg);
Expand Down

0 comments on commit a1287f7

Please sign in to comment.