Git Mailing List Archive on lore.kernel.org
 help / color / Atom feed
From: Phillip Wood <phillip.wood123@gmail.com>
To: Elijah Newren via GitGitGadget <gitgitgadget@gmail.com>,
	git@vger.kernel.org
Cc: Johannes.Schindelin@gmx.de, phillip.wood@dunelm.org.uk,
	liu.denton@gmail.com, gitster@pobox.com, plroskin@gmail.com,
	alban.gruin@gmail.com, szeder.dev@gmail.com, jrnieder@gmail.com,
	emilyshaffer@google.com, Elijah Newren <newren@gmail.com>
Subject: Re: [PATCH v4 03/19] rebase (interactive-backend): make --keep-empty the default
Date: Sun, 9 Feb 2020 15:59:40 +0000
Message-ID: <805e3edd-d479-16b8-c67f-d8ea34176389@gmail.com> (raw)
In-Reply-To: <179f82ab83e669ca8da7ce999607cae584bc18df.1579155273.git.gitgitgadget@gmail.com>

Hi Elijah

On 16/01/2020 06:14, Elijah Newren via GitGitGadget wrote:
> From: Elijah Newren <newren@gmail.com>
> 
> Different rebase backends have different treatment for commits which
> start empty (i.e. have no changes relative to their parent), and the
> --keep-empty option was added at some point to allow adjusting behavior
> for the interactive backend.

--keep-empty was introduced by 90e1818f9a ("git-rebase: add keep_empty 
flag", 2012-04-20) and was respected for am based rebases as well 
(although the implementation was broken if there were any conflicts)

> The handling of commits which start empty
> is actually quite similar to commit b00bf1c9a8dd (git-rebase: make
> --allow-empty-message the default, 2018-06-27), which pointed out that
> the behavior for various backends is often more happenstance than
> design.  The specific change made in that commit is actually quite
> relevant as well and much of the logic there directly applies here.
> 
> It makes a lot of sense in 'git commit' to error out on the creation of
> empty commits, unless an override flag is provided.  However, once
> someone determines that there is a rare case that merits using the
> manual override to create such a commit, it is somewhere between
> annoying and harmful to have to take extra steps to keep such
> intentional commits around.  Granted, empty commits are quite rare,
> which is why handling of them doesn't get considered much and folks tend
> to defer to existing (accidental) behavior and assume there was a reason
> for it, leading them to just add flags (--keep-empty in this case) that
> allow them to override the bad defaults.  Fix the interactive backend so
> that --keep-empty is the default, much like we did with
> --allow-empty-message.  The am backend should also be fixed to have
> --keep-empty semantics for commits that start empty, but that is not
> included in this patch other than a testcase documenting the failure.

The detailed commit message is great, however I think it would be good 
to point out that this changes things so that --no-keep-empty is parsed 
but does nothing - it is no longer possible to get that behavior. I'm 
not  entirely sure about making --keep-empty hidden given that the am 
backend has a different behavior without it but as we're changing the 
default backend it probably does not matter and it looks like 
--keep-empty combined with an am specific option will still error out 
which is good.

Overall I like this change and it simplifies the implementation which is 
nice. Hopefully no one was relying on --no-keep-empty

> Note that there was one test in t3421 which appears to have been written
> expecting --keep-empty to not be the default as correct behavior.  This
> test was introduced in commit 00b8be5a4d38 ("add tests for rebasing of
> empty commits", 2013-06-06), which was part of a series focusing on
> rebase topology and which had an interesting original cover letter at
> https://lore.kernel.org/git/1347949878-12578-1-git-send-email-martinvonz@gmail.com/
> which noted
>      Your input especially appreciated on whether you agree with the
>      intent of the test cases.
> and then went into a long example about how one of the many tests added
> had several questions about whether it was correct.  As such, I believe
> most the tests in that series were about testing rebase topology with as
> many different flags as possible and were not trying to state in general
> how those flags should behave otherwise.
> 
> Signed-off-by: Elijah Newren <newren@gmail.com>
> ---
>   Documentation/git-rebase.txt      | 24 ++++++-----
>   builtin/rebase.c                  | 28 ++++++++----
>   rebase-interactive.c              |  7 +--
>   rebase-interactive.h              |  2 +-
>   sequencer.c                       | 20 +++------
>   sequencer.h                       |  2 +-
>   t/t3421-rebase-topology-linear.sh | 16 +++----
>   t/t3424-rebase-empty.sh           | 72 +++++++++++++++++++++++++++++++
>   t/t3427-rebase-subtree.sh         | 12 +++---
>   9 files changed, 128 insertions(+), 55 deletions(-)
>   create mode 100755 t/t3424-rebase-empty.sh
> 
> diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
> index c83be7ffc2..1d19542d79 100644
> --- a/Documentation/git-rebase.txt
> +++ b/Documentation/git-rebase.txt
> @@ -259,10 +259,13 @@ See also INCOMPATIBLE OPTIONS below.
>   	unchanged as a result.
>   
>   --keep-empty::
> -	Keep the commits that do not change anything from its
> -	parents in the result.
> +	No-op.  Rebasing commits that started empty (had no change
> +	relative to their parent) used to fail and this option would
> +	override that behavior, allowing commits with empty changes to
> +	be rebased.  Now commits with no changes do not cause rebasing
> +	to halt.

Maybe mention that --no-keep-empty is no longer respected?

> -See also INCOMPATIBLE OPTIONS below.
> +See also BEHAVIORAL DIFFERENCES and INCOMPATIBLE OPTIONS below.
>   
>   --allow-empty-message::
>   	No-op.  Rebasing commits with an empty message used to fail
> @@ -577,15 +580,14 @@ There are some subtle differences how the backends behave.
>   Empty commits
>   ~~~~~~~~~~~~~
>   
> -The am backend drops any "empty" commits, regardless of whether the
> -commit started empty (had no changes relative to its parent to
> -start with) or ended empty (all changes were already applied
> -upstream in other commits).
> +The am backend unfortunately drops intentionally empty commits, i.e.
> +commits that started empty, though these are rare in practice.  It
> +also drops commits that become empty and has no option for controlling
> +this behavior.
>   
> -The interactive backend drops commits by default that
> -started empty and halts if it hits a commit that ended up empty.
> -The `--keep-empty` option exists for the interactive backend to allow
> -it to keep commits that started empty.
> +The interactive backend keeps intentionally empty commits.
> +Unfortunately, it always halts whenever it runs across a commit that
> +becomes empty, even when the rebase is not explicitly interactive.
>   
>   Directory rename detection
>   ~~~~~~~~~~~~~~~~~~~~~~~~~~
> diff --git a/builtin/rebase.c b/builtin/rebase.c
> index faa4e0d406..537b3241ce 100644
> --- a/builtin/rebase.c
> +++ b/builtin/rebase.c
> @@ -77,7 +77,6 @@ struct rebase_options {
>   	const char *action;
>   	int signoff;
>   	int allow_rerere_autoupdate;
> -	int keep_empty;
>   	int autosquash;
>   	char *gpg_sign_opt;
>   	int autostash;
> @@ -375,7 +374,6 @@ static int run_rebase_interactive(struct rebase_options *opts,
>   
>   	git_config_get_bool("rebase.abbreviatecommands", &abbreviate_commands);
>   
> -	flags |= opts->keep_empty ? TODO_LIST_KEEP_EMPTY : 0;
>   	flags |= abbreviate_commands ? TODO_LIST_ABBREVIATE_CMDS : 0;
>   	flags |= opts->rebase_merges ? TODO_LIST_REBASE_MERGES : 0;
>   	flags |= opts->rebase_cousins > 0 ? TODO_LIST_REBASE_COUSINS : 0;
> @@ -439,6 +437,17 @@ static int run_rebase_interactive(struct rebase_options *opts,
>   	return ret;
>   }
>   
> +static int parse_opt_keep_empty(const struct option *opt, const char *arg,
> +				int unset)
> +{
> +	struct rebase_options *opts = opt->value;
> +
> +	BUG_ON_OPT_ARG(arg);
> +
> +	opts->type = REBASE_INTERACTIVE;
> +	return 0;
> +}

This doesn't seem to handle --no-keep-empty (but looking through the 
patch that seems to be intentional)

>   static const char * const builtin_rebase_interactive_usage[] = {
>   	N_("git rebase--interactive [<options>]"),
>   	NULL
> @@ -452,7 +461,10 @@ int cmd_rebase__interactive(int argc, const char **argv, const char *prefix)
>   	struct option options[] = {
>   		OPT_NEGBIT(0, "ff", &opts.flags, N_("allow fast-forward"),
>   			   REBASE_FORCE),
> -		OPT_BOOL(0, "keep-empty", &opts.keep_empty, N_("keep empty commits")),
> +		{ OPTION_CALLBACK, 'k', "keep-empty", &options, NULL,
> +			N_("(DEPRECATED) keep empty commits"),
> +			PARSE_OPT_NOARG | PARSE_OPT_HIDDEN,
> +			parse_opt_keep_empty },
>   		OPT_BOOL_F(0, "allow-empty-message", &opts.allow_empty_message,
>   			   N_("allow commits with empty messages"),
>   			   PARSE_OPT_HIDDEN),
> @@ -1145,7 +1157,6 @@ static int run_specific_rebase(struct rebase_options *opts, enum action action)
>   		opts->allow_rerere_autoupdate ?
>   			opts->allow_rerere_autoupdate == RERERE_AUTOUPDATE ?
>   			"--rerere-autoupdate" : "--no-rerere-autoupdate" : "");
> -	add_var(&script_snippet, "keep_empty", opts->keep_empty ? "yes" : "");
>   	add_var(&script_snippet, "autosquash", opts->autosquash ? "t" : "");
>   	add_var(&script_snippet, "gpg_sign_opt", opts->gpg_sign_opt);
>   	add_var(&script_snippet, "cmd", opts->cmd);
> @@ -1483,8 +1494,10 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
>   				 "ignoring them"),
>   			      REBASE_PRESERVE_MERGES, PARSE_OPT_HIDDEN),
>   		OPT_RERERE_AUTOUPDATE(&options.allow_rerere_autoupdate),
> -		OPT_BOOL('k', "keep-empty", &options.keep_empty,
> -			 N_("preserve empty commits during rebase")),
> +		{ OPTION_CALLBACK, 'k', "keep-empty", &options, NULL,
> +			N_("(DEPRECATED) keep empty commits"),
> +			PARSE_OPT_NOARG | PARSE_OPT_HIDDEN,
> +			parse_opt_keep_empty },
>   		OPT_BOOL(0, "autosquash", &options.autosquash,
>   			 N_("move commits that begin with "
>   			    "squash!/fixup! under -i")),
> @@ -1747,9 +1760,6 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
>   	if (!(options.flags & REBASE_NO_QUIET))
>   		argv_array_push(&options.git_am_opts, "-q");
>   
> -	if (options.keep_empty)
> -		imply_interactive(&options, "--keep-empty");
> -
>   	if (gpg_sign) {
>   		free(options.gpg_sign_opt);
>   		options.gpg_sign_opt = xstrfmt("-S%s", gpg_sign);
> diff --git a/rebase-interactive.c b/rebase-interactive.c
> index aa18ae82b7..7e7b4061bf 100644
> --- a/rebase-interactive.c
> +++ b/rebase-interactive.c
> @@ -28,7 +28,7 @@ static enum missing_commit_check_level get_missing_commit_check_level(void)
>   	return MISSING_COMMIT_CHECK_IGNORE;
>   }
>   
> -void append_todo_help(unsigned keep_empty, int command_count,
> +void append_todo_help(int command_count,
>   		      const char *shortrevisions, const char *shortonto,
>   		      struct strbuf *buf)
>   {
> @@ -80,11 +80,6 @@ void append_todo_help(unsigned keep_empty, int command_count,
>   			"the rebase will be aborted.\n\n");
>   
>   	strbuf_add_commented_lines(buf, msg, strlen(msg));
> -
> -	if (!keep_empty) {
> -		msg = _("Note that empty commits are commented out");
> -		strbuf_add_commented_lines(buf, msg, strlen(msg));
> -	}
>   }
>   
>   int edit_todo_list(struct repository *r, struct todo_list *todo_list,
> diff --git a/rebase-interactive.h b/rebase-interactive.h
> index 44dbb06311..05354ca341 100644
> --- a/rebase-interactive.h
> +++ b/rebase-interactive.h
> @@ -5,7 +5,7 @@ struct strbuf;
>   struct repository;
>   struct todo_list;
>   
> -void append_todo_help(unsigned keep_empty, int command_count,
> +void append_todo_help(int command_count,
>   		      const char *shortrevisions, const char *shortonto,
>   		      struct strbuf *buf);
>   int edit_todo_list(struct repository *r, struct todo_list *todo_list,
> diff --git a/sequencer.c b/sequencer.c
> index b9dbf1adb0..c21fc202b1 100644
> --- a/sequencer.c
> +++ b/sequencer.c
> @@ -1489,7 +1489,7 @@ static int allow_empty(struct repository *r,
>   		       struct replay_opts *opts,
>   		       struct commit *commit)
>   {
> -	int index_unchanged, empty_commit;
> +	int index_unchanged, originally_empty;
>   
>   	/*
>   	 * Three cases:
> @@ -1513,10 +1513,10 @@ static int allow_empty(struct repository *r,
>   	if (opts->keep_redundant_commits)
>   		return 1;
>   
> -	empty_commit = is_original_commit_empty(commit);
> -	if (empty_commit < 0)
> -		return empty_commit;
> -	if (!empty_commit)
> +	originally_empty = is_original_commit_empty(commit);
> +	if (originally_empty < 0)
> +		return originally_empty;
> +	if (!originally_empty)
>   		return 0;
>   	else
>   		return 1;
> @@ -4566,7 +4566,6 @@ static int make_script_with_merges(struct pretty_print_context *pp,
>   				   struct rev_info *revs, struct strbuf *out,
>   				   unsigned flags)
>   {
> -	int keep_empty = flags & TODO_LIST_KEEP_EMPTY;
>   	int rebase_cousins = flags & TODO_LIST_REBASE_COUSINS;
>   	int root_with_onto = flags & TODO_LIST_ROOT_WITH_ONTO;
>   	struct strbuf buf = STRBUF_INIT, oneline = STRBUF_INIT;
> @@ -4629,8 +4628,6 @@ static int make_script_with_merges(struct pretty_print_context *pp,
>   		if (!to_merge) {
>   			/* non-merge commit: easy case */
>   			strbuf_reset(&buf);
> -			if (!keep_empty && is_empty)
> -				strbuf_addf(&buf, "%c ", comment_line_char);
>   			strbuf_addf(&buf, "%s %s %s", cmd_pick,
>   				    oid_to_hex(&commit->object.oid),
>   				    oneline.buf);
> @@ -4797,7 +4794,6 @@ int sequencer_make_script(struct repository *r, struct strbuf *out, int argc,
>   	struct pretty_print_context pp = {0};
>   	struct rev_info revs;
>   	struct commit *commit;
> -	int keep_empty = flags & TODO_LIST_KEEP_EMPTY;
>   	const char *insn = flags & TODO_LIST_ABBREVIATE_CMDS ? "p" : "pick";
>   	int rebase_merges = flags & TODO_LIST_REBASE_MERGES;
>   
> @@ -4833,12 +4829,10 @@ int sequencer_make_script(struct repository *r, struct strbuf *out, int argc,
>   		return make_script_with_merges(&pp, &revs, out, flags);
>   
>   	while ((commit = get_revision(&revs))) {
> -		int is_empty  = is_original_commit_empty(commit);
> +		int is_empty = is_original_commit_empty(commit);
>   
>   		if (!is_empty && (commit->object.flags & PATCHSAME))
>   			continue;
> -		if (!keep_empty && is_empty)
> -			strbuf_addf(out, "%c ", comment_line_char);
>   		strbuf_addf(out, "%s %s ", insn,
>   			    oid_to_hex(&commit->object.oid));
>   		pretty_print_commit(&pp, commit, out);
> @@ -4975,7 +4969,7 @@ int todo_list_write_to_file(struct repository *r, struct todo_list *todo_list,
>   
>   	todo_list_to_strbuf(r, todo_list, &buf, num, flags);
>   	if (flags & TODO_LIST_APPEND_TODO_HELP)
> -		append_todo_help(flags & TODO_LIST_KEEP_EMPTY, count_commands(todo_list),
> +		append_todo_help(count_commands(todo_list),
>   				 shortrevisions, shortonto, &buf);
>   
>   	res = write_message(buf.buf, buf.len, file, 0);
> diff --git a/sequencer.h b/sequencer.h
> index 9f9ae291e3..c165e0ff25 100644
> --- a/sequencer.h
> +++ b/sequencer.h
> @@ -132,7 +132,7 @@ int sequencer_rollback(struct repository *repo, struct replay_opts *opts);
>   int sequencer_skip(struct repository *repo, struct replay_opts *opts);
>   int sequencer_remove_state(struct replay_opts *opts);
>   
> -#define TODO_LIST_KEEP_EMPTY (1U << 0)
> +/* #define TODO_LIST_KEEP_EMPTY (1U << 0) */ /* No longer used */
>   #define TODO_LIST_SHORTEN_IDS (1U << 1)
>   #define TODO_LIST_ABBREVIATE_CMDS (1U << 2)
>   #define TODO_LIST_REBASE_MERGES (1U << 3)
> diff --git a/t/t3421-rebase-topology-linear.sh b/t/t3421-rebase-topology-linear.sh
> index 325072b0a3..57334dca7e 100755
> --- a/t/t3421-rebase-topology-linear.sh
> +++ b/t/t3421-rebase-topology-linear.sh
> @@ -205,17 +205,17 @@ test_expect_success 'setup of linear history for empty commit tests' '
>   test_run_rebase () {
>   	result=$1
>   	shift
> -	test_expect_$result "rebase $* drops empty commit" "
> +	test_expect_$result "rebase $* keeps begin-empty commits" "
>   		reset_rebase &&
> -		git rebase $* c l &&
> -		test_cmp_rev c HEAD~2 &&
> -		test_linear_range 'd l' c..
> +		git rebase $* j l &&
> +		test_cmp_rev c HEAD~4 &&
> +		test_linear_range 'j d k l' c..
>   	"
>   }
> -test_run_rebase success ''
> +test_run_rebase failure ''
>   test_run_rebase success -m
>   test_run_rebase success -i
> -test_have_prereq !REBASE_P || test_run_rebase success -p
> +test_have_prereq !REBASE_P || test_run_rebase failure -p
>   
>   test_run_rebase () {
>   	result=$1
> @@ -230,7 +230,7 @@ test_run_rebase () {
>   test_run_rebase success ''
>   test_run_rebase success -m
>   test_run_rebase success -i
> -test_have_prereq !REBASE_P || test_run_rebase failure -p
> +test_have_prereq !REBASE_P || test_run_rebase success -p
>   
>   test_run_rebase () {
>   	result=$1
> @@ -245,7 +245,7 @@ test_run_rebase () {
>   test_run_rebase success ''
>   test_run_rebase success -m
>   test_run_rebase success -i
> -test_have_prereq !REBASE_P || test_run_rebase failure -p
> +test_have_prereq !REBASE_P || test_run_rebase success -p
>   test_run_rebase success --rebase-merges
>   
>   #       m
> diff --git a/t/t3424-rebase-empty.sh b/t/t3424-rebase-empty.sh
> new file mode 100755
> index 0000000000..22d97e143b
> --- /dev/null
> +++ b/t/t3424-rebase-empty.sh
> @@ -0,0 +1,72 @@
> +#!/bin/sh
> +
> +test_description='git rebase of commits that start or become empty'
> +
> +. ./test-lib.sh
> +
> +test_expect_success 'setup test repository' '
> +	test_write_lines 1 2 3 4 5 6 7 8 9 10 >numbers &&
> +	test_write_lines A B C D E F G H I J >letters &&
> +	git add numbers letters &&
> +	git commit -m A &&
> +
> +	git branch upstream &&
> +	git branch localmods &&
> +
> +	git checkout upstream &&
> +	test_write_lines A B C D E >letters &&
> +	git add letters &&
> +	git commit -m B &&
> +
> +	test_write_lines 1 2 3 4 five 6 7 8 9 ten >numbers &&
> +	git add numbers &&
> +	git commit -m C &&
> +
> +	git checkout localmods &&
> +	test_write_lines 1 2 3 4 five 6 7 8 9 10 >numbers &&
> +	git add numbers &&
> +	git commit -m C2 &&
> +
> +	git commit --allow-empty -m D &&
> +
> +	test_write_lines A B C D E >letters &&
> +	git add letters &&
> +	git commit -m "Five letters ought to be enough for anybody"
> +'
> +
> +test_expect_failure 'rebase (am-backend) with a variety of empty commits' '
> +	test_when_finished "git rebase --abort" &&
> +	git checkout -B testing localmods &&
> +	# rebase (--am) should not drop commits that start empty
> +	git rebase upstream &&
> +
> +	test_write_lines D C B A >expect &&
> +	git log --format=%s >actual &&
> +	test_cmp expect actual
> +'
> +
> +test_expect_failure 'rebase --merge with a variety of empty commits' '
> +	test_when_finished "git rebase --abort" &&
> +	git checkout -B testing localmods &&
> +	# rebase --merge should not halt on the commit that becomes empty
> +	git rebase --merge upstream &&
> +
> +	test_write_lines D C B A >expect &&
> +	git log --format=%s >actual &&
> +	test_cmp expect actual
> +'
> +
> +GIT_SEQUENCE_EDITOR=: && export GIT_SEQUENCE_EDITOR

This doesn't do any harm but I'm not sure it's needed

Best Wishes

Phillip

> +test_expect_success 'rebase --interactive with a variety of empty commits' '
> +	git checkout -B testing localmods &&
> +	test_must_fail git rebase --interactive upstream &&
> +
> +	git rebase --skip &&
> +
> +	test_write_lines D C B A >expect &&
> +	git log --format=%s >actual &&
> +	test_cmp expect actual
> +'
> +
> +test_done
> diff --git a/t/t3427-rebase-subtree.sh b/t/t3427-rebase-subtree.sh
> index bec48e6a1f..8dceef61cf 100755
> --- a/t/t3427-rebase-subtree.sh
> +++ b/t/t3427-rebase-subtree.sh
> @@ -85,23 +85,23 @@ test_expect_failure REBASE_P 'Rebase -Xsubtree --keep-empty --preserve-merges --
>   	verbose test "$(commit_message HEAD)" = "Empty commit"
>   '
>   
> -test_expect_success 'Rebase -Xsubtree --keep-empty --onto commit' '
> +test_expect_success 'Rebase -Xsubtree --onto commit' '
>   	reset_rebase &&
>   	git checkout -b rebase-onto to-rebase &&
> -	test_must_fail git rebase -Xsubtree=files_subtree --keep-empty --onto files-master master &&
> +	test_must_fail git rebase -Xsubtree=files_subtree --onto files-master master &&
>   	: first pick results in no changes &&
> -	git rebase --continue &&
> +	git rebase --skip &&
>   	verbose test "$(commit_message HEAD~2)" = "master4" &&
>   	verbose test "$(commit_message HEAD~)" = "files_subtree/master5" &&
>   	verbose test "$(commit_message HEAD)" = "Empty commit"
>   '
>   
> -test_expect_success 'Rebase -Xsubtree --keep-empty --rebase-merges --onto commit' '
> +test_expect_success 'Rebase -Xsubtree --rebase-merges --onto commit' '
>   	reset_rebase &&
>   	git checkout -b rebase-merges-onto to-rebase &&
> -	test_must_fail git rebase -Xsubtree=files_subtree --keep-empty --rebase-merges --onto files-master --root &&
> +	test_must_fail git rebase -Xsubtree=files_subtree --rebase-merges --onto files-master --root &&
>   	: first pick results in no changes &&
> -	git rebase --continue &&
> +	git rebase --skip &&
>   	verbose test "$(commit_message HEAD~2)" = "master4" &&
>   	verbose test "$(commit_message HEAD~)" = "files_subtree/master5" &&
>   	verbose test "$(commit_message HEAD)" = "Empty commit"
> 

  reply index

Thread overview: 161+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-20 17:09 [PATCH 00/15] rebase: make the default backend configurable Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 01/15] rebase: extend the options for handling of empty commits Elijah Newren via GitGitGadget
2019-12-20 21:29   ` Junio C Hamano
2019-12-21  0:32     ` Elijah Newren
2019-12-21 18:52       ` Elijah Newren
2019-12-21 23:49       ` Junio C Hamano
2019-12-20 17:09 ` [PATCH 02/15] t3406: simplify an already simple test Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 03/15] rebase, sequencer: remove the broken GIT_QUIET handling Elijah Newren via GitGitGadget
2019-12-20 21:34   ` Junio C Hamano
2019-12-20 17:09 ` [PATCH 04/15] rebase: make sure to pass along the quiet flag to the sequencer Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 05/15] rebase: fix handling of restrict_revision Elijah Newren via GitGitGadget
2019-12-20 21:37   ` Junio C Hamano
2019-12-20 17:09 ` [PATCH 06/15] t3432: make these tests work with either am or merge backends Elijah Newren via GitGitGadget
2019-12-22  5:11   ` Denton Liu
2019-12-23 17:17     ` Elijah Newren
2019-12-20 17:09 ` [PATCH 07/15] rebase: allow more types of rebases to fast-forward Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 08/15] git-rebase.txt: add more details about behavioral differences of backends Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 09/15] rebase: move incompatibility checks between backend options a bit earlier Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 10/15] rebase: add an --am option Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 11/15] contrib: change the prompt for am-based rebases Elijah Newren via GitGitGadget
2019-12-20 23:07   ` SZEDER Gábor
2019-12-21  0:17     ` Elijah Newren
2019-12-20 17:09 ` [PATCH 12/15] rebase tests: mark tests specific to the am-backend with --am Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 13/15] rebase tests: repeat some tests using the merge backend instead of am Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 14/15] rebase: make the backend configurable via config setting Elijah Newren via GitGitGadget
2019-12-20 17:09 ` [PATCH 15/15] rebase: change the default backend from "am" to "merge" Elijah Newren via GitGitGadget
2019-12-20 18:51 ` [PATCH 00/15] rebase: make the default backend configurable Alban Gruin
2019-12-20 18:55   ` Elijah Newren
2019-12-23 18:49 ` [PATCH v2 " Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 01/15] rebase: extend the options for handling of empty commits Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 02/15] t3406: simplify an already simple test Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 03/15] rebase, sequencer: remove the broken GIT_QUIET handling Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 04/15] rebase: make sure to pass along the quiet flag to the sequencer Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 05/15] rebase: fix handling of restrict_revision Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 06/15] t3432: make these tests work with either am or merge backends Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 07/15] rebase: allow more types of rebases to fast-forward Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 08/15] git-rebase.txt: add more details about behavioral differences of backends Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 09/15] rebase: move incompatibility checks between backend options a bit earlier Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 10/15] rebase: add an --am option Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 11/15] contrib: change the prompt for interactive-based rebases Elijah Newren via GitGitGadget
2019-12-23 22:00     ` Denton Liu
2019-12-23 18:49   ` [PATCH v2 12/15] rebase tests: mark tests specific to the am-backend with --am Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 13/15] rebase tests: repeat some tests using the merge backend instead of am Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 14/15] rebase: make the backend configurable via config setting Elijah Newren via GitGitGadget
2019-12-23 18:49   ` [PATCH v2 15/15] rebase: change the default backend from "am" to "merge" Elijah Newren via GitGitGadget
2019-12-24 19:54   ` [PATCH v3 00/15] rebase: make the default backend configurable Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 01/15] rebase: extend the options for handling of empty commits Elijah Newren via GitGitGadget
2020-01-07 14:37       ` Phillip Wood
2020-01-07 19:15         ` Elijah Newren
2020-01-08 14:27           ` Phillip Wood
2020-01-09 21:32             ` Johannes Schindelin
2019-12-24 19:54     ` [PATCH v3 02/15] t3406: simplify an already simple test Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 03/15] rebase, sequencer: remove the broken GIT_QUIET handling Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 04/15] rebase: make sure to pass along the quiet flag to the sequencer Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 05/15] rebase: fix handling of restrict_revision Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 06/15] t3432: make these tests work with either am or merge backends Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 07/15] rebase: allow more types of rebases to fast-forward Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 08/15] git-rebase.txt: add more details about behavioral differences of backends Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 09/15] rebase: move incompatibility checks between backend options a bit earlier Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 10/15] rebase: add an --am option Elijah Newren via GitGitGadget
2020-01-07 14:43       ` Phillip Wood
2020-01-07 19:26         ` Elijah Newren
2020-01-07 20:11           ` Junio C Hamano
2020-01-08 14:32             ` Phillip Wood
2020-01-08 17:18               ` Junio C Hamano
2020-01-08 18:55                 ` Phillip Wood
2019-12-24 19:54     ` [PATCH v3 11/15] git-prompt: change the prompt for interactive-based rebases Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 12/15] rebase tests: mark tests specific to the am-backend with --am Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 13/15] rebase tests: repeat some tests using the merge backend instead of am Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 14/15] rebase: make the backend configurable via config setting Elijah Newren via GitGitGadget
2019-12-24 19:54     ` [PATCH v3 15/15] rebase: change the default backend from "am" to "merge" Elijah Newren via GitGitGadget
2020-01-10 23:14       ` Jonathan Nieder
2020-01-11  1:16         ` Elijah Newren
2020-01-11 14:41           ` Phillip Wood
2020-01-12 17:59             ` Johannes Schindelin
2020-01-16  6:32               ` Elijah Newren
2020-01-16  7:58                 ` Jonathan Nieder
2020-01-16  8:06                   ` Jonathan Nieder
2020-01-16 16:18                     ` Elijah Newren
2020-01-16 20:35                       ` Jonathan Nieder
2020-01-16 21:30                         ` Elijah Newren
2020-01-16 22:39                           ` Jonathan Nieder
2020-01-16 23:19                             ` Elijah Newren
2020-01-16 23:25                           ` Junio C Hamano
2020-01-17  0:51                             ` Elijah Newren
2020-01-16 15:35                   ` Elijah Newren
2020-01-16 20:05                   ` Junio C Hamano
2020-01-16 10:48                 ` Johannes Schindelin
2020-01-12 21:23             ` Junio C Hamano
2020-01-15 19:50             ` Jonathan Nieder
2020-01-15 21:59               ` Emily Shaffer
2020-01-16  6:14     ` [PATCH v4 00/19] rebase: make the default backend configurable Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 01/19] git-rebase.txt: update description of --allow-empty-message Elijah Newren via GitGitGadget
2020-02-09 15:59         ` Phillip Wood
2020-02-13 18:35           ` Elijah Newren
2020-01-16  6:14       ` [PATCH v4 02/19] t3404: directly test the behavior of interest Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 03/19] rebase (interactive-backend): make --keep-empty the default Elijah Newren via GitGitGadget
2020-02-09 15:59         ` Phillip Wood [this message]
2020-02-13 18:52           ` Elijah Newren
2020-01-16  6:14       ` [PATCH v4 04/19] rebase (interactive-backend): fix handling of commits that become empty Elijah Newren via GitGitGadget
2020-02-10 14:27         ` Phillip Wood
2020-02-13 18:54           ` Elijah Newren
2020-02-16 14:46             ` Phillip Wood
2020-01-16  6:14       ` [PATCH v4 05/19] t3406: simplify an already simple test Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 06/19] rebase, sequencer: remove the broken GIT_QUIET handling Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 07/19] rebase: make sure to pass along the quiet flag to the sequencer Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 08/19] rebase: fix handling of restrict_revision Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 09/19] t3432: make these tests work with either am or merge backends Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 10/19] rebase: allow more types of rebases to fast-forward Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 11/19] git-rebase.txt: add more details about behavioral differences of backends Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 12/19] rebase: move incompatibility checks between backend options a bit earlier Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 13/19] rebase: add an --am option Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 14/19] git-prompt: change the prompt for interactive-based rebases Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 15/19] rebase: drop '-i' from the reflog " Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 16/19] rebase tests: mark tests specific to the am-backend with --am Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 17/19] rebase tests: repeat some tests using the merge backend instead of am Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 18/19] rebase: make the backend configurable via config setting Elijah Newren via GitGitGadget
2020-01-16  6:14       ` [PATCH v4 19/19] rebase: change the default backend from "am" to "merge" Elijah Newren via GitGitGadget
2020-01-17 16:58       ` [PATCH v4 00/19] rebase: make the default backend configurable Phillip Wood
2020-02-05 21:06         ` Junio C Hamano
2020-02-05 22:38           ` Elijah Newren
2020-02-15 21:36       ` [PATCH v5 00/20] rebase: make the default backend configurable and change the default Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 01/20] git-rebase.txt: update description of --allow-empty-message Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 02/20] t3404: directly test the behavior of interest Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 03/20] rebase (interactive-backend): make --keep-empty the default Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 04/20] rebase (interactive-backend): fix handling of commits that become empty Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 05/20] t3406: simplify an already simple test Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 06/20] rebase, sequencer: remove the broken GIT_QUIET handling Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 07/20] rebase: make sure to pass along the quiet flag to the sequencer Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 08/20] rebase: fix handling of restrict_revision Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 09/20] t3432: make these tests work with either am or merge backends Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 10/20] rebase: allow more types of rebases to fast-forward Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 11/20] git-rebase.txt: add more details about behavioral differences of backends Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 12/20] rebase: move incompatibility checks between backend options a bit earlier Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 13/20] rebase: add an --am option Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 14/20] git-prompt: change the prompt for interactive-based rebases Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 15/20] rebase: drop '-i' from the reflog " Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 16/20] rebase tests: mark tests specific to the am-backend with --am Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 17/20] rebase tests: repeat some tests using the merge backend instead of am Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 18/20] rebase: make the backend configurable via config setting Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 19/20] rebase: change the default backend from "am" to "merge" Elijah Newren via GitGitGadget
2020-02-15 21:36         ` [PATCH v5 20/20] rebase: rename the two primary rebase backends Elijah Newren via GitGitGadget
2020-03-12 15:13           ` Emily Shaffer
2020-03-12 16:33             ` Elijah Newren
2020-03-12 17:55               ` Jonathan Nieder
2020-03-12 18:39                 ` Elijah Newren
2020-03-12 18:46                   ` Jonathan Nieder
2020-03-12 19:31                     ` Elijah Newren
2020-03-17  2:58                       ` Jonathan Nieder
2020-03-17  4:45                         ` Elijah Newren
2020-03-12 19:54                     ` Junio C Hamano
2020-03-12 19:07               ` Junio C Hamano
2020-03-12 19:12                 ` Jonathan Nieder
2020-03-12 19:12               ` Junio C Hamano
2020-03-12 19:29                 ` Elijah Newren
2020-03-12 20:37                   ` Jeff King
2020-03-12 21:27                     ` Junio C Hamano
2020-03-12 22:06                       ` Elijah Newren
2020-03-13  0:04                         ` Junio C Hamano
2020-03-12 23:30                       ` Jonathan Nieder
2020-02-16 15:01         ` [PATCH v5 00/20] rebase: make the default backend configurable and change the default Phillip Wood

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=805e3edd-d479-16b8-c67f-d8ea34176389@gmail.com \
    --to=phillip.wood123@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=alban.gruin@gmail.com \
    --cc=emilyshaffer@google.com \
    --cc=git@vger.kernel.org \
    --cc=gitgitgadget@gmail.com \
    --cc=gitster@pobox.com \
    --cc=jrnieder@gmail.com \
    --cc=liu.denton@gmail.com \
    --cc=newren@gmail.com \
    --cc=phillip.wood@dunelm.org.uk \
    --cc=plroskin@gmail.com \
    --cc=szeder.dev@gmail.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

Git Mailing List Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/git/0 git/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 git git/ https://lore.kernel.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.git


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git