From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Hommey Subject: [PATCH] clone: call transport_set_verbosity before anything else on the newly created transport Date: Tue, 12 May 2015 13:30:16 +0900 Message-ID: <1431405016-2520-1-git-send-email-mh@glandium.org> References: Cc: git@vger.kernel.org To: Junio C Hamano X-From: git-owner@vger.kernel.org Tue May 12 06:30:39 2015 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Ys1pt-0006gG-9U for gcvg-git-2@plane.gmane.org; Tue, 12 May 2015 06:30:29 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752420AbbELEaZ (ORCPT ); Tue, 12 May 2015 00:30:25 -0400 Received: from ns332406.ip-37-187-123.eu ([37.187.123.207]:38307 "EHLO glandium.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752418AbbELEaY (ORCPT ); Tue, 12 May 2015 00:30:24 -0400 Received: from glandium by zenigata with local (Exim 4.85) (envelope-from ) id 1Ys1pg-0000fJ-JK; Tue, 12 May 2015 13:30:16 +0900 X-Mailer: git-send-email 2.4.0.1.gde5e018 In-Reply-To: Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Commit 2879bc3 made the progress and verbosity options sent to remote helper earlier than they previously were. But nothing else after that would send updates if the value is changed later on with transport_set_verbosity. While for fetch and push, transport_set_verbosity is the first thing that is done after creating the transport, it was not the case for clone. So commit 2879bc3 broke changing progress and verbosity for clone, for urls requiring a remote helper only (so, not git:// urls, for instance). Moving transport_set_verbosity to just after the transport is created works around the issue. Signed-off-by: Mike Hommey --- builtin/clone.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Note that another long term way to fix this would be to move setting verbosity and progress to the transport_get call itself. The patch is against v2.4.0, if that matters. diff --git a/builtin/clone.c b/builtin/clone.c index 53a2e5a..13030ee 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -906,6 +906,8 @@ int cmd_clone(int argc, const char **argv, const char *prefix) remote = remote_get(option_origin); transport = transport_get(remote, remote->url[0]); + transport_set_verbosity(transport, option_verbosity, option_progress); + path = get_repo_path(remote->url[0], &is_bundle); is_local = option_local != 0 && path && !is_bundle; if (is_local) { @@ -932,8 +934,6 @@ int cmd_clone(int argc, const char **argv, const char *prefix) if (option_single_branch) transport_set_option(transport, TRANS_OPT_FOLLOWTAGS, "1"); - transport_set_verbosity(transport, option_verbosity, option_progress); - if (option_upload_pack) transport_set_option(transport, TRANS_OPT_UPLOADPACK, option_upload_pack); -- 2.4.0.1.gde5e018