From: "Alex Reed via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: Alex Reed <acreed4@gmail.com>, Alex Reed <acreed4@gmail.com>
Subject: [PATCH] branch, config: teach branch.autosetuprebase about 'merges' mode
Date: Wed, 20 Jan 2021 00:20:09 +0000 [thread overview]
Message-ID: <pull.948.git.git.1611102010142.gitgitgadget@gmail.com> (raw)
From: Alex Reed <acreed4@gmail.com>
branch.autoSetupRebase now honors the 'merges' flag, allowing tracking branches
to be auto-populated with 'branch.<name>.rebase = merges'. This allows complex
workflows to more easily retain non-trivial merges while rebasing branches on
pull operations. Seeding new branches with 'branch.<name>.rebase = true' is not
always sufficient (read: not project default) and requiring developers to
manually reconfigure every new branch is cumbersome and error-prone.
Signed-off-by: Alex Reed <acreed4@gmail.com>
---
branch, config: teach branch.autosetuprebase about 'merges' mode
branch.autoSetupRebase now honors the 'merges' flag, allowing tracking
branches to be auto-populated with 'branch..rebase = merges'. This
allows complex workflows to more easily retain non-trivial merges while
rebasing branches on pull operations. Seeding new branches with
'branch..rebase = true' is not always sufficient (read: not project
default) and requiring developers to manually reconfigure every new
branch is cumbersome and error-prone.
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-948%2Facr4%2Fautosetuprebase-merges-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-948/acr4/autosetuprebase-merges-v1
Pull-Request: https://github.com/git/git/pull/948
Documentation/config/branch.txt | 2 +
branch.c | 70 ++++++++++++++++++++-------------
cache.h | 3 +-
config.c | 2 +
t/t3200-branch.sh | 45 +++++++++++++++++++++
t/t5601-clone.sh | 14 ++++++-
6 files changed, 107 insertions(+), 29 deletions(-)
diff --git a/Documentation/config/branch.txt b/Documentation/config/branch.txt
index cc5f3249fc5..98410bf003f 100644
--- a/Documentation/config/branch.txt
+++ b/Documentation/config/branch.txt
@@ -21,6 +21,8 @@ branch.autoSetupRebase::
remote-tracking branches.
When `always`, rebase will be set to true for all tracking
branches.
+ When `merges`, rebase will be set to `merges` for all tracking
+ branches.
See "branch.autoSetupMerge" for details on how to set up a
branch to track another branch.
This option defaults to never.
diff --git a/branch.c b/branch.c
index 9c9dae1eae3..b197fe0bbb3 100644
--- a/branch.c
+++ b/branch.c
@@ -34,17 +34,25 @@ static int find_tracked_branch(struct remote *remote, void *priv)
return 0;
}
-static int should_setup_rebase(const char *origin)
+typedef enum {
+ REBASE_FALSE,
+ REBASE_TRUE,
+ REBASE_MERGES
+} rebase_type;
+
+static rebase_type should_setup_rebase(const char *origin)
{
switch (autorebase) {
case AUTOREBASE_NEVER:
- return 0;
+ return REBASE_FALSE;
case AUTOREBASE_LOCAL:
- return origin == NULL;
+ return origin == NULL ? REBASE_TRUE : REBASE_FALSE;
case AUTOREBASE_REMOTE:
- return origin != NULL;
+ return origin != NULL ? REBASE_TRUE : REBASE_FALSE;
case AUTOREBASE_ALWAYS:
- return 1;
+ return REBASE_TRUE;
+ case AUTOREBASE_MERGES:
+ return REBASE_MERGES;
}
return 0;
}
@@ -59,7 +67,8 @@ int install_branch_config(int flag, const char *local, const char *origin, const
{
const char *shortname = NULL;
struct strbuf key = STRBUF_INIT;
- int rebasing = should_setup_rebase(origin);
+ rebase_type rebasing = should_setup_rebase(origin);
+ struct strbuf method = STRBUF_INIT;
if (skip_prefix(remote, "refs/heads/", &shortname)
&& !strcmp(local, shortname)
@@ -78,44 +87,51 @@ int install_branch_config(int flag, const char *local, const char *origin, const
if (git_config_set_gently(key.buf, remote) < 0)
goto out_err;
- if (rebasing) {
- strbuf_reset(&key);
- strbuf_addf(&key, "branch.%s.rebase", local);
- if (git_config_set_gently(key.buf, "true") < 0)
- goto out_err;
+ strbuf_reset(&key);
+ strbuf_addf(&key, "branch.%s.rebase", local);
+ switch(rebasing) {
+ case REBASE_TRUE:
+ strbuf_addstr(&method, " by rebasing");
+ if(git_config_set_gently(key.buf, "true") < 0)
+ goto out_err;
+ break;
+ case REBASE_MERGES:
+ strbuf_addstr(&method, " by rebasing while preserving merges");
+ if (git_config_set_gently(key.buf, "merges") < 0)
+ goto out_err;
+ break;
+ default:;
}
strbuf_release(&key);
if (flag & BRANCH_CONFIG_VERBOSE) {
if (shortname) {
if (origin)
- printf_ln(rebasing ?
- _("Branch '%s' set up to track remote branch '%s' from '%s' by rebasing.") :
- _("Branch '%s' set up to track remote branch '%s' from '%s'."),
- local, shortname, origin);
+ printf_ln(
+ _("Branch '%s' set up to track remote branch '%s' from '%s'%s."),
+ local, shortname, origin, method.buf);
else
- printf_ln(rebasing ?
- _("Branch '%s' set up to track local branch '%s' by rebasing.") :
- _("Branch '%s' set up to track local branch '%s'."),
- local, shortname);
+ printf_ln(
+ _("Branch '%s' set up to track local branch '%s'%s."),
+ local, shortname, method.buf);
} else {
if (origin)
- printf_ln(rebasing ?
- _("Branch '%s' set up to track remote ref '%s' by rebasing.") :
- _("Branch '%s' set up to track remote ref '%s'."),
- local, remote);
+ printf_ln(
+ _("Branch '%s' set up to track remote ref '%s'%s."),
+ local, remote, method.buf);
else
- printf_ln(rebasing ?
- _("Branch '%s' set up to track local ref '%s' by rebasing.") :
- _("Branch '%s' set up to track local ref '%s'."),
- local, remote);
+ printf_ln(
+ _("Branch '%s' set up to track local ref '%s'%s."),
+ local, remote, method.buf);
}
}
+ strbuf_release(&method);
return 0;
out_err:
strbuf_release(&key);
+ strbuf_release(&method);
error(_("Unable to write upstream branch configuration"));
advise(_(tracking_advice),
diff --git a/cache.h b/cache.h
index eefa93b08f8..5a378bcd534 100644
--- a/cache.h
+++ b/cache.h
@@ -995,7 +995,8 @@ enum rebase_setup_type {
AUTOREBASE_NEVER = 0,
AUTOREBASE_LOCAL,
AUTOREBASE_REMOTE,
- AUTOREBASE_ALWAYS
+ AUTOREBASE_ALWAYS,
+ AUTOREBASE_MERGES
};
enum push_default_type {
diff --git a/config.c b/config.c
index 4c0cf3a1c15..28d813f2595 100644
--- a/config.c
+++ b/config.c
@@ -1443,6 +1443,8 @@ static int git_default_branch_config(const char *var, const char *value)
autorebase = AUTOREBASE_REMOTE;
else if (!strcmp(value, "always"))
autorebase = AUTOREBASE_ALWAYS;
+ else if (!strcmp(value, "merges"))
+ autorebase = AUTOREBASE_MERGES;
else
return error(_("malformed value for %s"), var);
return 0;
diff --git a/t/t3200-branch.sh b/t/t3200-branch.sh
index 0af3b85d172..056192f72c5 100755
--- a/t/t3200-branch.sh
+++ b/t/t3200-branch.sh
@@ -1222,6 +1222,51 @@ test_expect_success 'autosetuprebase always on an untracked remote branch' '
test "z$(git config branch.myr20.rebase)" = z
'
+test_expect_success 'autosetuprebase merges on a tracked local branch' '
+ git config branch.autosetuprebase merges &&
+ git config remote.local.url . &&
+ git config remote.local.fetch refs/heads/*:refs/remotes/local/* &&
+ (git show-ref -q refs/remotes/local/o || git fetch local) &&
+ git branch mybase21 &&
+ git branch --track myr21 mybase3 &&
+ test "$(git config branch.myr21.remote)" = . &&
+ test "$(git config branch.myr21.merge)" = refs/heads/mybase3 &&
+ test "$(git config branch.myr21.rebase)" = merges
+'
+
+test_expect_success 'autosetuprebase merges on a tracked remote branch' '
+ git config branch.autosetuprebase merges &&
+ git config remote.local.url . &&
+ git config remote.local.fetch refs/heads/*:refs/remotes/local/* &&
+ (git show-ref -q refs/remotes/local/main || git fetch local) &&
+ git branch --track myr22 local/main &&
+ test "$(git config branch.myr22.remote)" = local &&
+ test "$(git config branch.myr22.merge)" = refs/heads/main &&
+ test "$(git config branch.myr22.rebase)" = merges
+'
+
+test_expect_success 'autosetuprebase merges on an untracked local branch' '
+ git config branch.autosetuprebase merges &&
+ git config remote.local.url . &&
+ git config remote.local.fetch refs/heads/*:refs/remotes/local/* &&
+ (git show-ref -q refs/remotes/local/main || git fetch local) &&
+ git branch --no-track myr23 mybase2 &&
+ test "z$(git config branch.myr23.remote)" = z &&
+ test "z$(git config branch.myr23.merge)" = z &&
+ test "z$(git config branch.myr23.rebase)" = z
+'
+
+test_expect_success 'autosetuprebase merges on an untracked remote branch' '
+ git config branch.autosetuprebase merges &&
+ git config remote.local.url . &&
+ git config remote.local.fetch refs/heads/*:refs/remotes/local/* &&
+ (git show-ref -q refs/remotes/local/main || git fetch local) &&
+ git branch --no-track myr24 local/main &&
+ test "z$(git config branch.myr24.remote)" = z &&
+ test "z$(git config branch.myr24.merge)" = z &&
+ test "z$(git config branch.myr24.rebase)" = z
+'
+
test_expect_success 'autosetuprebase always on detached HEAD' '
git config branch.autosetupmerge always &&
test_when_finished git checkout main &&
diff --git a/t/t5601-clone.sh b/t/t5601-clone.sh
index 7df3c5373ae..10983191439 100755
--- a/t/t5601-clone.sh
+++ b/t/t5601-clone.sh
@@ -210,7 +210,7 @@ test_expect_success 'clone a void' '
test_cmp target-6/.git/config target-7/.git/config
'
-test_expect_success 'clone respects global branch.autosetuprebase' '
+test_expect_success 'clone respects global branch.autosetuprebase remote' '
(
test_config="$HOME/.gitconfig" &&
git config -f "$test_config" branch.autosetuprebase remote &&
@@ -222,6 +222,18 @@ test_expect_success 'clone respects global branch.autosetuprebase' '
)
'
+test_expect_success 'clone respects global branch.autosetuprebase merges' '
+ (
+ test_config="$HOME/.gitconfig" &&
+ git config -f "$test_config" branch.autosetuprebase merges &&
+ rm -fr dst &&
+ git clone src dst &&
+ cd dst &&
+ actual="z$(git config branch.master.rebase)" &&
+ test zmerges = $actual
+ )
+'
+
test_expect_success 'respect url-encoding of file://' '
git init x+y &&
git clone "file://$PWD/x+y" xy-url-1 &&
base-commit: 66e871b6647ffea61a77a0f82c7ef3415f1ee79c
--
gitgitgadget
reply other threads:[~2021-01-20 0:21 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pull.948.git.git.1611102010142.gitgitgadget@gmail.com \
--to=gitgitgadget@gmail.com \
--cc=acreed4@gmail.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).