Skip to content

Commit 471f67a

Browse files
committed
Merge branch 'po/rebase-preserve-merges' into next
Various error messages that talk about the removal of "--preserve-merges" in "rebase" have been strengthened, and "rebase --abort" learned to get out of a state that was left by an earlier use of the option. * po/rebase-preserve-merges: rebase: translate a die(preserve-merges) message rebase: note `preserve` merges may be a pull config option rebase: help users when dying with `preserve-merges` rebase.c: state preserve-merges has been removed
2 parents eebb0c9 + f007713 commit 471f67a

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

builtin/rebase.c

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1110,8 +1110,8 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
11101110
PARSE_OPT_NOARG | PARSE_OPT_NONEG,
11111111
parse_opt_interactive),
11121112
OPT_SET_INT_F('p', "preserve-merges", &preserve_merges_selected,
1113-
N_("(DEPRECATED) try to recreate merges instead of "
1114-
"ignoring them"),
1113+
N_("(REMOVED) was: try to recreate merges "
1114+
"instead of ignoring them"),
11151115
1, PARSE_OPT_HIDDEN),
11161116
OPT_RERERE_AUTOUPDATE(&options.allow_rerere_autoupdate),
11171117
OPT_CALLBACK_F(0, "empty", &options, "{drop,keep,ask}",
@@ -1182,8 +1182,10 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
11821182
} else if (is_directory(merge_dir())) {
11831183
strbuf_reset(&buf);
11841184
strbuf_addf(&buf, "%s/rewritten", merge_dir());
1185-
if (is_directory(buf.buf)) {
1186-
die("`rebase -p` is no longer supported");
1185+
if (!(action == ACTION_ABORT) && is_directory(buf.buf)) {
1186+
die(_("`rebase --preserve-merges` (-p) is no longer supported.\n"
1187+
"Use `git rebase --abort` to terminate current rebase.\n"
1188+
"Or downgrade to v2.33, or earlier, to complete the rebase."));
11871189
} else {
11881190
strbuf_reset(&buf);
11891191
strbuf_addf(&buf, "%s/interactive", merge_dir());
@@ -1203,7 +1205,9 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
12031205
builtin_rebase_usage, 0);
12041206

12051207
if (preserve_merges_selected)
1206-
die(_("--preserve-merges was replaced by --rebase-merges"));
1208+
die(_("--preserve-merges was replaced by --rebase-merges\n"
1209+
"Note: Your `pull.rebase` configuration may also be set to 'preserve',\n"
1210+
"which is no longer supported; use 'merges' instead"));
12071211

12081212
if (action != ACTION_NONE && total_argc != 2) {
12091213
usage_with_options(builtin_rebase_usage,

0 commit comments

Comments
 (0)