From patchwork Tue Dec 7 18:23:00 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Abhradeep Chakraborty X-Patchwork-Id: 12662411 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8EA47C433EF for ; Tue, 7 Dec 2021 18:23:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239945AbhLGS11 (ORCPT ); Tue, 7 Dec 2021 13:27:27 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45262 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235273AbhLGS10 (ORCPT ); Tue, 7 Dec 2021 13:27:26 -0500 Received: from mail-pg1-x530.google.com (mail-pg1-x530.google.com [IPv6:2607:f8b0:4864:20::530]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 76661C061748 for ; Tue, 7 Dec 2021 10:23:56 -0800 (PST) Received: by mail-pg1-x530.google.com with SMTP id f125so14679871pgc.0 for ; Tue, 07 Dec 2021 10:23:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=hTVpx6vSPAop/D9HrUp1aLpEjRBDNtOcvti60xKtB04=; b=SqzfomOaDYaxJgf0gpVmjx92D6gvJVz/ItYsy2eOZo7G/pEyKunK275xj8U8hyyXxS HN9WXfzfxcTrqwTkf4DQUXRBMAtx+/f0pq4ZFdzcMUr+6NAx3PtypBcRqq4Kz0FYd2bp o45f9ho7hgoiElE3I20QDoJX6Rnz9/x8HeKssjr4HMc66z/XRNQbX2bRcErUHMdRXY2I oA/6lzq+5dKChk+1EZ59nXqDoQN5zRT4pvmN8O5xSEXsbayjWZyXlh7xPUX7/fk/Y7u+ MKIPeiG3xj4/g5qArNnXO+Tz23G1RHxU9nQGhv3V4foJMAVWfGS8LkMl8YAENP6BTE8Q iuEA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=hTVpx6vSPAop/D9HrUp1aLpEjRBDNtOcvti60xKtB04=; b=Vn64zHf/hdztfTnM0LVqH0QD1YsLRtgRXUZhq8vKQ5wiR6GqVwND/9FMAAl3FodwOp vvjHQHHAqo+/vdHuK/mcYhzrXPrseQPexrUb3mTQEpzXdhXRhJHMuUEONReZoy93+Y2s M3Ao6pLeNASgzn9GvzUSSvifvm5UojQwZqF+v+HvrtT2E2uufjaB0d8W4AgNSTOnfvGk qvANgqpdgpRFVN/tbT/EOv/IzBYCXNCG9/7raRXZFvLmm040v4ijYyQ51uC+58zCDJ2b nQ2nseqdZZSl2tvOepgvq+EQ/W4DTEw7FJ7viCZgi9Z3jxh8llSJmzMF7UH4DKOtP0B5 63AA== X-Gm-Message-State: AOAM533T86sODkAN1y5WCdnNtS5Zric/uRqtCjPc4CV15vkw1DSTKZVe uzahhUoEliJujB6UKQFS3MTK0EWPpsU= X-Google-Smtp-Source: ABdhPJymZOJmYFkFiAAmdE6zkli8Ckw08PmqTMJ96SKTKiZipS0UzuuB0XsSfLyXQbwi3EyhYVg7lw== X-Received: by 2002:a63:d245:: with SMTP id t5mr24642853pgi.483.1638901435660; Tue, 07 Dec 2021 10:23:55 -0800 (PST) Received: from LAPTOP-FJDAS7G4.localdomain ([157.40.240.182]) by smtp.gmail.com with ESMTPSA id q8sm413574pfk.152.2021.12.07.10.23.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Dec 2021 10:23:55 -0800 (PST) From: Abhradeep Chakraborty To: git@vger.kernel.org Cc: Abhradeep Chakraborty Subject: [PATCH v2 1/1] push: make '-u' have default arguments Date: Tue, 7 Dec 2021 23:53:00 +0530 Message-Id: <20211207182300.4361-2-chakrabortyabhradeep79@gmail.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20211207182300.4361-1-chakrabortyabhradeep79@gmail.com> References: <20211202144354.17416-1-chakrabortyabhradeep79@gmail.com> <20211207182300.4361-1-chakrabortyabhradeep79@gmail.com> Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org "git push -u" (set-upstream) requires where to push to and what to push. Often people push only the current branch to update the branch of the same name at the 'origin' repository. For them, it would be convenient if "git push -u" without repository or refspec, defaulted to push and set upstream to the branch as configured by the "push.default" setting, of the remote repository that is used by default. Teach "git push -u" not to require repository and refspec. When the user do not give what repository to push to, or which branch(es) to push, behave as if the default remote repository and a refspec (depending on the "push.default" configuration) are given. If "push.default"=matching, push all the branches matched on both remote and local side and set those remote branches as the upstream of their respective local matched branches. Otherwise, set the refspec to the refspec for current branch. Signed-off-by: Abhradeep Chakraborty --- Documentation/git-push.txt | 10 ++++ builtin/push.c | 11 +++- t/t5523-push-upstream.sh | 114 +++++++++++++++++++++++++++++++++++++ 3 files changed, 133 insertions(+), 2 deletions(-) diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt index 2f25aa3a29..048c087e23 100644 --- a/Documentation/git-push.txt +++ b/Documentation/git-push.txt @@ -375,6 +375,16 @@ Specifying `--no-force-if-includes` disables this behavior. upstream (tracking) reference, used by argument-less linkgit:git-pull[1] and other commands. For more information, see `branch..merge` in linkgit:git-config[1]. ++ +`-u` can also work with zero arguments( i.e. no `` and +`` are given). In that case, it tries to get the `` +value from `branch.*.remote` configuration. If not found, it defaults to +`origin`. If `remote.pushDefault` is set then it uses that instead. The +value of `` depends on the current `push.default` configuration. +If `push.default` is set to `matching`, all remote branches to which +local branches pushed, will be set as upstream of respective local +branches. For all other values of `push.default`, current branch's +`` will be used as the ``. --[no-]thin:: These options are passed to linkgit:git-send-pack[1]. A thin transfer diff --git a/builtin/push.c b/builtin/push.c index 4b026ce6c6..8bc206c9d8 100644 --- a/builtin/push.c +++ b/builtin/push.c @@ -202,11 +202,12 @@ static const char *get_upstream_ref(struct branch *branch, const char *remote_na return branch->merge[0]->src; } -static void setup_default_push_refspecs(struct remote *remote) +static void setup_default_push_refspecs(struct remote *remote, int flags) { struct branch *branch; const char *dst; int same_remote; + int is_default_u = (flags & TRANSPORT_PUSH_SET_UPSTREAM); switch (push_default) { case PUSH_DEFAULT_MATCHING: @@ -214,6 +215,8 @@ static void setup_default_push_refspecs(struct remote *remote) return; case PUSH_DEFAULT_NOTHING: + if (is_default_u) + break; die(_("You didn't specify any refspecs to push, and " "push.default is \"nothing\".")); return; @@ -234,11 +237,15 @@ static void setup_default_push_refspecs(struct remote *remote) case PUSH_DEFAULT_SIMPLE: if (!same_remote) break; + if (is_default_u) + break; if (strcmp(branch->refname, get_upstream_ref(branch, remote->name))) die_push_simple(branch, remote); break; case PUSH_DEFAULT_UPSTREAM: + if (is_default_u) + break; if (!same_remote) die(_("You are pushing to remote '%s', which is not the upstream of\n" "your current branch '%s', without telling me what to push\n" @@ -401,7 +408,7 @@ static int do_push(int flags, if (remote->push.nr) { push_refspec = &remote->push; } else if (!(flags & TRANSPORT_PUSH_MIRROR)) - setup_default_push_refspecs(remote); + setup_default_push_refspecs(remote, flags); } errs = 0; url_nr = push_url_of_remote(remote, &url); diff --git a/t/t5523-push-upstream.sh b/t/t5523-push-upstream.sh index fdb4292056..fbcdf303ef 100755 --- a/t/t5523-push-upstream.sh +++ b/t/t5523-push-upstream.sh @@ -60,6 +60,75 @@ test_expect_success 'push -u :topic_2' ' check_config topic_2 upstream refs/heads/other2 ' +default_u_setup() { + git checkout main + remote=$(git config --get branch.main.remote) + if [ ! -z "$remote" ]; then + git branch --unset-upstream + fi + git config push.default $1 + git config remote.pushDefault upstream +} + +test_expect_success 'push -u with push.default=simple' ' + default_u_setup simple && + git push -u && + check_config main upstream refs/heads/main && + git push -u upstream main:other && + git push -u && + check_config main upstream refs/heads/main +' + +test_expect_success 'push -u with push.default=current' ' + default_u_setup current && + git push -u && + check_config main upstream refs/heads/main && + git push -u upstream main:other && + git push -u && + check_config main upstream refs/heads/main +' + +test_expect_success 'push -u with push.default=upstream' ' + default_u_setup upstream && + git push -u && + check_config main upstream refs/heads/main && + git push -u upstream main:other && + git push -u && + check_config main upstream refs/heads/main +' + +check_empty_config() { + test_expect_code 1 git config "branch.$1.remote" + test_expect_code 1 git config "branch.$1.merge" +} + +test_expect_success 'push -u with push.default=matching' ' + default_u_setup matching && + git branch test_u && + git branch test_u2 && + git push upstream main:test_u2 && + git push -u && + check_config main upstream refs/heads/main && + check_config test_u2 upstream refs/heads/test_u2 && + check_empty_config test_u +' + +test_expect_success 'push -u with push.default=nothing' ' + default_u_setup nothing && + git push -u && + check_config main upstream refs/heads/main && + git push -u upstream main:other && + git push -u && + check_config main upstream refs/heads/main +' + +test_expect_success 'push -u --dry-run' ' + git checkout main && + git push -u upstream main:other && + git push -u --dry-run && + check_config main upstream refs/heads/other +' + test_expect_success 'push -u --all' ' git branch all1 && git branch all2 && @@ -81,6 +150,13 @@ test_expect_success TTY 'progress messages go to tty' ' test_i18ngrep "Writing objects" err ' +test_expect_success TTY 'progress messages go to tty with default -u' ' + ensure_fresh_upstream && + + test_terminal git push -u >out 2>err && + test_i18ngrep "Writing objects" err +' + test_expect_success 'progress messages do not go to non-tty' ' ensure_fresh_upstream && @@ -89,6 +165,14 @@ test_expect_success 'progress messages do not go to non-tty' ' test_i18ngrep ! "Writing objects" err ' +test_expect_success 'progress messagesdo not go to non-tty (default -u)' ' + ensure_fresh_upstream && + + # skip progress messages, since stderr is non-tty + git push -u >out 2>err && + test_i18ngrep ! "Writing objects" err +' + test_expect_success 'progress messages go to non-tty (forced)' ' ensure_fresh_upstream && @@ -97,6 +181,14 @@ test_expect_success 'progress messages go to non-tty (forced)' ' test_i18ngrep "Writing objects" err ' +test_expect_success 'progress messages go to non-tty with default -u (forced)' ' + ensure_fresh_upstream && + + # force progress messages to stderr, even though it is non-tty + git push -u --progress >out 2>err && + test_i18ngrep "Writing objects" err +' + test_expect_success TTY 'push -q suppresses progress' ' ensure_fresh_upstream && @@ -104,6 +196,13 @@ test_expect_success TTY 'push -q suppresses progress' ' test_i18ngrep ! "Writing objects" err ' +test_expect_success TTY 'push -q suppresses progress (with default -u)' ' + ensure_fresh_upstream && + + test_terminal git push -u -q >out 2>err && + test_i18ngrep ! "Writing objects" err +' + test_expect_success TTY 'push --no-progress suppresses progress' ' ensure_fresh_upstream && @@ -112,6 +211,14 @@ test_expect_success TTY 'push --no-progress suppresses progress' ' test_i18ngrep ! "Writing objects" err ' +test_expect_success TTY 'push --no-progress suppresses progress (default -u)' ' + ensure_fresh_upstream && + + test_terminal git push -u --no-progress >out 2>err && + test_i18ngrep ! "Unpacking objects" err && + test_i18ngrep ! "Writing objects" err +' + test_expect_success TTY 'quiet push' ' ensure_fresh_upstream && @@ -126,4 +233,11 @@ test_expect_success TTY 'quiet push -u' ' test_must_be_empty output ' +test_expect_success TTY 'quiet push -u (default -u)' ' + ensure_fresh_upstream && + + test_terminal git push --quiet -u --no-progress 2>&1 | tee output && + test_must_be_empty output +' + test_done