From: Warren He <pickydaemon@gmail.com>
To: git@vger.kernel.org
Cc: Warren He <wh109@yahoo.com>
Subject: [PATCH] rebase: introduce --update-branches option
Date: Mon, 2 Sep 2019 16:41:09 -0700 [thread overview]
Message-ID: <20190902234109.2922-2-wh109@yahoo.com> (raw)
In-Reply-To: <20190902234109.2922-1-wh109@yahoo.com>
Rebasing normally updates the current branch to the rewritten version.
If any other branches point to commits rewritten along the way, those
remain untouched. This commit adds an `--update-branches` option, which
instructs the command to update any such branches that it encounters to
point to the rewritten versions of those commits.
Signed-off-by: Warren He <wh109@yahoo.com>
---
Documentation/git-rebase.txt | 8 +++++
builtin/rebase.c | 13 ++++++--
sequencer.c | 68 ++++++++++++++++++++++++++++++++++++++-
sequencer.h | 6 ++--
t/t3431-rebase-update-branches.sh | 64 ++++++++++++++++++++++++++++++++++++
5 files changed, 154 insertions(+), 5 deletions(-)
create mode 100755 t/t3431-rebase-update-branches.sh
diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
index 6156609..c37a0db 100644
--- a/Documentation/git-rebase.txt
+++ b/Documentation/git-rebase.txt
@@ -246,6 +246,13 @@ leave out at most one of A and B, in which case it defaults to HEAD.
+
See also INCOMPATIBLE OPTIONS below.
+--update-branches::
+ If there are branch refs that point to commits that will be
+ reapplied, add shell commands to the todo list to update those
+ refs to point to the commits in the final history.
++
+See also INCOMPATIBLE OPTIONS below.
+
--allow-empty-message::
By default, rebasing commits with an empty message will fail.
This option overrides that behavior, allowing commits with empty
@@ -535,6 +542,7 @@ are incompatible with the following options:
* --interactive
* --exec
* --keep-empty
+ * --update-branches
* --edit-todo
* --root when used in combination with --onto
diff --git a/builtin/rebase.c b/builtin/rebase.c
index 670096c..cf87c53 100644
--- a/builtin/rebase.c
+++ b/builtin/rebase.c
@@ -78,6 +78,7 @@ struct rebase_options {
int signoff;
int allow_rerere_autoupdate;
int keep_empty;
+ int update_branches;
int autosquash;
char *gpg_sign_opt;
int autostash;
@@ -349,8 +350,8 @@ static int do_interactive_rebase(struct rebase_options *opts, unsigned flags)
split_exec_commands(opts->cmd, &commands);
ret = complete_action(the_repository, &replay, flags,
- shortrevisions, opts->onto_name, opts->onto, head_hash,
- &commands, opts->autosquash, &todo_list);
+ shortrevisions, opts->onto_name, opts->onto, opts->head_name,
+ head_hash, &commands, opts->autosquash, &todo_list);
}
string_list_clear(&commands, 0);
@@ -375,6 +376,7 @@ static int run_rebase_interactive(struct rebase_options *opts,
flags |= opts->rebase_merges ? TODO_LIST_REBASE_MERGES : 0;
flags |= opts->rebase_cousins > 0 ? TODO_LIST_REBASE_COUSINS : 0;
flags |= command == ACTION_SHORTEN_OIDS ? TODO_LIST_SHORTEN_IDS : 0;
+ flags |= opts->update_branches ? TODO_LIST_UPDATE_BRANCHES : 0;
switch (command) {
case ACTION_NONE: {
@@ -447,6 +449,8 @@ int cmd_rebase__interactive(int argc, const char **argv, const char *prefix)
OPT_NEGBIT(0, "ff", &opts.flags, N_("allow fast-forward"),
REBASE_FORCE),
OPT_BOOL(0, "keep-empty", &opts.keep_empty, N_("keep empty commits")),
+ OPT_BOOL(0, "update-branches", &opts.update_branches,
+ N_("update branches that point to reapplied commits")),
OPT_BOOL(0, "allow-empty-message", &opts.allow_empty_message,
N_("allow commits with empty messages")),
OPT_BOOL(0, "rebase-merges", &opts.rebase_merges, N_("rebase merge commits")),
@@ -1453,6 +1457,8 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
OPT_RERERE_AUTOUPDATE(&options.allow_rerere_autoupdate),
OPT_BOOL('k', "keep-empty", &options.keep_empty,
N_("preserve empty commits during rebase")),
+ OPT_BOOL(0, "update-branches", &options.update_branches,
+ N_("update branches that point to reapplied commits")),
OPT_BOOL(0, "autosquash", &options.autosquash,
N_("move commits that begin with "
"squash!/fixup! under -i")),
@@ -1710,6 +1716,9 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
if (options.keep_empty)
imply_interactive(&options, "--keep-empty");
+ if (options.update_branches)
+ imply_interactive(&options, "--update-branches");
+
if (gpg_sign) {
free(options.gpg_sign_opt);
options.gpg_sign_opt = xstrfmt("-S%s", gpg_sign);
diff --git a/sequencer.c b/sequencer.c
index 34ebf8e..c6749ff 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4901,6 +4901,69 @@ void todo_list_add_exec_commands(struct todo_list *todo_list,
todo_list->alloc = alloc;
}
+/*
+ * Add commands to update branch refs after the todo list would pick a commit
+ * that a branch ref points to.
+ */
+static void todo_list_add_branch_updates(struct todo_list *todo_list,
+ const char *head_name)
+{
+ struct strbuf *buf = &todo_list->buf;
+ int i, nr = 0, alloc = 0;
+ struct todo_item *items = NULL;
+
+ load_ref_decorations(NULL, 0);
+
+ for (i = 0; i < todo_list->nr; i++) {
+ const struct todo_item *item = &todo_list->items[i];
+ enum todo_command command = item->command;
+ const struct name_decoration *decoration;
+
+ ALLOC_GROW(items, nr + 1, alloc);
+ items[nr++] = todo_list->items[i];
+
+ switch (command) {
+ case TODO_PICK:
+ case TODO_MERGE:
+ break;
+ default:
+ continue;
+ }
+
+ decoration = get_name_decoration(&item->commit->object);
+ for (; decoration; decoration = decoration->next) {
+ size_t base_offset, pretty_name_len;
+ const char *pretty_name;
+
+ if (decoration->type != DECORATION_REF_LOCAL)
+ continue;
+ if (!strcmp(decoration->name, head_name))
+ // Rebase itself will update the current branch for us.
+ continue;
+
+ base_offset = buf->len;
+ pretty_name = prettify_refname(decoration->name);
+ pretty_name_len = strlen(pretty_name);
+ strbuf_addstr(buf, "exec git branch -f ");
+ strbuf_addstr(buf, pretty_name);
+ strbuf_addch(buf, '\n');
+
+ ALLOC_GROW(items, nr + 1, alloc);
+ items[nr++] = (struct todo_item) {
+ .command = TODO_EXEC,
+ .offset_in_buf = base_offset,
+ .arg_offset = base_offset + strlen("exec "),
+ .arg_len = strlen("git branch -f ") + pretty_name_len,
+ };
+ }
+ }
+
+ FREE_AND_NULL(todo_list->items);
+ todo_list->items = items;
+ todo_list->nr = nr;
+ todo_list->alloc = alloc;
+}
+
static void todo_list_to_strbuf(struct repository *r, struct todo_list *todo_list,
struct strbuf *buf, int num, unsigned flags)
{
@@ -5051,7 +5114,7 @@ static int skip_unnecessary_picks(struct repository *r,
int complete_action(struct repository *r, struct replay_opts *opts, unsigned flags,
const char *shortrevisions, const char *onto_name,
- struct commit *onto, const char *orig_head,
+ struct commit *onto, const char *head_name, const char *orig_head,
struct string_list *commands, unsigned autosquash,
struct todo_list *todo_list)
{
@@ -5076,6 +5139,9 @@ int complete_action(struct repository *r, struct replay_opts *opts, unsigned fla
if (commands->nr)
todo_list_add_exec_commands(todo_list, commands);
+ if (flags & TODO_LIST_UPDATE_BRANCHES)
+ todo_list_add_branch_updates(todo_list, head_name);
+
if (count_commands(todo_list) == 0) {
apply_autostash(opts);
sequencer_remove_state(opts);
diff --git a/sequencer.h b/sequencer.h
index 6704acb..69c6f71 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -143,6 +143,7 @@ int sequencer_remove_state(struct replay_opts *opts);
*/
#define TODO_LIST_REBASE_COUSINS (1U << 4)
#define TODO_LIST_APPEND_TODO_HELP (1U << 5)
+#define TODO_LIST_UPDATE_BRANCHES (1U << 6)
int sequencer_make_script(struct repository *r, struct strbuf *out, int argc,
const char **argv, unsigned flags);
@@ -152,8 +153,9 @@ void todo_list_add_exec_commands(struct todo_list *todo_list,
int check_todo_list_from_file(struct repository *r);
int complete_action(struct repository *r, struct replay_opts *opts, unsigned flags,
const char *shortrevisions, const char *onto_name,
- struct commit *onto, const char *orig_head, struct string_list *commands,
- unsigned autosquash, struct todo_list *todo_list);
+ struct commit *onto, const char *head_name, const char *orig_head,
+ struct string_list *commands, unsigned autosquash,
+ struct todo_list *todo_list);
int todo_list_rearrange_squash(struct todo_list *todo_list);
/*
diff --git a/t/t3431-rebase-update-branches.sh b/t/t3431-rebase-update-branches.sh
new file mode 100755
index 0000000..221c25d
--- /dev/null
+++ b/t/t3431-rebase-update-branches.sh
@@ -0,0 +1,64 @@
+#!/bin/sh
+
+test_description='git rebase -i --update-branches
+
+This test runs git rebase, moving branch refs that point to commits
+that are reapplied.
+
+Initial setup:
+
+ A - B (master)
+ |\
+ | C (linear-early)
+ | \
+ | D (linear-late)
+ |\
+ | E (feat-e)
+ \ \
+ F | (feat-f)
+ \|
+ G (interim)
+ \
+ H (my-dev)
+'
+. ./test-lib.sh
+
+test_expect_success 'setup linear' '
+ test_commit A &&
+ test_commit B &&
+ git checkout -b linear-early A &&
+ test_commit C &&
+ git checkout -b linear-late &&
+ test_commit D
+'
+
+test_expect_success 'smoketest linear' '
+ git rebase --update-branches master
+'
+
+test_expect_success 'check linear' '
+ git rev-parse linear-early:B.t
+'
+
+test_expect_success 'setup merge' '
+ git checkout -b feat-e A &&
+ test_commit E &&
+ git checkout -b feat-f A &&
+ test_commit F &&
+ git checkout -b interim &&
+ test_merge G feat-e &&
+ git checkout -b my-dev &&
+ test_commit H
+'
+
+test_expect_success 'smoketest merge' '
+ git rebase -r --update-branches master
+'
+
+test_expect_success 'check merge' '
+ git rev-parse feat-e:B.t &&
+ git rev-parse feat-f:B.t &&
+ git rev-parse interim:B.t
+'
+
+test_done
--
2.7.4
next prev parent reply other threads:[~2019-09-02 23:41 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-02 23:41 [PATCH] rebase: introduce --update-branches option Warren He
2019-09-02 23:41 ` Warren He [this message]
2019-09-03 13:26 ` Johannes Schindelin
2019-09-07 23:44 ` [PATCH v2] " Warren He
2019-09-07 23:44 ` Warren He
2019-09-09 10:44 ` Phillip Wood
2019-09-09 14:13 ` Johannes Schindelin
2019-09-09 18:11 ` Junio C Hamano
2019-09-23 9:40 ` Phillip Wood
2019-09-03 0:50 ` [PATCH] " brian m. carlson
2019-09-03 1:21 ` Junio C Hamano
2019-09-03 1:39 ` Taylor Blau
2019-09-07 23:41 ` Warren He
2019-09-08 18:04 ` brian m. carlson
2019-09-03 12:19 ` Phillip Wood
2019-09-07 23:43 ` Warren He
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190902234109.2922-2-wh109@yahoo.com \
--to=pickydaemon@gmail.com \
--cc=git@vger.kernel.org \
--cc=wh109@yahoo.com \
/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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).