Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
pass transport verbosity down to git_connect
While working in connect.c to perform non-blocking connections,
I noticed calling "git fetch -v" was not causing the progress
messages inside git_tcp_connect_sock to be emitted as I
expected.

Looking at history, it seems connect_setup has never been called
with the verbose parameter.  Since transport already has a
"verbose" field, use that field instead of another parameter
in connect_setup.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Eric Wong authored and Junio C Hamano committed Jan 28, 2016
1 parent 1b0b6dd commit f3ee9ca
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions transport.c
Expand Up @@ -481,17 +481,18 @@ static int set_git_option(struct git_transport_options *opts,
return 1;
}

static int connect_setup(struct transport *transport, int for_push, int verbose)
static int connect_setup(struct transport *transport, int for_push)
{
struct git_transport_data *data = transport->data;
int flags = transport->verbose > 0 ? CONNECT_VERBOSE : 0;

if (data->conn)
return 0;

data->conn = git_connect(data->fd, transport->url,
for_push ? data->options.receivepack :
data->options.uploadpack,
verbose ? CONNECT_VERBOSE : 0);
flags);

return 0;
}
Expand All @@ -501,7 +502,7 @@ static struct ref *get_refs_via_connect(struct transport *transport, int for_pus
struct git_transport_data *data = transport->data;
struct ref *refs;

connect_setup(transport, for_push, 0);
connect_setup(transport, for_push);
get_remote_heads(data->fd[0], NULL, 0, &refs,
for_push ? REF_NORMAL : 0,
&data->extra_have,
Expand Down Expand Up @@ -536,7 +537,7 @@ static int fetch_refs_via_pack(struct transport *transport,
args.update_shallow = data->options.update_shallow;

if (!data->got_remote_heads) {
connect_setup(transport, 0, 0);
connect_setup(transport, 0);
get_remote_heads(data->fd[0], NULL, 0, &refs_tmp, 0,
NULL, &data->shallow);
data->got_remote_heads = 1;
Expand Down Expand Up @@ -812,7 +813,7 @@ static int git_transport_push(struct transport *transport, struct ref *remote_re

if (!data->got_remote_heads) {
struct ref *tmp_refs;
connect_setup(transport, 1, 0);
connect_setup(transport, 1);

get_remote_heads(data->fd[0], NULL, 0, &tmp_refs, REF_NORMAL,
NULL, &data->shallow);
Expand Down

0 comments on commit f3ee9ca

Please sign in to comment.