Remove unnecessary debugging from builtin-fetch

The older git-fetch client did not produce all of this debugging
information to stdout.  Most end-users and Porcelain (e.g. StGIT,
git-gui, qgit) do not want to see these low-level details on the
console so they should be removed.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin-fetch-pack.c b/builtin-fetch-pack.c
index 3b217d96f074c1af1429ccdc90499338cfee04ea..e77cd267193843c19f75a5debfb57841b0d5f211 100644 (file)
--- a/builtin-fetch-pack.c
+++ b/builtin-fetch-pack.c
@@ -767,8 +767,6 @@ struct ref *fetch_pack(const char *dest, int nr_heads, char **heads)
                        st.st_mtime = 0;
        }
 
-       printf("connect to %s\n", dest);
-
        pid = git_connect(fd, (char *)dest, uploadpack,
                           verbose ? CONNECT_VERBOSE : 0);
        if (pid < 0)

diff --git a/builtin-fetch.c b/builtin-fetch.c
index 016c6e43ece348ac0763a21c1cb6ae971e164f94..33b740cd105d03fca9d63b5a68c2ebec3977114f 100644 (file)
--- a/builtin-fetch.c
+++ b/builtin-fetch.c
@@ -370,15 +370,6 @@ static int do_fetch(struct transport *transport,
        for (rm = ref_map; rm; rm = rm->next) {
                if (rm->peer_ref)
                        read_ref(rm->peer_ref->name, rm->peer_ref->old_sha1);
-
-               printf("%s : %s\n", rm->name, rm->peer_ref ? rm->peer_ref->name : NULL);
-               printf("  < %s\n", sha1_to_hex(rm->old_sha1));
-               if (rm->peer_ref)
-                       printf("  > %s\n", sha1_to_hex(rm->peer_ref->old_sha1));
-               if (!rm->peer_ref ||
-                   hashcmp(rm->old_sha1, rm->peer_ref->old_sha1)) {
-                       printf("%s needs update.\n", rm->name);
-               }
        }
 
        if (fetch_refs(transport, ref_map)) {