* [PATCH] checkout: add --autostash option for branch switching
@ 2026-03-12 13:26 Harald Nordgren via GitGitGadget
2026-03-12 14:40 ` Junio C Hamano
2026-03-12 19:33 ` [PATCH v2] " Harald Nordgren via GitGitGadget
0 siblings, 2 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-12 13:26 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches, local modifications in the working tree can
prevent the checkout from succeeding. While "git rebase" and "git
merge" already support --autostash to handle this case automatically,
"git checkout" and "git switch" require users to manually stash and
unstash their changes.
Teach "git checkout" and "git switch" to accept --autostash and
--no-autostash options that automatically create a temporary stash
entry before the branch switch begins and apply it after the switch
completes. If the stash application results in conflicts, the stash
entry is saved to the stash list so the user can resolve them later.
Also add a checkout.autoStash configuration option that enables this
behavior by default, which can be overridden with --no-autostash on
the command line.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
checkout: 'autostash' for branch switching
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v1
Pull-Request: https://github.com/git/git/pull/2234
Documentation/config/checkout.adoc | 12 ++
Documentation/git-checkout.adoc | 9 ++
Documentation/git-switch.adoc | 9 ++
builtin/checkout.c | 16 +++
t/meson.build | 1 +
t/t2061-switch-autostash.sh | 181 +++++++++++++++++++++++++++++
t/t9902-completion.sh | 1 +
7 files changed, 229 insertions(+)
create mode 100755 t/t2061-switch-autostash.sh
diff --git a/Documentation/config/checkout.adoc b/Documentation/config/checkout.adoc
index e35d212969..2e157c5398 100644
--- a/Documentation/config/checkout.adoc
+++ b/Documentation/config/checkout.adoc
@@ -36,6 +36,18 @@ with a small number of cores, the default sequential checkout often performs
better. The size and compression level of a repository might also influence how
well the parallel version performs.
+`checkout.autoStash`::
+ When set to true, automatically create a temporary stash entry
+ before the operation begins, and apply it after the operation
+ ends. This means that you can run `git checkout` or `git switch`
+ on a dirty worktree. However, use with care: the final stash
+ application after a successful branch switch might result in
+ non-trivial conflicts.
+ This option can be overridden by the `--no-autostash` and
+ `--autostash` options of linkgit:git-checkout[1] and
+ linkgit:git-switch[1].
+ Defaults to false.
+
`checkout.thresholdForParallelism`::
When running parallel checkout with a small number of files, the cost
of subprocess spawning and inter-process communication might outweigh
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..96d9bf9203 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -272,6 +272,15 @@ When switching branches with `--merge`, staged changes may be lost.
`merge.conflictStyle` configuration variable. Possible values are
`merge` (default), `diff3`, and `zdiff3`.
+`--autostash`::
+`--no-autostash`::
+ When switching branches, automatically create a temporary stash
+ entry before the operation begins, and apply it after the
+ operation ends. This means that you can switch branches on a
+ dirty worktree. However, use with care: the final stash
+ application after a successful branch switch might result in
+ non-trivial conflicts.
+
`-p`::
`--patch`::
Interactively select hunks in the difference between the
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..b296df2a0b 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -142,6 +142,15 @@ should result in deletion of the path).
`merge.conflictStyle` configuration variable. Possible values are
`merge` (default), `diff3`, and `zdiff3`.
+`--autostash`::
+`--no-autostash`::
+ Automatically create a temporary stash entry before the
+ operation begins, and apply it after the operation ends.
+ This means that you can switch branches on a dirty worktree.
+ However, use with care: the final stash application after a
+ successful branch switch might result in non-trivial
+ conflicts.
+
`-q`::
`--quiet`::
Quiet, suppress feedback messages.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..453dbe3230 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -30,6 +30,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -68,6 +69,7 @@ struct checkout_opts {
int only_merge_on_switching_branches;
int can_switch_when_in_progress;
int orphan_from_empty_tree;
+ int autostash;
int empty_pathspec_ok;
int checkout_index;
int checkout_worktree;
@@ -1202,9 +1204,16 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (opts->autostash) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ create_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
branch_info_release(&old_branch_info);
return ret;
}
@@ -1215,6 +1224,8 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
@@ -1236,6 +1247,10 @@ static int git_checkout_config(const char *var, const char *value,
opts->dwim_new_local_branch = git_config_bool(var, value);
return 0;
}
+ if (!strcmp(var, "checkout.autostash")) {
+ opts->autostash = git_config_bool(var, value);
+ return 0;
+ }
if (starts_with(var, "submodule."))
return git_default_submodule_config(var, value, NULL);
@@ -1745,6 +1760,7 @@ static struct option *add_common_switch_branch_options(
PARSE_OPT_NOCOMPLETE),
OPT_BOOL(0, "ignore-other-worktrees", &opts->ignore_other_worktrees,
N_("do not check if another worktree is using this branch")),
+ OPT_AUTOSTASH(&opts->autostash),
OPT_END()
};
struct option *newopts = parse_options_concat(prevopts, options);
diff --git a/t/meson.build b/t/meson.build
index f66a73f8a0..0645253d25 100644
--- a/t/meson.build
+++ b/t/meson.build
@@ -275,6 +275,7 @@ integration_tests = [
't2030-unresolve-info.sh',
't2050-git-dir-relative.sh',
't2060-switch.sh',
+ 't2061-switch-autostash.sh',
't2070-restore.sh',
't2071-restore-patch.sh',
't2072-restore-pathspec-file.sh',
diff --git a/t/t2061-switch-autostash.sh b/t/t2061-switch-autostash.sh
new file mode 100755
index 0000000000..6409a2afbf
--- /dev/null
+++ b/t/t2061-switch-autostash.sh
@@ -0,0 +1,181 @@
+#!/bin/sh
+
+test_description='checkout/switch --autostash tests'
+
+GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
+export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+
+. ./test-lib.sh
+
+test_expect_success 'setup' '
+ echo file0content >file0 &&
+ echo file1content >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "initial commit" &&
+ git branch other-branch &&
+ echo file1main >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "modify file1 on main" &&
+ git checkout other-branch &&
+ echo file1other >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "modify file1 on other-branch" &&
+ echo file2content >file2 &&
+ git add . &&
+ test_tick &&
+ git commit -m "add file2 on other-branch" &&
+ git checkout main
+'
+
+test_expect_success 'switch --autostash on dirty worktree' '
+ git branch branch1 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash branch1 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'checkout --autostash on dirty worktree' '
+ git branch branch2 other-branch &&
+ echo dirty >file0 &&
+ git checkout --autostash branch2 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
+'
+
+test_expect_success 'switch: checkout.autostash config' '
+ git branch branch3 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git switch branch3 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'checkout: checkout.autostash config' '
+ git branch branch4 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git checkout branch4 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
+'
+
+test_expect_success '--no-autostash overrides checkout.autostash' '
+ git branch branch5 other-branch &&
+ echo dirty >file1 &&
+ test_config checkout.autostash true &&
+ test_must_fail git switch --no-autostash branch5 2>stderr &&
+ test_grep ! "Created autostash" stderr &&
+ git checkout -- file1
+'
+
+test_expect_success '--autostash overrides checkout.autostash=false' '
+ git branch branch6 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash false &&
+ git switch --autostash branch6 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash with dirty index' '
+ git branch branch7 other-branch &&
+ echo dirty-index >file0 &&
+ git add file0 &&
+ git switch --autostash branch7 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty-index >expected &&
+ test_cmp expected file0 &&
+ git checkout -- file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash bypasses conflicting local changes' '
+ git branch branch8 other-branch &&
+ echo dirty >file1 &&
+ test_must_fail git switch branch8 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git switch --autostash branch8 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success 'autostash is a no-op with clean worktree' '
+ git branch branch9 other-branch &&
+ git switch --autostash branch9 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ git switch main
+'
+
+test_expect_success '--autostash with --merge stashes and switches' '
+ git branch branch10 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash --merge branch10 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash with staged conflicting changes' '
+ git branch branch11 other-branch &&
+ echo staged-change >file1 &&
+ git add file1 &&
+ git switch --autostash branch11 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success '--autostash with --force preserves dirty changes' '
+ git branch branch12 other-branch &&
+ echo dirty-force >file1 &&
+ git switch --autostash --force branch12 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success '--autostash with new branch creation' '
+ echo dirty >file0 &&
+ git switch --autostash -c branch13 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main &&
+ git branch -D branch13
+'
+
+test_done
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index 2f9a597ec7..f33ca543a9 100755
--- a/t/t9902-completion.sh
+++ b/t/t9902-completion.sh
@@ -2602,6 +2602,7 @@ test_expect_success 'double dash "git checkout"' '
--ignore-other-worktrees Z
--recurse-submodules Z
--auto-advance Z
+ --autostash Z
--progress Z
--guess Z
--no-guess Z
base-commit: 7f19e4e1b6a3ad259e2ed66033e01e03b8b74c5e
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-12 13:26 [PATCH] checkout: add --autostash option for branch switching Harald Nordgren via GitGitGadget
@ 2026-03-12 14:40 ` Junio C Hamano
2026-03-12 19:33 ` [PATCH v31 0/2] status: add status.compareBranches config for multiple branch comparisons Harald Nordgren
2026-03-13 14:29 ` [PATCH] checkout: add --autostash option for branch switching Phillip Wood
2026-03-12 19:33 ` [PATCH v2] " Harald Nordgren via GitGitGadget
1 sibling, 2 replies; 29+ messages in thread
From: Junio C Hamano @ 2026-03-12 14:40 UTC (permalink / raw)
To: Harald Nordgren via GitGitGadget; +Cc: git, Harald Nordgren
"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Harald Nordgren <haraldnordgren@gmail.com>
>
> When switching branches, local modifications in the working tree can
> prevent the checkout from succeeding. While "git rebase" and "git
> merge" already support --autostash to handle this case automatically,
> "git checkout" and "git switch" require users to manually stash and
> unstash their changes.
>
> Teach "git checkout" and "git switch" to accept --autostash and
> --no-autostash options that automatically create a temporary stash
> entry before the branch switch begins and apply it after the switch
> completes. If the stash application results in conflicts, the stash
> entry is saved to the stash list so the user can resolve them later.
>
> Also add a checkout.autoStash configuration option that enables this
> behavior by default, which can be overridden with --no-autostash on
> the command line.
Unconditionally always stash when checkout happens? This feature as
implemented does not have to be a separate feature. It can be done
by end-users as a short-cut for "stash" followed by "checkout" via
alias or custom command.
Perhaps doing it this way would make it more worth doing?
- At the beginning of branch switching, ask a new helper function
that takes the branch we are switching to as an argument this
question:
Do any paths that are different between the current branch and
the branch we are switching to have local (i.e., either in the
index or in the working tree) change [Yes/No]?
- When the answer is "yes", save the local changes to a new stash
entry, and clear the local changes from the index and from the
working tree. If not, do not bother with stash at all.
- Switch to other branch the usual way. This will never conflict.
- If we created a stash entry earlier, try to unstash it. It may
conflict or it may not.
- If it does not conflict, then we are done. We drop that stash
entry, and tell nothing about the stash to the user, as there
is nothing they can do to the now-consumed stash.
- If it does conflict, tell the user that the original change is
in the stash, and can be used to recover if you botch the
conflict resolution, and also tell the user that they need to
drop the stash entry once they are done with the change that
caused this current conflict.
Essentially, the new "autostaash only when needed" would become a
much better reimplementation of the "-m" option. From the point of
view of a user who is used to "checkout -m", the basic workflow
would be the same, only with vast improvement.
- It may not conflict and merge cleanly, in which case they do not
have to do anything. This is the same as status quo.
- It may conflict and they find it too involved to resolve right at
the moment, in which case they now have a choiceto say "git reset
--hard", essentially declaring "I prioritize working on this new
branch; I'll deal with the work in progress I started on the
previous branch later", and then later they can "git stash pop"
to deal with it.
Which is a vast improvement over the current "-m" that gives you
only one chance to resolve it right.
- It may conflict and they may be able to resolve cleanly, in which
case they have to remember that they need to do an extra thing
(i.e., drop the stash we created just in case) but that may not
be too bad a tradeoff.
If we can sell it as an improved implementation of "-m", we probably
can lose some code that the current "-m" implementation uses to do
its merge; we'd be instead using the "unstash" code paths.
And the new helper function to detect if switching from one commit
to another commit would never conflict can later be used to enhance
"git rebase" as well---we could call it N times to rebase a branch
with N commits and if all steps are clear, we do not have to insist
that there is no local changes like we do currently.
Hmm?
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v31 0/2] status: add status.compareBranches config for multiple branch comparisons
2026-03-12 14:40 ` Junio C Hamano
@ 2026-03-12 19:33 ` Harald Nordgren
2026-03-13 14:29 ` [PATCH] checkout: add --autostash option for branch switching Phillip Wood
1 sibling, 0 replies; 29+ messages in thread
From: Harald Nordgren @ 2026-03-12 19:33 UTC (permalink / raw)
To: gitster; +Cc: git, gitgitgadget, haraldnordgren
Good point! I have been running this today, and it gets a bit annoying when
it's stashing and unstashing needlessly.
I updated the code!
Harald
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v2] checkout: add --autostash option for branch switching
2026-03-12 13:26 [PATCH] checkout: add --autostash option for branch switching Harald Nordgren via GitGitGadget
2026-03-12 14:40 ` Junio C Hamano
@ 2026-03-12 19:33 ` Harald Nordgren via GitGitGadget
2026-03-12 19:50 ` Junio C Hamano
2026-03-13 9:23 ` [PATCH v3] " Harald Nordgren via GitGitGadget
1 sibling, 2 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-12 19:33 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches, local modifications in the working tree can
prevent the checkout from succeeding. While "git rebase" and "git
merge" already support --autostash to handle this case automatically,
"git checkout" and "git switch" require users to manually stash and
unstash their changes.
Teach "git checkout" and "git switch" to accept --autostash and
--no-autostash options that automatically create a temporary stash
entry before the branch switch begins and apply it after the switch
completes. If the stash application results in conflicts, the stash
entry is saved to the stash list so the user can resolve them later.
Also add a checkout.autoStash configuration option that enables this
behavior by default, which can be overridden with --no-autostash on
the command line.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
checkout: 'autostash' for branch switching
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v2
Pull-Request: https://github.com/git/git/pull/2234
Range-diff vs v1:
1: 12194f6fbe ! 1: be2b697c54 checkout: add --autostash option for branch switching
@@ Documentation/config/checkout.adoc: with a small number of cores, the default se
well the parallel version performs.
+`checkout.autoStash`::
-+ When set to true, automatically create a temporary stash entry
-+ before the operation begins, and apply it after the operation
-+ ends. This means that you can run `git checkout` or `git switch`
-+ on a dirty worktree. However, use with care: the final stash
-+ application after a successful branch switch might result in
-+ non-trivial conflicts.
++ When set to true, enable autostash for `git checkout` and
++ `git switch` branch switching operations. When a branch
++ switch would fail because local changes (in the index or the
++ working tree) overlap with paths that differ between the
++ current and target branch, a temporary stash entry is
++ automatically created before the switch and applied after
++ it completes. If the local changes do not overlap with the
++ branch difference, the switch proceeds normally without
++ stashing.
+ This option can be overridden by the `--no-autostash` and
+ `--autostash` options of linkgit:git-checkout[1] and
+ linkgit:git-switch[1].
@@ Documentation/git-checkout.adoc: When switching branches with `--merge`, staged
+`--autostash`::
+`--no-autostash`::
-+ When switching branches, automatically create a temporary stash
-+ entry before the operation begins, and apply it after the
-+ operation ends. This means that you can switch branches on a
-+ dirty worktree. However, use with care: the final stash
-+ application after a successful branch switch might result in
-+ non-trivial conflicts.
++ When switching branches, if any of the paths that differ
++ between the current branch and the target branch have local
++ changes (in the index or the working tree), automatically
++ create a temporary stash entry before the operation begins,
++ and apply it after the operation ends. If the local changes
++ do not overlap with the branch difference, the switch proceeds
++ without stashing. When a stash entry is created and the
++ subsequent application results in conflicts, the stash entry
++ is saved so that you can use `git stash pop` to recover and
++ `git stash drop` when done. Use with `--force` to always
++ stash local changes regardless of conflicts.
+
`-p`::
`--patch`::
@@ Documentation/git-switch.adoc: should result in deletion of the path).
+`--autostash`::
+`--no-autostash`::
-+ Automatically create a temporary stash entry before the
-+ operation begins, and apply it after the operation ends.
-+ This means that you can switch branches on a dirty worktree.
-+ However, use with care: the final stash application after a
-+ successful branch switch might result in non-trivial
-+ conflicts.
++ When switching branches, if any of the paths that differ
++ between the current branch and the target branch have local
++ changes (in the index or the working tree), automatically
++ create a temporary stash entry before the operation begins,
++ and apply it after the operation ends. If the local changes
++ do not overlap with the branch difference, the switch proceeds
++ without stashing. When a stash entry is created and the
++ subsequent application results in conflicts, the stash entry
++ is saved so that you can use `git stash pop` to recover and
++ `git stash drop` when done. Use with `--force` to always
++ stash local changes regardless of conflicts.
+
`-q`::
`--quiet`::
@@ builtin/checkout.c: struct checkout_opts {
int empty_pathspec_ok;
int checkout_index;
int checkout_worktree;
+@@ builtin/checkout.c: static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
+ release_revisions(&revs);
+ }
+
++static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
++ struct branch_info *new_branch_info)
++{
++ struct tree_desc trees[2];
++ struct tree *old_tree, *new_tree;
++ struct unpack_trees_options topts;
++ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
++ const struct object_id *old_commit_oid;
++ int ret;
++
++ if (!new_branch_info->commit)
++ return 0;
++
++ old_commit_oid = old_branch_info->commit ?
++ &old_branch_info->commit->object.oid :
++ the_hash_algo->empty_tree;
++ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
++ if (!old_tree)
++ return 0;
++
++ new_tree = repo_get_commit_tree(the_repository,
++ new_branch_info->commit);
++ if (!new_tree)
++ return 0;
++ if (repo_parse_tree(the_repository, new_tree) < 0)
++ return 0;
++
++ memset(&topts, 0, sizeof(topts));
++ topts.head_idx = -1;
++ topts.src_index = the_repository->index;
++ topts.dst_index = &tmp_index;
++ topts.initial_checkout = is_index_unborn(the_repository->index);
++ topts.merge = 1;
++ topts.update = 1;
++ topts.dry_run = 1;
++ topts.quiet = 1;
++ topts.fn = twoway_merge;
++
++ init_tree_desc(&trees[0], &old_tree->object.oid,
++ old_tree->buffer, old_tree->size);
++ init_tree_desc(&trees[1], &new_tree->object.oid,
++ new_tree->buffer, new_tree->size);
++
++ ret = unpack_trees(2, trees, &topts);
++ discard_index(&tmp_index);
++
++ return ret != 0;
++}
++
+ static int switch_branches(const struct checkout_opts *opts,
+ struct branch_info *new_branch_info)
+ {
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ if (opts->autostash) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
-+ create_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
++ if (opts->discard_changes ||
++ checkout_would_clobber_changes(&old_branch_info,
++ new_branch_info))
++ create_autostash_ref(the_repository,
++ "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
@@ t/t2061-switch-autostash.sh (new)
+ git checkout main
+'
+
-+test_expect_success 'switch --autostash on dirty worktree' '
++test_expect_success 'switch --autostash skips stash when no conflict' '
+ git branch branch1 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash branch1 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
-+test_expect_success 'checkout --autostash on dirty worktree' '
++test_expect_success 'checkout --autostash skips stash when no conflict' '
+ git branch branch2 other-branch &&
+ echo dirty >file0 &&
+ git checkout --autostash branch2 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
+'
+
-+test_expect_success 'switch: checkout.autostash config' '
++test_expect_success 'switch: checkout.autostash config skips stash when no conflict' '
+ git branch branch3 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git switch branch3 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
-+test_expect_success 'checkout: checkout.autostash config' '
++test_expect_success 'checkout: checkout.autostash config skips stash when no conflict' '
+ git branch branch4 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git checkout branch4 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
@@ t/t2061-switch-autostash.sh (new)
+ echo dirty >file0 &&
+ test_config checkout.autostash false &&
+ git switch --autostash branch6 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
-+test_expect_success 'autostash with dirty index' '
++test_expect_success 'autostash with non-conflicting dirty index' '
+ git branch branch7 other-branch &&
+ echo dirty-index >file0 &&
+ git add file0 &&
+ git switch --autostash branch7 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty-index >expected &&
+ test_cmp expected file0 &&
+ git checkout -- file0 &&
@@ t/t2061-switch-autostash.sh (new)
+ git switch main
+'
+
-+test_expect_success '--autostash with --merge stashes and switches' '
++test_expect_success '--autostash with --merge skips stash when no conflict' '
+ git branch branch10 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash --merge branch10 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
@@ t/t2061-switch-autostash.sh (new)
+ git switch main
+'
+
-+test_expect_success '--autostash with new branch creation' '
++test_expect_success '--autostash with new branch creation skips stash' '
+ echo dirty >file0 &&
+ git switch --autostash -c branch13 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main &&
+ git branch -D branch13
+'
+
++test_expect_success 'autostash with conflicting changes that apply cleanly' '
++ git branch branch14 other-branch &&
++ echo file1other >file1 &&
++ git switch --autostash branch14 >actual 2>&1 &&
++ test_grep "Created autostash" actual &&
++ test_grep "Applied autostash" actual &&
++ echo file1other >expected &&
++ test_cmp expected file1 &&
++ git switch main
++'
++
+test_done
## t/t9902-completion.sh ##
Documentation/config/checkout.adoc | 15 +++
Documentation/git-checkout.adoc | 14 +++
Documentation/git-switch.adoc | 14 +++
builtin/checkout.c | 69 +++++++++++
t/meson.build | 1 +
t/t2061-switch-autostash.sh | 184 +++++++++++++++++++++++++++++
t/t9902-completion.sh | 1 +
7 files changed, 298 insertions(+)
create mode 100755 t/t2061-switch-autostash.sh
diff --git a/Documentation/config/checkout.adoc b/Documentation/config/checkout.adoc
index e35d212969..268b7b1b8b 100644
--- a/Documentation/config/checkout.adoc
+++ b/Documentation/config/checkout.adoc
@@ -36,6 +36,21 @@ with a small number of cores, the default sequential checkout often performs
better. The size and compression level of a repository might also influence how
well the parallel version performs.
+`checkout.autoStash`::
+ When set to true, enable autostash for `git checkout` and
+ `git switch` branch switching operations. When a branch
+ switch would fail because local changes (in the index or the
+ working tree) overlap with paths that differ between the
+ current and target branch, a temporary stash entry is
+ automatically created before the switch and applied after
+ it completes. If the local changes do not overlap with the
+ branch difference, the switch proceeds normally without
+ stashing.
+ This option can be overridden by the `--no-autostash` and
+ `--autostash` options of linkgit:git-checkout[1] and
+ linkgit:git-switch[1].
+ Defaults to false.
+
`checkout.thresholdForParallelism`::
When running parallel checkout with a small number of files, the cost
of subprocess spawning and inter-process communication might outweigh
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..19da63c5be 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -272,6 +272,20 @@ When switching branches with `--merge`, staged changes may be lost.
`merge.conflictStyle` configuration variable. Possible values are
`merge` (default), `diff3`, and `zdiff3`.
+`--autostash`::
+`--no-autostash`::
+ When switching branches, if any of the paths that differ
+ between the current branch and the target branch have local
+ changes (in the index or the working tree), automatically
+ create a temporary stash entry before the operation begins,
+ and apply it after the operation ends. If the local changes
+ do not overlap with the branch difference, the switch proceeds
+ without stashing. When a stash entry is created and the
+ subsequent application results in conflicts, the stash entry
+ is saved so that you can use `git stash pop` to recover and
+ `git stash drop` when done. Use with `--force` to always
+ stash local changes regardless of conflicts.
+
`-p`::
`--patch`::
Interactively select hunks in the difference between the
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..6028479fef 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -142,6 +142,20 @@ should result in deletion of the path).
`merge.conflictStyle` configuration variable. Possible values are
`merge` (default), `diff3`, and `zdiff3`.
+`--autostash`::
+`--no-autostash`::
+ When switching branches, if any of the paths that differ
+ between the current branch and the target branch have local
+ changes (in the index or the working tree), automatically
+ create a temporary stash entry before the operation begins,
+ and apply it after the operation ends. If the local changes
+ do not overlap with the branch difference, the switch proceeds
+ without stashing. When a stash entry is created and the
+ subsequent application results in conflicts, the stash entry
+ is saved so that you can use `git stash pop` to recover and
+ `git stash drop` when done. Use with `--force` to always
+ stash local changes regardless of conflicts.
+
`-q`::
`--quiet`::
Quiet, suppress feedback messages.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..4a251a8732 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -30,6 +30,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -68,6 +69,7 @@ struct checkout_opts {
int only_merge_on_switching_branches;
int can_switch_when_in_progress;
int orphan_from_empty_tree;
+ int autostash;
int empty_pathspec_ok;
int checkout_index;
int checkout_worktree;
@@ -1157,6 +1159,55 @@ static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
+static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
+ struct branch_info *new_branch_info)
+{
+ struct tree_desc trees[2];
+ struct tree *old_tree, *new_tree;
+ struct unpack_trees_options topts;
+ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
+ const struct object_id *old_commit_oid;
+ int ret;
+
+ if (!new_branch_info->commit)
+ return 0;
+
+ old_commit_oid = old_branch_info->commit ?
+ &old_branch_info->commit->object.oid :
+ the_hash_algo->empty_tree;
+ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
+ if (!old_tree)
+ return 0;
+
+ new_tree = repo_get_commit_tree(the_repository,
+ new_branch_info->commit);
+ if (!new_tree)
+ return 0;
+ if (repo_parse_tree(the_repository, new_tree) < 0)
+ return 0;
+
+ memset(&topts, 0, sizeof(topts));
+ topts.head_idx = -1;
+ topts.src_index = the_repository->index;
+ topts.dst_index = &tmp_index;
+ topts.initial_checkout = is_index_unborn(the_repository->index);
+ topts.merge = 1;
+ topts.update = 1;
+ topts.dry_run = 1;
+ topts.quiet = 1;
+ topts.fn = twoway_merge;
+
+ init_tree_desc(&trees[0], &old_tree->object.oid,
+ old_tree->buffer, old_tree->size);
+ init_tree_desc(&trees[1], &new_tree->object.oid,
+ new_tree->buffer, new_tree->size);
+
+ ret = unpack_trees(2, trees, &topts);
+ discard_index(&tmp_index);
+
+ return ret != 0;
+}
+
static int switch_branches(const struct checkout_opts *opts,
struct branch_info *new_branch_info)
{
@@ -1202,9 +1253,20 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (opts->autostash) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (opts->discard_changes ||
+ checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info))
+ create_autostash_ref(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
branch_info_release(&old_branch_info);
return ret;
}
@@ -1215,6 +1277,8 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
@@ -1236,6 +1300,10 @@ static int git_checkout_config(const char *var, const char *value,
opts->dwim_new_local_branch = git_config_bool(var, value);
return 0;
}
+ if (!strcmp(var, "checkout.autostash")) {
+ opts->autostash = git_config_bool(var, value);
+ return 0;
+ }
if (starts_with(var, "submodule."))
return git_default_submodule_config(var, value, NULL);
@@ -1745,6 +1813,7 @@ static struct option *add_common_switch_branch_options(
PARSE_OPT_NOCOMPLETE),
OPT_BOOL(0, "ignore-other-worktrees", &opts->ignore_other_worktrees,
N_("do not check if another worktree is using this branch")),
+ OPT_AUTOSTASH(&opts->autostash),
OPT_END()
};
struct option *newopts = parse_options_concat(prevopts, options);
diff --git a/t/meson.build b/t/meson.build
index f66a73f8a0..0645253d25 100644
--- a/t/meson.build
+++ b/t/meson.build
@@ -275,6 +275,7 @@ integration_tests = [
't2030-unresolve-info.sh',
't2050-git-dir-relative.sh',
't2060-switch.sh',
+ 't2061-switch-autostash.sh',
't2070-restore.sh',
't2071-restore-patch.sh',
't2072-restore-pathspec-file.sh',
diff --git a/t/t2061-switch-autostash.sh b/t/t2061-switch-autostash.sh
new file mode 100755
index 0000000000..d37d6f4b8b
--- /dev/null
+++ b/t/t2061-switch-autostash.sh
@@ -0,0 +1,184 @@
+#!/bin/sh
+
+test_description='checkout/switch --autostash tests'
+
+GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
+export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+
+. ./test-lib.sh
+
+test_expect_success 'setup' '
+ echo file0content >file0 &&
+ echo file1content >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "initial commit" &&
+ git branch other-branch &&
+ echo file1main >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "modify file1 on main" &&
+ git checkout other-branch &&
+ echo file1other >file1 &&
+ git add . &&
+ test_tick &&
+ git commit -m "modify file1 on other-branch" &&
+ echo file2content >file2 &&
+ git add . &&
+ test_tick &&
+ git commit -m "add file2 on other-branch" &&
+ git checkout main
+'
+
+test_expect_success 'switch --autostash skips stash when no conflict' '
+ git branch branch1 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash branch1 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'checkout --autostash skips stash when no conflict' '
+ git branch branch2 other-branch &&
+ echo dirty >file0 &&
+ git checkout --autostash branch2 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
+'
+
+test_expect_success 'switch: checkout.autostash config skips stash when no conflict' '
+ git branch branch3 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git switch branch3 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'checkout: checkout.autostash config skips stash when no conflict' '
+ git branch branch4 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash true &&
+ git checkout branch4 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git checkout main
+'
+
+test_expect_success '--no-autostash overrides checkout.autostash' '
+ git branch branch5 other-branch &&
+ echo dirty >file1 &&
+ test_config checkout.autostash true &&
+ test_must_fail git switch --no-autostash branch5 2>stderr &&
+ test_grep ! "Created autostash" stderr &&
+ git checkout -- file1
+'
+
+test_expect_success '--autostash overrides checkout.autostash=false' '
+ git branch branch6 other-branch &&
+ echo dirty >file0 &&
+ test_config checkout.autostash false &&
+ git switch --autostash branch6 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash with non-conflicting dirty index' '
+ git branch branch7 other-branch &&
+ echo dirty-index >file0 &&
+ git add file0 &&
+ git switch --autostash branch7 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty-index >expected &&
+ test_cmp expected file0 &&
+ git checkout -- file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash bypasses conflicting local changes' '
+ git branch branch8 other-branch &&
+ echo dirty >file1 &&
+ test_must_fail git switch branch8 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git switch --autostash branch8 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success 'autostash is a no-op with clean worktree' '
+ git branch branch9 other-branch &&
+ git switch --autostash branch9 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ git switch main
+'
+
+test_expect_success '--autostash with --merge skips stash when no conflict' '
+ git branch branch10 other-branch &&
+ echo dirty >file0 &&
+ git switch --autostash --merge branch10 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main
+'
+
+test_expect_success 'autostash with staged conflicting changes' '
+ git branch branch11 other-branch &&
+ echo staged-change >file1 &&
+ git add file1 &&
+ git switch --autostash branch11 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success '--autostash with --force preserves dirty changes' '
+ git branch branch12 other-branch &&
+ echo dirty-force >file1 &&
+ git switch --autostash --force branch12 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard &&
+ git switch main
+'
+
+test_expect_success '--autostash with new branch creation skips stash' '
+ echo dirty >file0 &&
+ git switch --autostash -c branch13 >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ echo dirty >expected &&
+ test_cmp expected file0 &&
+ git switch main &&
+ git branch -D branch13
+'
+
+test_expect_success 'autostash with conflicting changes that apply cleanly' '
+ git branch branch14 other-branch &&
+ echo file1other >file1 &&
+ git switch --autostash branch14 >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ echo file1other >expected &&
+ test_cmp expected file1 &&
+ git switch main
+'
+
+test_done
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index 2f9a597ec7..f33ca543a9 100755
--- a/t/t9902-completion.sh
+++ b/t/t9902-completion.sh
@@ -2602,6 +2602,7 @@ test_expect_success 'double dash "git checkout"' '
--ignore-other-worktrees Z
--recurse-submodules Z
--auto-advance Z
+ --autostash Z
--progress Z
--guess Z
--no-guess Z
base-commit: 7f19e4e1b6a3ad259e2ed66033e01e03b8b74c5e
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH v2] checkout: add --autostash option for branch switching
2026-03-12 19:33 ` [PATCH v2] " Harald Nordgren via GitGitGadget
@ 2026-03-12 19:50 ` Junio C Hamano
2026-03-13 9:22 ` [PATCH] " Harald Nordgren
2026-03-13 9:23 ` [PATCH v3] " Harald Nordgren via GitGitGadget
1 sibling, 1 reply; 29+ messages in thread
From: Junio C Hamano @ 2026-03-12 19:50 UTC (permalink / raw)
To: Harald Nordgren via GitGitGadget; +Cc: git, Harald Nordgren
"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Harald Nordgren <haraldnordgren@gmail.com>
>
> When switching branches, local modifications in the working tree can
> prevent the checkout from succeeding. While "git rebase" and "git
> merge" already support --autostash to handle this case automatically,
> "git checkout" and "git switch" require users to manually stash and
> unstash their changes.
>
> Teach "git checkout" and "git switch" to accept --autostash and
> --no-autostash options that automatically create a temporary stash
> entry before the branch switch begins and apply it after the switch
> completes. If the stash application results in conflicts, the stash
> entry is saved to the stash list so the user can resolve them later.
>
> Also add a checkout.autoStash configuration option that enables this
> behavior by default, which can be overridden with --no-autostash on
> the command line.
With this, shouldn't "-m" become a synonym for "--autostash"?
For users of "checkout -m", this is a strictly improved version of
the same feature, it seems.
Also, "stash" is merely an implementation detail of how we make the
merge safer, so from end-user's point of view, this feature is more
like "switch to the other branch, while merging the local changes
there", so calling it "--merge" or something may be much better than
calling it "--autostash".
Other than that, I like the implementation in general.
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-12 19:50 ` Junio C Hamano
@ 2026-03-13 9:22 ` Harald Nordgren
0 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren @ 2026-03-13 9:22 UTC (permalink / raw)
To: gitster; +Cc: git, gitgitgadget, haraldnordgren
> With this, shouldn't "-m" become a synonym for "--autostash"?
> For users of "checkout -m", this is a strictly improved version of
> the same feature, it seems.
>
> Also, "stash" is merely an implementation detail of how we make the
> merge safer, so from end-user's point of view, this feature is more
> like "switch to the other branch, while merging the local changes
> there", so calling it "--merge" or something may be much better than
> calling it "--autostash".
That's an interesting idea and I gave it a shot! I do worry about breaking
the existing '-m' flow because I don't understand its fundamentals.
But tests old seem to be passing (well, they did break and then I fixed the
code to make them pass again), so that's promising. Although hard for me to
say if the old test coverage protects against all possible regressions.
> Other than that, I like the implementation in general.
Thanks for all your support, Junio!
Harald
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v3] checkout: add --autostash option for branch switching
2026-03-12 19:33 ` [PATCH v2] " Harald Nordgren via GitGitGadget
2026-03-12 19:50 ` Junio C Hamano
@ 2026-03-13 9:23 ` Harald Nordgren via GitGitGadget
2026-03-13 17:16 ` Junio C Hamano
2026-03-14 9:59 ` [PATCH v4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
1 sibling, 2 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-13 9:23 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches, local modifications in the working tree can
prevent the checkout from succeeding. While "git rebase" and "git
merge" already support --autostash to handle this case automatically,
"git checkout" and "git switch" require users to manually stash and
unstash their changes.
Teach "git checkout" and "git switch" to accept --autostash and
--no-autostash options that automatically create a temporary stash
entry before the branch switch begins and apply it after the switch
completes. If the stash application results in conflicts, the stash
entry is saved to the stash list so the user can resolve them later.
Also add a checkout.autoStash configuration option that enables this
behavior by default, which can be overridden with --no-autostash on
the command line.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
checkout: 'autostash' for branch switching
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v3
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v3
Pull-Request: https://github.com/git/git/pull/2234
Range-diff vs v2:
1: be2b697c54 ! 1: 05f1e53163 checkout: add --autostash option for branch switching
@@ Commit message
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
- ## Documentation/config/checkout.adoc ##
-@@ Documentation/config/checkout.adoc: with a small number of cores, the default sequential checkout often performs
- better. The size and compression level of a repository might also influence how
- well the parallel version performs.
-
-+`checkout.autoStash`::
-+ When set to true, enable autostash for `git checkout` and
-+ `git switch` branch switching operations. When a branch
-+ switch would fail because local changes (in the index or the
-+ working tree) overlap with paths that differ between the
-+ current and target branch, a temporary stash entry is
-+ automatically created before the switch and applied after
-+ it completes. If the local changes do not overlap with the
-+ branch difference, the switch proceeds normally without
-+ stashing.
-+ This option can be overridden by the `--no-autostash` and
-+ `--autostash` options of linkgit:git-checkout[1] and
-+ linkgit:git-switch[1].
-+ Defaults to false.
-+
- `checkout.thresholdForParallelism`::
- When running parallel checkout with a small number of files, the cost
- of subprocess spawning and inter-process communication might outweigh
-
## Documentation/git-checkout.adoc ##
-@@ Documentation/git-checkout.adoc: When switching branches with `--merge`, staged changes may be lost.
- `merge.conflictStyle` configuration variable. Possible values are
- `merge` (default), `diff3`, and `zdiff3`.
+@@ Documentation/git-checkout.adoc: working tree, by copying them from elsewhere, extracting a tarball, etc.
+ are different between the current branch and the branch to
+ which you are switching, the command refuses to switch
+ branches in order to preserve your modifications in context.
+- However, with this option, a three-way merge between the current
+- branch, your working tree contents, and the new branch
+- is done, and you will be on the new branch.
+-+
+-When a merge conflict happens, the index entries for conflicting
+-paths are left unmerged, and you need to resolve the conflicts
+-and mark the resolved paths with `git add` (or `git rm` if the merge
+-should result in deletion of the path).
++ With this option, the conflicting local changes are
++ automatically stashed before the switch and reapplied
++ afterwards. If the local changes do not overlap with the
++ differences between branches, the switch proceeds without
++ stashing. If reapplying the stash results in conflicts, the
++ entry is saved to the stash list so you can use `git stash
++ pop` to recover and `git stash drop` when done.
+ +
+ When checking out paths from the index, this option lets you recreate
+ the conflicted merge in the specified paths. This option cannot be
+ used when checking out paths from a tree-ish.
+-+
+-When switching branches with `--merge`, staged changes may be lost.
+
+ `--conflict=<style>`::
+ The same as `--merge` option above, but changes the way the
+@@ Documentation/git-checkout.adoc: $ git checkout mytopic
+ error: You have local changes to 'frotz'; not switching branches.
+ ------------
+
+-You can give the `-m` flag to the command, which would try a
+-three-way merge:
++You can give the `-m` flag to the command, which would save the local
++changes in a stash entry and reset the working tree to allow switching:
+
+ ------------
+ $ git checkout -m mytopic
+-Auto-merging frotz
++Created autostash: 7a9afa3
++Applied autostash.
+ ------------
+
+-After this three-way merge, the local modifications are _not_
++After the switch, the local modifications are reapplied and are _not_
+ registered in your index file, so `git diff` would show you what
+ changes you made since the tip of the new branch.
+
+ === 3. Merge conflict
+
+-When a merge conflict happens during switching branches with
+-the `-m` option, you would see something like this:
++When the locally modified files overlap with files that need to be
++updated by the branch switch, the changes are stashed and reapplied
++after the switch. If the stash application results in conflicts,
++they are not resolved and the stash is saved to the stash list:
+
+ ------------
+ $ git checkout -m mytopic
+-Auto-merging frotz
+-ERROR: Merge conflict in frotz
+-fatal: merge program failed
++Created autostash: 7a9afa3
++Applying autostash resulted in conflicts.
++Your changes are safe in the stash.
++You can run "git stash pop" or "git stash drop" at any time.
+ ------------
-+`--autostash`::
-+`--no-autostash`::
-+ When switching branches, if any of the paths that differ
-+ between the current branch and the target branch have local
-+ changes (in the index or the working tree), automatically
-+ create a temporary stash entry before the operation begins,
-+ and apply it after the operation ends. If the local changes
-+ do not overlap with the branch difference, the switch proceeds
-+ without stashing. When a stash entry is created and the
-+ subsequent application results in conflicts, the stash entry
-+ is saved so that you can use `git stash pop` to recover and
-+ `git stash drop` when done. Use with `--force` to always
-+ stash local changes regardless of conflicts.
-+
- `-p`::
- `--patch`::
- Interactively select hunks in the difference between the
+-At this point, `git diff` shows the changes cleanly merged as in
+-the previous example, as well as the changes in the conflicted
+-files. Edit and resolve the conflict and mark it resolved with
+-`git add` as usual:
+-
+-------------
+-$ edit frotz
+-$ git add frotz
+-------------
++At this point, `git stash pop` can be used to recover and resolve
++the conflicts, and `git stash drop` to discard the stash when done.
+
+ CONFIGURATION
+ -------------
## Documentation/git-switch.adoc ##
-@@ Documentation/git-switch.adoc: should result in deletion of the path).
- `merge.conflictStyle` configuration variable. Possible values are
- `merge` (default), `diff3`, and `zdiff3`.
+@@ Documentation/git-switch.adoc: variable.
+ If you have local modifications to one or more files that are
+ different between the current branch and the branch to which
+ you are switching, the command refuses to switch branches in
+- order to preserve your modifications in context. However,
+- with this option, a three-way merge between the current
+- branch, your working tree contents, and the new branch is
+- done, and you will be on the new branch.
+-+
+-When a merge conflict happens, the index entries for conflicting
+-paths are left unmerged, and you need to resolve the conflicts
+-and mark the resolved paths with `git add` (or `git rm` if the merge
+-should result in deletion of the path).
++ order to preserve your modifications in context. With this
++ option, the conflicting local changes are automatically
++ stashed before the switch and reapplied afterwards. If the
++ local changes do not overlap with the differences between
++ branches, the switch proceeds without stashing. If
++ reapplying the stash results in conflicts, the entry is
++ saved to the stash list so you can use `git stash pop` to
++ recover and `git stash drop` when done.
+
+ `--conflict=<style>`::
+ The same as `--merge` option above, but changes the way the
+@@ Documentation/git-switch.adoc: $ git switch mytopic
+ error: You have local changes to 'frotz'; not switching branches.
+ ------------
+
+-You can give the `-m` flag to the command, which would try a three-way
+-merge:
++You can give the `-m` flag to the command, which would save the local
++changes in a stash entry and reset the working tree to allow switching:
+
+ ------------
+ $ git switch -m mytopic
+-Auto-merging frotz
++Created autostash: 7a9afa3
++Applied autostash.
+ ------------
+
+-After this three-way merge, the local modifications are _not_
++After the switch, the local modifications are reapplied and are _not_
+ registered in your index file, so `git diff` would show you what
+ changes you made since the tip of the new branch.
-+`--autostash`::
-+`--no-autostash`::
-+ When switching branches, if any of the paths that differ
-+ between the current branch and the target branch have local
-+ changes (in the index or the working tree), automatically
-+ create a temporary stash entry before the operation begins,
-+ and apply it after the operation ends. If the local changes
-+ do not overlap with the branch difference, the switch proceeds
-+ without stashing. When a stash entry is created and the
-+ subsequent application results in conflicts, the stash entry
-+ is saved so that you can use `git stash pop` to recover and
-+ `git stash drop` when done. Use with `--force` to always
-+ stash local changes regardless of conflicts.
-+
- `-q`::
- `--quiet`::
- Quiet, suppress feedback messages.
## builtin/checkout.c ##
+@@
+ #include "merge-ll.h"
+ #include "lockfile.h"
+ #include "mem-pool.h"
+-#include "merge-ort-wrappers.h"
+ #include "object-file.h"
+ #include "object-name.h"
+ #include "odb.h"
@@
#include "repo-settings.h"
#include "resolve-undo.h"
@@ builtin/checkout.c
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
-@@ builtin/checkout.c: struct checkout_opts {
- int only_merge_on_switching_branches;
- int can_switch_when_in_progress;
- int orphan_from_empty_tree;
-+ int autostash;
- int empty_pathspec_ok;
- int checkout_index;
- int checkout_worktree;
+@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
+
+ ret = unpack_trees(2, trees, &topts);
+ clear_unpack_trees_porcelain(&topts);
+- if (ret == -1) {
+- /*
+- * Unpack couldn't do a trivial merge; either
+- * give up or do a real merge, depending on
+- * whether the merge flag was used.
+- */
+- struct tree *work;
+- struct tree *old_tree;
+- struct merge_options o;
+- struct strbuf sb = STRBUF_INIT;
+- struct strbuf old_commit_shortname = STRBUF_INIT;
+-
+- if (!opts->merge)
+- return 1;
+-
+- /*
+- * Without old_branch_info->commit, the below is the same as
+- * the two-tree unpack we already tried and failed.
+- */
+- if (!old_branch_info->commit)
+- return 1;
+- old_tree = repo_get_commit_tree(the_repository,
+- old_branch_info->commit);
+-
+- if (repo_index_has_changes(the_repository, old_tree, &sb))
+- die(_("cannot continue with staged changes in "
+- "the following files:\n%s"), sb.buf);
+- strbuf_release(&sb);
+-
+- /* Do more real merge */
+-
+- /*
+- * We update the index fully, then write the
+- * tree from the index, then merge the new
+- * branch with the current tree, with the old
+- * branch as the base. Then we reset the index
+- * (but not the working tree) to the new
+- * branch, leaving the working tree as the
+- * merged version, but skipping unmerged
+- * entries in the index.
+- */
+-
+- add_files_to_cache(the_repository, NULL, NULL, NULL, 0,
+- 0, 0);
+- init_ui_merge_options(&o, the_repository);
+- o.verbosity = 0;
+- work = write_in_core_index_as_tree(the_repository);
+-
+- ret = reset_tree(new_tree,
+- opts, 1,
+- writeout_error, new_branch_info);
+- if (ret)
+- return ret;
+- o.ancestor = old_branch_info->name;
+- if (!old_branch_info->name) {
+- strbuf_add_unique_abbrev(&old_commit_shortname,
+- &old_branch_info->commit->object.oid,
+- DEFAULT_ABBREV);
+- o.ancestor = old_commit_shortname.buf;
+- }
+- o.branch1 = new_branch_info->name;
+- o.branch2 = "local";
+- o.conflict_style = opts->conflict_style;
+- ret = merge_ort_nonrecursive(&o,
+- new_tree,
+- work,
+- old_tree);
+- if (ret < 0)
+- die(NULL);
+- ret = reset_tree(new_tree,
+- opts, 0,
+- writeout_error, new_branch_info);
+- strbuf_release(&o.obuf);
+- strbuf_release(&old_commit_shortname);
+- if (ret)
+- return ret;
+- }
++ if (ret == -1)
++ return 1;
+ }
+
+ if (!cache_tree_fully_valid(the_repository->index->cache_tree))
+@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
+ if (write_locked_index(the_repository->index, &lock_file, COMMIT_LOCK))
+ die(_("unable to write new index file"));
+
+- if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
+- show_local_changes(&new_branch_info->commit->object, &opts->diff_options);
+-
+ return 0;
+ }
+
@@ builtin/checkout.c: static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
-+ if (opts->autostash) {
++ if (opts->merge) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
-+ if (opts->discard_changes ||
-+ checkout_would_clobber_changes(&old_branch_info,
++ if (checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info))
+ create_autostash_ref(the_repository,
+ "CHECKOUT_AUTOSTASH");
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
++ if (opts->conflict_style >= 0) {
++ struct strbuf cfg = STRBUF_INIT;
++ strbuf_addf(&cfg, "merge.conflictStyle=%s",
++ conflict_style_name(opts->conflict_style));
++ git_config_push_parameter(cfg.buf);
++ strbuf_release(&cfg);
++ }
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
++
++ discard_index(the_repository->index);
++ if (repo_read_index(the_repository) < 0)
++ die(_("index file corrupt"));
++
++ if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
++ show_local_changes(&new_branch_info->commit->object,
++ &opts->diff_options);
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
-@@ builtin/checkout.c: static int git_checkout_config(const char *var, const char *value,
- opts->dwim_new_local_branch = git_config_bool(var, value);
- return 0;
- }
-+ if (!strcmp(var, "checkout.autostash")) {
-+ opts->autostash = git_config_bool(var, value);
-+ return 0;
-+ }
-
- if (starts_with(var, "submodule."))
- return git_default_submodule_config(var, value, NULL);
-@@ builtin/checkout.c: static struct option *add_common_switch_branch_options(
- PARSE_OPT_NOCOMPLETE),
- OPT_BOOL(0, "ignore-other-worktrees", &opts->ignore_other_worktrees,
- N_("do not check if another worktree is using this branch")),
-+ OPT_AUTOSTASH(&opts->autostash),
- OPT_END()
- };
- struct option *newopts = parse_options_concat(prevopts, options);
- ## t/meson.build ##
-@@ t/meson.build: integration_tests = [
- 't2030-unresolve-info.sh',
- 't2050-git-dir-relative.sh',
- 't2060-switch.sh',
-+ 't2061-switch-autostash.sh',
- 't2070-restore.sh',
- 't2071-restore-patch.sh',
- 't2072-restore-pathspec-file.sh',
+ ## sequencer.c ##
+@@ sequencer.c: static void create_autostash_internal(struct repository *r,
+ &oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
+ }
+
+- printf(_("Created autostash: %s\n"), buf.buf);
++ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
+ if (reset_head(r, &ropts) < 0)
+ die(_("could not reset --hard"));
+ discard_index(r->index);
- ## t/t2061-switch-autostash.sh (new) ##
-@@
-+#!/bin/sh
-+
-+test_description='checkout/switch --autostash tests'
-+
-+GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
-+export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
-+
-+. ./test-lib.sh
-+
-+test_expect_success 'setup' '
-+ echo file0content >file0 &&
-+ echo file1content >file1 &&
-+ git add . &&
-+ test_tick &&
-+ git commit -m "initial commit" &&
-+ git branch other-branch &&
-+ echo file1main >file1 &&
-+ git add . &&
-+ test_tick &&
-+ git commit -m "modify file1 on main" &&
-+ git checkout other-branch &&
-+ echo file1other >file1 &&
-+ git add . &&
-+ test_tick &&
-+ git commit -m "modify file1 on other-branch" &&
-+ echo file2content >file2 &&
-+ git add . &&
-+ test_tick &&
-+ git commit -m "add file2 on other-branch" &&
-+ git checkout main
+ ## t/t7201-co.sh ##
+@@ t/t7201-co.sh: test_expect_success 'format of merge conflict from checkout -m' '
+ test_cmp expect current &&
+
+ cat <<-EOF >expect &&
+- <<<<<<< simple
++ <<<<<<< Updated upstream
+ a
+ c
+ e
+ =======
+ b
+ d
+- >>>>>>> local
++ >>>>>>> Stashed changes
+ EOF
+ test_cmp expect two
+ '
+@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
+ git checkout --merge --conflict=diff3 simple &&
+
+ cat <<-EOF >expect &&
+- <<<<<<< simple
++ <<<<<<< Updated upstream
+ a
+ c
+ e
+- ||||||| main
++ ||||||| Stash base
+ a
+ b
+ c
+@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
+ =======
+ b
+ d
+- >>>>>>> local
++ >>>>>>> Stashed changes
+ EOF
+ test_cmp expect two
+ '
+
++test_expect_success 'checkout --merge --conflict=zdiff3 <branch>' '
++ git checkout -f main &&
++ git reset --hard &&
++ git clean -f &&
++
++ fill a b X d e >two &&
++ git checkout --merge --conflict=zdiff3 simple &&
++
++ cat <<-EOF >expect &&
++ a
++ <<<<<<< Updated upstream
++ c
++ ||||||| Stash base
++ b
++ c
++ d
++ =======
++ b
++ X
++ d
++ >>>>>>> Stashed changes
++ e
++ EOF
++ test_cmp expect two
+'
+
-+test_expect_success 'switch --autostash skips stash when no conflict' '
-+ git branch branch1 other-branch &&
-+ echo dirty >file0 &&
-+ git switch --autostash branch1 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git switch main
++test_expect_success 'checkout -m respects merge.conflictStyle config' '
++ git checkout -f main &&
++ git reset --hard &&
++ git clean -f &&
++
++ test_config merge.conflictStyle diff3 &&
++ fill b d >two &&
++ git checkout -m simple &&
++
++ cat <<-EOF >expect &&
++ <<<<<<< Updated upstream
++ a
++ c
++ e
++ ||||||| Stash base
++ a
++ b
++ c
++ d
++ e
++ =======
++ b
++ d
++ >>>>>>> Stashed changes
++ EOF
++ test_cmp expect two
+'
+
-+test_expect_success 'checkout --autostash skips stash when no conflict' '
-+ git branch branch2 other-branch &&
-+ echo dirty >file0 &&
-+ git checkout --autostash branch2 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git checkout main
-+'
++test_expect_success 'checkout -m skips stash when no conflict' '
++ git checkout -f main &&
++ git clean -f &&
+
-+test_expect_success 'switch: checkout.autostash config skips stash when no conflict' '
-+ git branch branch3 other-branch &&
-+ echo dirty >file0 &&
-+ test_config checkout.autostash true &&
-+ git switch branch3 >actual 2>&1 &&
++ fill 0 x y z >same &&
++ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git switch main
++ fill 0 x y z >expect &&
++ test_cmp expect same
+'
+
-+test_expect_success 'checkout: checkout.autostash config skips stash when no conflict' '
-+ git branch branch4 other-branch &&
-+ echo dirty >file0 &&
-+ test_config checkout.autostash true &&
-+ git checkout branch4 >actual 2>&1 &&
++test_expect_success 'checkout -m skips stash with non-conflicting dirty index' '
++ git checkout -f main &&
++ git clean -f &&
++
++ fill 0 x y z >same &&
++ git add same &&
++ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git checkout main
++ fill 0 x y z >expect &&
++ test_cmp expect same
+'
+
-+test_expect_success '--no-autostash overrides checkout.autostash' '
-+ git branch branch5 other-branch &&
-+ echo dirty >file1 &&
-+ test_config checkout.autostash true &&
-+ test_must_fail git switch --no-autostash branch5 2>stderr &&
-+ test_grep ! "Created autostash" stderr &&
-+ git checkout -- file1
-+'
++test_expect_success 'checkout -m stashes and applies on conflicting changes' '
++ git checkout -f main &&
++ git clean -f &&
+
-+test_expect_success '--autostash overrides checkout.autostash=false' '
-+ git branch branch6 other-branch &&
-+ echo dirty >file0 &&
-+ test_config checkout.autostash false &&
-+ git switch --autostash branch6 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git switch main
++ fill 1 2 3 4 5 6 7 >one &&
++ git checkout -m side >actual 2>&1 &&
++ test_grep "Created autostash" actual &&
++ test_grep "Applied autostash" actual &&
++ fill 1 2 3 4 5 6 7 >expect &&
++ test_cmp expect one
+'
+
-+test_expect_success 'autostash with non-conflicting dirty index' '
-+ git branch branch7 other-branch &&
-+ echo dirty-index >file0 &&
-+ git add file0 &&
-+ git switch --autostash branch7 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ echo dirty-index >expected &&
-+ test_cmp expected file0 &&
-+ git checkout -- file0 &&
-+ git switch main
++test_expect_success 'checkout -m with mixed staged and unstaged changes' '
++ git checkout -f main &&
++ git clean -f &&
++
++ fill 0 x y z >same &&
++ git add same &&
++ fill 1 2 3 4 5 6 7 >one &&
++ git checkout -m side >actual 2>&1 &&
++ test_grep "Created autostash" actual &&
++ test_grep "Applied autostash" actual &&
++ fill 0 x y z >expect &&
++ test_cmp expect same &&
++ fill 1 2 3 4 5 6 7 >expect &&
++ test_cmp expect one
+'
+
-+test_expect_success 'autostash bypasses conflicting local changes' '
-+ git branch branch8 other-branch &&
-+ echo dirty >file1 &&
-+ test_must_fail git switch branch8 2>stderr &&
++test_expect_success 'checkout -m stashes on truly conflicting changes' '
++ git checkout -f main &&
++ git clean -f &&
++
++ fill 1 2 3 4 5 >one &&
++ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
-+ git switch --autostash branch8 >actual 2>&1 &&
++ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
-+ git reset --hard &&
-+ git switch main
++ git reset --hard
+'
+
-+test_expect_success 'autostash is a no-op with clean worktree' '
-+ git branch branch9 other-branch &&
-+ git switch --autostash branch9 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ git switch main
-+'
-+
-+test_expect_success '--autostash with --merge skips stash when no conflict' '
-+ git branch branch10 other-branch &&
-+ echo dirty >file0 &&
-+ git switch --autostash --merge branch10 >actual 2>&1 &&
-+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git switch main
-+'
++test_expect_success 'checkout -m produces usable stash on conflict' '
++ git checkout -f main &&
++ git clean -f &&
+
-+test_expect_success 'autostash with staged conflicting changes' '
-+ git branch branch11 other-branch &&
-+ echo staged-change >file1 &&
-+ git add file1 &&
-+ git switch --autostash branch11 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applying autostash resulted in conflicts" actual &&
++ fill 1 2 3 4 5 >one &&
++ git checkout -m side >actual 2>&1 &&
+ test_grep "Your changes are safe in the stash" actual &&
-+ git stash drop &&
-+ git reset --hard &&
-+ git switch main
++ git checkout -f main &&
++ git stash pop &&
++ fill 1 2 3 4 5 >expect &&
++ test_cmp expect one
+'
+
-+test_expect_success '--autostash with --force preserves dirty changes' '
-+ git branch branch12 other-branch &&
-+ echo dirty-force >file1 &&
-+ git switch --autostash --force branch12 >actual 2>&1 &&
++test_expect_success 'checkout -m stashes on staged conflicting changes' '
++ git checkout -f main &&
++ git clean -f &&
++
++ fill 1 2 3 4 5 >one &&
++ git add one &&
++ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
-+ git reset --hard &&
-+ git switch main
++ git reset --hard
+'
+
-+test_expect_success '--autostash with new branch creation skips stash' '
-+ echo dirty >file0 &&
-+ git switch --autostash -c branch13 >actual 2>&1 &&
++test_expect_success 'checkout -m -b skips stash with dirty tree' '
++ git checkout -f main &&
++ git clean -f &&
++
++ fill 0 x y z >same &&
++ git checkout -m -b newbranch >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
-+ echo dirty >expected &&
-+ test_cmp expected file0 &&
-+ git switch main &&
-+ git branch -D branch13
++ fill 0 x y z >expect &&
++ test_cmp expect same &&
++ git checkout main &&
++ git branch -D newbranch
+'
+
-+test_expect_success 'autostash with conflicting changes that apply cleanly' '
-+ git branch branch14 other-branch &&
-+ echo file1other >file1 &&
-+ git switch --autostash branch14 >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applied autostash" actual &&
-+ echo file1other >expected &&
-+ test_cmp expected file1 &&
-+ git switch main
-+'
+ test_expect_success 'switch to another branch while carrying a deletion' '
+ git checkout -f main &&
+ git reset --hard &&
+
+ ## xdiff-interface.c ##
+@@ xdiff-interface.c: int parse_conflict_style_name(const char *value)
+ return -1;
+ }
+
++const char *conflict_style_name(int style)
++{
++ switch (style) {
++ case XDL_MERGE_DIFF3:
++ return "diff3";
++ case XDL_MERGE_ZEALOUS_DIFF3:
++ return "zdiff3";
++ default:
++ return "merge";
++ }
++}
+
-+test_done
+ int git_xmerge_style = -1;
+
+ int git_xmerge_config(const char *var, const char *value,
- ## t/t9902-completion.sh ##
-@@ t/t9902-completion.sh: test_expect_success 'double dash "git checkout"' '
- --ignore-other-worktrees Z
- --recurse-submodules Z
- --auto-advance Z
-+ --autostash Z
- --progress Z
- --guess Z
- --no-guess Z
+ ## xdiff-interface.h ##
+@@ xdiff-interface.h: void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
+ void xdiff_clear_find_func(xdemitconf_t *xecfg);
+ struct config_context;
+ int parse_conflict_style_name(const char *value);
++const char *conflict_style_name(int style);
+ int git_xmerge_config(const char *var, const char *value,
+ const struct config_context *ctx, void *cb);
+ extern int git_xmerge_style;
Documentation/git-checkout.adoc | 50 +++++-----
Documentation/git-switch.adoc | 26 ++---
builtin/checkout.c | 160 +++++++++++++++---------------
sequencer.c | 2 +-
t/t7201-co.sh | 170 +++++++++++++++++++++++++++++++-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
7 files changed, 293 insertions(+), 128 deletions(-)
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..1e8adf6ef3 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -251,20 +251,17 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
are different between the current branch and the branch to
which you are switching, the command refuses to switch
branches in order to preserve your modifications in context.
- However, with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch
- is done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ With this option, the conflicting local changes are
+ automatically stashed before the switch and reapplied
+ afterwards. If the local changes do not overlap with the
+ differences between branches, the switch proceeds without
+ stashing. If reapplying the stash results in conflicts, the
+ entry is saved to the stash list so you can use `git stash
+ pop` to recover and `git stash drop` when done.
+
When checking out paths from the index, this option lets you recreate
the conflicted merge in the specified paths. This option cannot be
used when checking out paths from a tree-ish.
-+
-When switching branches with `--merge`, staged changes may be lost.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -578,39 +575,36 @@ $ git checkout mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a
-three-way merge:
+You can give the `-m` flag to the command, which would save the local
+changes in a stash entry and reset the working tree to allow switching:
------------
$ git checkout -m mytopic
-Auto-merging frotz
+Created autostash: 7a9afa3
+Applied autostash.
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
=== 3. Merge conflict
-When a merge conflict happens during switching branches with
-the `-m` option, you would see something like this:
+When the locally modified files overlap with files that need to be
+updated by the branch switch, the changes are stashed and reapplied
+after the switch. If the stash application results in conflicts,
+they are not resolved and the stash is saved to the stash list:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-ERROR: Merge conflict in frotz
-fatal: merge program failed
+Created autostash: 7a9afa3
+Applying autostash resulted in conflicts.
+Your changes are safe in the stash.
+You can run "git stash pop" or "git stash drop" at any time.
------------
-At this point, `git diff` shows the changes cleanly merged as in
-the previous example, as well as the changes in the conflicted
-files. Edit and resolve the conflict and mark it resolved with
-`git add` as usual:
-
-------------
-$ edit frotz
-$ git add frotz
-------------
+At this point, `git stash pop` can be used to recover and resolve
+the conflicts, and `git stash drop` to discard the stash when done.
CONFIGURATION
-------------
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..350e760fd2 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -126,15 +126,14 @@ variable.
If you have local modifications to one or more files that are
different between the current branch and the branch to which
you are switching, the command refuses to switch branches in
- order to preserve your modifications in context. However,
- with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch is
- done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ order to preserve your modifications in context. With this
+ option, the conflicting local changes are automatically
+ stashed before the switch and reapplied afterwards. If the
+ local changes do not overlap with the differences between
+ branches, the switch proceeds without stashing. If
+ reapplying the stash results in conflicts, the entry is
+ saved to the stash list so you can use `git stash pop` to
+ recover and `git stash drop` when done.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -217,15 +216,16 @@ $ git switch mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a three-way
-merge:
+You can give the `-m` flag to the command, which would save the local
+changes in a stash entry and reset the working tree to allow switching:
------------
$ git switch -m mytopic
-Auto-merging frotz
+Created autostash: 7a9afa3
+Applied autostash.
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..d3b3b815a7 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -17,7 +17,6 @@
#include "merge-ll.h"
#include "lockfile.h"
#include "mem-pool.h"
-#include "merge-ort-wrappers.h"
#include "object-file.h"
#include "object-name.h"
#include "odb.h"
@@ -30,6 +29,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -845,83 +845,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
ret = unpack_trees(2, trees, &topts);
clear_unpack_trees_porcelain(&topts);
- if (ret == -1) {
- /*
- * Unpack couldn't do a trivial merge; either
- * give up or do a real merge, depending on
- * whether the merge flag was used.
- */
- struct tree *work;
- struct tree *old_tree;
- struct merge_options o;
- struct strbuf sb = STRBUF_INIT;
- struct strbuf old_commit_shortname = STRBUF_INIT;
-
- if (!opts->merge)
- return 1;
-
- /*
- * Without old_branch_info->commit, the below is the same as
- * the two-tree unpack we already tried and failed.
- */
- if (!old_branch_info->commit)
- return 1;
- old_tree = repo_get_commit_tree(the_repository,
- old_branch_info->commit);
-
- if (repo_index_has_changes(the_repository, old_tree, &sb))
- die(_("cannot continue with staged changes in "
- "the following files:\n%s"), sb.buf);
- strbuf_release(&sb);
-
- /* Do more real merge */
-
- /*
- * We update the index fully, then write the
- * tree from the index, then merge the new
- * branch with the current tree, with the old
- * branch as the base. Then we reset the index
- * (but not the working tree) to the new
- * branch, leaving the working tree as the
- * merged version, but skipping unmerged
- * entries in the index.
- */
-
- add_files_to_cache(the_repository, NULL, NULL, NULL, 0,
- 0, 0);
- init_ui_merge_options(&o, the_repository);
- o.verbosity = 0;
- work = write_in_core_index_as_tree(the_repository);
-
- ret = reset_tree(new_tree,
- opts, 1,
- writeout_error, new_branch_info);
- if (ret)
- return ret;
- o.ancestor = old_branch_info->name;
- if (!old_branch_info->name) {
- strbuf_add_unique_abbrev(&old_commit_shortname,
- &old_branch_info->commit->object.oid,
- DEFAULT_ABBREV);
- o.ancestor = old_commit_shortname.buf;
- }
- o.branch1 = new_branch_info->name;
- o.branch2 = "local";
- o.conflict_style = opts->conflict_style;
- ret = merge_ort_nonrecursive(&o,
- new_tree,
- work,
- old_tree);
- if (ret < 0)
- die(NULL);
- ret = reset_tree(new_tree,
- opts, 0,
- writeout_error, new_branch_info);
- strbuf_release(&o.obuf);
- strbuf_release(&old_commit_shortname);
- if (ret)
- return ret;
- }
+ if (ret == -1)
+ return 1;
}
if (!cache_tree_fully_valid(the_repository->index->cache_tree))
@@ -930,9 +855,6 @@ static int merge_working_tree(const struct checkout_opts *opts,
if (write_locked_index(the_repository->index, &lock_file, COMMIT_LOCK))
die(_("unable to write new index file"));
- if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
- show_local_changes(&new_branch_info->commit->object, &opts->diff_options);
-
return 0;
}
@@ -1157,6 +1079,55 @@ static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
+static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
+ struct branch_info *new_branch_info)
+{
+ struct tree_desc trees[2];
+ struct tree *old_tree, *new_tree;
+ struct unpack_trees_options topts;
+ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
+ const struct object_id *old_commit_oid;
+ int ret;
+
+ if (!new_branch_info->commit)
+ return 0;
+
+ old_commit_oid = old_branch_info->commit ?
+ &old_branch_info->commit->object.oid :
+ the_hash_algo->empty_tree;
+ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
+ if (!old_tree)
+ return 0;
+
+ new_tree = repo_get_commit_tree(the_repository,
+ new_branch_info->commit);
+ if (!new_tree)
+ return 0;
+ if (repo_parse_tree(the_repository, new_tree) < 0)
+ return 0;
+
+ memset(&topts, 0, sizeof(topts));
+ topts.head_idx = -1;
+ topts.src_index = the_repository->index;
+ topts.dst_index = &tmp_index;
+ topts.initial_checkout = is_index_unborn(the_repository->index);
+ topts.merge = 1;
+ topts.update = 1;
+ topts.dry_run = 1;
+ topts.quiet = 1;
+ topts.fn = twoway_merge;
+
+ init_tree_desc(&trees[0], &old_tree->object.oid,
+ old_tree->buffer, old_tree->size);
+ init_tree_desc(&trees[1], &new_tree->object.oid,
+ new_tree->buffer, new_tree->size);
+
+ ret = unpack_trees(2, trees, &topts);
+ discard_index(&tmp_index);
+
+ return ret != 0;
+}
+
static int switch_branches(const struct checkout_opts *opts,
struct branch_info *new_branch_info)
{
@@ -1202,9 +1173,19 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (opts->merge) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info))
+ create_autostash_ref(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
branch_info_release(&old_branch_info);
return ret;
}
@@ -1215,6 +1196,23 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ if (opts->conflict_style >= 0) {
+ struct strbuf cfg = STRBUF_INIT;
+ strbuf_addf(&cfg, "merge.conflictStyle=%s",
+ conflict_style_name(opts->conflict_style));
+ git_config_push_parameter(cfg.buf);
+ strbuf_release(&cfg);
+ }
+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
+
+ discard_index(the_repository->index);
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+
+ if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
+ show_local_changes(&new_branch_info->commit->object,
+ &opts->diff_options);
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
diff --git a/sequencer.c b/sequencer.c
index aafd0bc959..72f0afe609 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4677,7 +4677,7 @@ static void create_autostash_internal(struct repository *r,
&oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
}
- printf(_("Created autostash: %s\n"), buf.buf);
+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
if (reset_head(r, &ropts) < 0)
die(_("could not reset --hard"));
discard_index(r->index);
diff --git a/t/t7201-co.sh b/t/t7201-co.sh
index 9bcf7c0b40..9ace3962ba 100755
--- a/t/t7201-co.sh
+++ b/t/t7201-co.sh
@@ -171,14 +171,14 @@ test_expect_success 'format of merge conflict from checkout -m' '
test_cmp expect current &&
cat <<-EOF >expect &&
- <<<<<<< simple
+ <<<<<<< Updated upstream
a
c
e
=======
b
d
- >>>>>>> local
+ >>>>>>> Stashed changes
EOF
test_cmp expect two
'
@@ -192,11 +192,11 @@ test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
git checkout --merge --conflict=diff3 simple &&
cat <<-EOF >expect &&
- <<<<<<< simple
+ <<<<<<< Updated upstream
a
c
e
- ||||||| main
+ ||||||| Stash base
a
b
c
@@ -205,11 +205,171 @@ test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
=======
b
d
- >>>>>>> local
+ >>>>>>> Stashed changes
EOF
test_cmp expect two
'
+test_expect_success 'checkout --merge --conflict=zdiff3 <branch>' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ fill a b X d e >two &&
+ git checkout --merge --conflict=zdiff3 simple &&
+
+ cat <<-EOF >expect &&
+ a
+ <<<<<<< Updated upstream
+ c
+ ||||||| Stash base
+ b
+ c
+ d
+ =======
+ b
+ X
+ d
+ >>>>>>> Stashed changes
+ e
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m respects merge.conflictStyle config' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ test_config merge.conflictStyle diff3 &&
+ fill b d >two &&
+ git checkout -m simple &&
+
+ cat <<-EOF >expect &&
+ <<<<<<< Updated upstream
+ a
+ c
+ e
+ ||||||| Stash base
+ a
+ b
+ c
+ d
+ e
+ =======
+ b
+ d
+ >>>>>>> Stashed changes
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m skips stash when no conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m skips stash with non-conflicting dirty index' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m stashes and applies on conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m with mixed staged and unstaged changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on truly conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m produces usable stash on conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git checkout -f main &&
+ git stash pop &&
+ fill 1 2 3 4 5 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on staged conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git add one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "Created autostash" actual &&
+ test_grep "Applying autostash resulted in conflicts" actual &&
+ test_grep "Your changes are safe in the stash" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m -b skips stash with dirty tree' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m -b newbranch >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ git checkout main &&
+ git branch -D newbranch
+'
+
test_expect_success 'switch to another branch while carrying a deletion' '
git checkout -f main &&
git reset --hard &&
diff --git a/xdiff-interface.c b/xdiff-interface.c
index f043330f2a..5ee2b96d0a 100644
--- a/xdiff-interface.c
+++ b/xdiff-interface.c
@@ -325,6 +325,18 @@ int parse_conflict_style_name(const char *value)
return -1;
}
+const char *conflict_style_name(int style)
+{
+ switch (style) {
+ case XDL_MERGE_DIFF3:
+ return "diff3";
+ case XDL_MERGE_ZEALOUS_DIFF3:
+ return "zdiff3";
+ default:
+ return "merge";
+ }
+}
+
int git_xmerge_style = -1;
int git_xmerge_config(const char *var, const char *value,
diff --git a/xdiff-interface.h b/xdiff-interface.h
index fbc4ceec40..ce54e1c0e0 100644
--- a/xdiff-interface.h
+++ b/xdiff-interface.h
@@ -55,6 +55,7 @@ void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
void xdiff_clear_find_func(xdemitconf_t *xecfg);
struct config_context;
int parse_conflict_style_name(const char *value);
+const char *conflict_style_name(int style);
int git_xmerge_config(const char *var, const char *value,
const struct config_context *ctx, void *cb);
extern int git_xmerge_style;
base-commit: 67006b9db8b772423ad0706029286096307d2567
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-12 14:40 ` Junio C Hamano
2026-03-12 19:33 ` [PATCH v31 0/2] status: add status.compareBranches config for multiple branch comparisons Harald Nordgren
@ 2026-03-13 14:29 ` Phillip Wood
2026-03-14 17:17 ` Junio C Hamano
1 sibling, 1 reply; 29+ messages in thread
From: Phillip Wood @ 2026-03-13 14:29 UTC (permalink / raw)
To: Junio C Hamano, Harald Nordgren via GitGitGadget; +Cc: git, Harald Nordgren
On 12/03/2026 14:40, Junio C Hamano wrote:
>
> Perhaps doing it this way would make it more worth doing?
>
> - At the beginning of branch switching, ask a new helper function
> that takes the branch we are switching to as an argument this
> question:
>
> Do any paths that are different between the current branch and
> the branch we are switching to have local (i.e., either in the
> index or in the working tree) change [Yes/No]?
>
> - When the answer is "yes", save the local changes to a new stash
> entry, and clear the local changes from the index and from the
> working tree. If not, do not bother with stash at all.
Can we avoid the extra check and stash if the user passed "--autostash"
and unpack_trees() fails because it would overwrite local changes in
merge_working_tree()?
> If we can sell it as an improved implementation of "-m", we probably
> can lose some code that the current "-m" implementation uses to do
> its merge; we'd be instead using the "unstash" code paths.
That would be nice but I think "git checkout --recurse-submodules -m
<branch>" currently updates submodules whereas "git stash" does not know
how to recurse submodules.
It would be nice to teach "git stash" to recurse submodules but I don't
think it is completly straight forward as we'd need to store the object
id of the submodule's stash commit in the parent stash.
Thanks
Phillip
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v3] checkout: add --autostash option for branch switching
2026-03-13 9:23 ` [PATCH v3] " Harald Nordgren via GitGitGadget
@ 2026-03-13 17:16 ` Junio C Hamano
2026-03-13 19:33 ` [PATCH] " Harald Nordgren
2026-03-14 9:59 ` [PATCH v4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
1 sibling, 1 reply; 29+ messages in thread
From: Junio C Hamano @ 2026-03-13 17:16 UTC (permalink / raw)
To: Harald Nordgren via GitGitGadget; +Cc: git, Harald Nordgren
"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Harald Nordgren <haraldnordgren@gmail.com>
>
> When switching branches, local modifications in the working tree can
> prevent the checkout from succeeding. While "git rebase" and "git
> merge" already support --autostash to handle this case automatically,
> "git checkout" and "git switch" require users to manually stash and
> unstash their changes.
>
> Teach "git checkout" and "git switch" to accept --autostash and
> --no-autostash options that automatically create a temporary stash
> entry before the branch switch begins and apply it after the switch
> completes. If the stash application results in conflicts, the stash
> entry is saved to the stash list so the user can resolve them later.
>
> Also add a checkout.autoStash configuration option that enables this
> behavior by default, which can be overridden with --no-autostash on
> the command line.
>
> Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
> ---
> checkout: 'autostash' for branch switching
Almost all of the above is now stale, as we no longer call this the
"autostash" feature. It turned into a project to vastly improve the
"checkout --merge" option, so the proposed log message needs to be
revamped to match.
Using the autostash mechanism to simplify checkout.c is a great
direction, I think, provided we can maintain the high-quality
conflict information (branch labels) that users expect from '-m'.
For a new iteration that invalidates almost everything from the
previous round, it is easier if range-diff were omitted from the
reviewers' point of view (I am not a GGG user, so I do not know if
there is such an option to do so). It is not a huge deal as we can
simply ignore the large block of lines as irrelevant ;-)
> Documentation/git-checkout.adoc | 50 +++++-----
> Documentation/git-switch.adoc | 26 ++---
> builtin/checkout.c | 160 +++++++++++++++---------------
> sequencer.c | 2 +-
> t/t7201-co.sh | 170 +++++++++++++++++++++++++++++++-
> xdiff-interface.c | 12 +++
> xdiff-interface.h | 1 +
> 7 files changed, 293 insertions(+), 128 deletions(-)
>
> diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
> index 43ccf47cf6..1e8adf6ef3 100644
> --- a/Documentation/git-checkout.adoc
> +++ b/Documentation/git-checkout.adoc
> @@ -251,20 +251,17 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
> are different between the current branch and the branch to
> which you are switching, the command refuses to switch
> branches in order to preserve your modifications in context.
> + With this option, the conflicting local changes are
> + automatically stashed before the switch and reapplied
> + afterwards. If the local changes do not overlap with the
> + differences between branches, the switch proceeds without
> + stashing. If reapplying the stash results in conflicts, the
> + entry is saved to the stash list so you can use `git stash
> + pop` to recover and `git stash drop` when done.
Great.
> -+
> -When switching branches with `--merge`, staged changes may be lost.
Hmph, I have to admit that I never do "checkout" to go to a different
branch with a dirty index that does not match the current HEAD, with
or without "--merge"; in fact, not just "I never do" myself, I never
thought about what happens when somebody does so, so I do not offhand
know what the current code without this patch does, or what the updated
code would. In "stash && switch && unstash" flow, the last "unstash"
is what decides if the original changes in the index is reapplied to
the index or not, IIRC, so we still may lose the distinction between
what has been and what has not been added to the index after all,
depending on the way how the new code does the "unstash" step.
> @@ -578,39 +575,36 @@ $ git checkout mytopic
> error: You have local changes to 'frotz'; not switching branches.
> ------------
>
> -You can give the `-m` flag to the command, which would try a
> -three-way merge:
> +You can give the `-m` flag to the command, which would save the local
> +changes in a stash entry and reset the working tree to allow switching:
>
> ------------
> $ git checkout -m mytopic
> -Auto-merging frotz
> +Created autostash: 7a9afa3
> +Applied autostash.
> ------------
>
> -After this three-way merge, the local modifications are _not_
> +After the switch, the local modifications are reapplied and are _not_
> registered in your index file, so `git diff` would show you what
> changes you made since the tip of the new branch.
OK.
> === 3. Merge conflict
>
> -When a merge conflict happens during switching branches with
> -the `-m` option, you would see something like this:
> +When the locally modified files overlap with files that need to be
> +updated by the branch switch, the changes are stashed and reapplied
> +after the switch. If the stash application results in conflicts,
In the original text of this third example (and as I understand,
these examples are written to be read more-or-less independently),
the `-m` option was explicitly mentioned to make it clear that this
"conflict" situation is relevant only when the option is used. In
the updated text, it is not. We should mention `-m` somewhere in
the updated text as well: When the `--merge` (`-m`) option is in
effect, _this_ happens.
> +they are not resolved and the stash is saved to the stash list:
>
> ------------
> $ git checkout -m mytopic
> -Auto-merging frotz
> -ERROR: Merge conflict in frotz
> -fatal: merge program failed
> +Created autostash: 7a9afa3
> +Applying autostash resulted in conflicts.
> +Your changes are safe in the stash.
> +You can run "git stash pop" or "git stash drop" at any time.
> ------------
Nice. It may probably be on a bit too verbose side, though. The
fact we are creating a stash entry or the object name of it is not
interesting to the user unless there is a conflict, so I would have
thought we would not say "Created autostash" at all, and the last
three lines would be replaced with something like the following
and shown _only_ when the application of the stashed changes did not
succeed cleanly:
Your local changes are stashed, however, applying it to carry
forward your local changes resulted in conflicts:
- You can try resolving them now. If you resolved them
successfully, discard the stash entry with "git stash drop".
- Alternatively you can "git reset --hard" if you do not want
to deal with them right now, and later "git stash pop" to
recover your local changes.
> -At this point, `git diff` shows the changes cleanly merged as in
> -the previous example, as well as the changes in the conflicted
> -files. Edit and resolve the conflict and mark it resolved with
> -`git add` as usual:
> -
> -------------
> -$ edit frotz
> -$ git add frotz
> -------------
After resolving and adding, they need to do "git stash drop" it, but
other than that, this part of the example is still valid, isn't it?
The improvement this patch brings in is that the user now has
another choice, i.e.,
Instead of editing and resolving the conflicts right now, you
can clear the slate with `git reset --hard` and resurrect the
local changes, safely stored in the stash at your leisure, with
`git stash pop`.
> +At this point, `git stash pop` can be used to recover and resolve
> +the conflicts, and `git stash drop` to discard the stash when done.
This is not giving a wrong instruction per-se, but my suggestion
above can be used to extend it if we wanted to. It may make it
easier to understand to the readers, or it may make it way too
verbose. I dunno.
> diff --git a/builtin/checkout.c b/builtin/checkout.c
> index 1d1667fa4c..d3b3b815a7 100644
> --- a/builtin/checkout.c
> +++ b/builtin/checkout.c
> @@ -17,7 +17,6 @@
> #include "merge-ll.h"
> #include "lockfile.h"
> #include "mem-pool.h"
> -#include "merge-ort-wrappers.h"
> #include "object-file.h"
> #include "object-name.h"
> #include "odb.h"
> @@ -30,6 +29,7 @@
> #include "repo-settings.h"
> #include "resolve-undo.h"
> #include "revision.h"
> +#include "sequencer.h"
> #include "setup.h"
> #include "submodule.h"
> #include "symlinks.h"
> @@ -845,83 +845,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
>
> ret = unpack_trees(2, trees, &topts);
> clear_unpack_trees_porcelain(&topts);
> - if (ret == -1) {
> - /*
> - * Unpack couldn't do a trivial merge; either
> - * give up or do a real merge, depending on
> - * whether the merge flag was used.
> - */
> -...
> - }
> + if (ret == -1)
> + return 1;
> }
So pleased to see so much custom code getting removed ;-).
> diff --git a/sequencer.c b/sequencer.c
> index aafd0bc959..72f0afe609 100644
> --- a/sequencer.c
> +++ b/sequencer.c
> @@ -4677,7 +4677,7 @@ static void create_autostash_internal(struct repository *r,
> &oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
> }
>
> - printf(_("Created autostash: %s\n"), buf.buf);
> + fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
> if (reset_head(r, &ropts) < 0)
> die(_("could not reset --hard"));
> discard_index(r->index);
Is this change something all the callers that passes control through
this part desire? Don't some of them want the output go to the
standard output, not standard error?
If we are going in the direction I suggested above (i.e., be silent
about our use of stashes before it becomes relevant to the user),
this part may have to become a conditional
if (!silent)
printf(_("Created autostash: %s\n"), buf.buf);
or something, where all the current callers before the "checkout
--merge" adds a new caller would pass silent==0 to.
Whatever changes this function needs, it looks more like a separate
preparatory patch to me.
> diff --git a/t/t7201-co.sh b/t/t7201-co.sh
> index 9bcf7c0b40..9ace3962ba 100755
> --- a/t/t7201-co.sh
> +++ b/t/t7201-co.sh
> @@ -171,14 +171,14 @@ test_expect_success 'format of merge conflict from checkout -m' '
> test_cmp expect current &&
>
> cat <<-EOF >expect &&
> - <<<<<<< simple
> + <<<<<<< Updated upstream
> a
> c
> e
> =======
> b
> d
> - >>>>>>> local
> + >>>>>>> Stashed changes
> EOF
> test_cmp expect two
> '
Historically, 'git checkout -m' provided the branch names (e.g.,
"<<<<<<< simple"), which is very helpful context. It would be great
if we could find a way to pass the branch names through as labels to
the stash application. It may require a preparatory change to "git
stash apply" to allow customizing the labels---the underlying
machinery at the lowest level does have ability to use labels the
callers want it to use, so it may be just the matter of plumbing it
through the callchain. I didn't check.
> diff --git a/xdiff-interface.c b/xdiff-interface.c
> index f043330f2a..5ee2b96d0a 100644
> --- a/xdiff-interface.c
> +++ b/xdiff-interface.c
> @@ -325,6 +325,18 @@ int parse_conflict_style_name(const char *value)
> return -1;
> }
>
> +const char *conflict_style_name(int style)
> +{
> + switch (style) {
> + case XDL_MERGE_DIFF3:
> + return "diff3";
> + case XDL_MERGE_ZEALOUS_DIFF3:
> + return "zdiff3";
> + default:
> + return "merge";
> + }
> +}
> +
> int git_xmerge_style = -1;
>
> int git_xmerge_config(const char *var, const char *value,
> diff --git a/xdiff-interface.h b/xdiff-interface.h
> index fbc4ceec40..ce54e1c0e0 100644
> --- a/xdiff-interface.h
> +++ b/xdiff-interface.h
> @@ -55,6 +55,7 @@ void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
> void xdiff_clear_find_func(xdemitconf_t *xecfg);
> struct config_context;
> int parse_conflict_style_name(const char *value);
> +const char *conflict_style_name(int style);
> int git_xmerge_config(const char *var, const char *value,
> const struct config_context *ctx, void *cb);
> extern int git_xmerge_style;
This may be a useful addition, but it would probably need to become
a separate preparatory patch, possibly with its own test.
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-13 17:16 ` Junio C Hamano
@ 2026-03-13 19:33 ` Harald Nordgren
2026-03-13 20:30 ` Junio C Hamano
0 siblings, 1 reply; 29+ messages in thread
From: Harald Nordgren @ 2026-03-13 19:33 UTC (permalink / raw)
To: gitster; +Cc: git, gitgitgadget, haraldnordgren
> Almost all of the above is now stale, as we no longer call this the
> "autostash" feature. It turned into a project to vastly improve the
> "checkout --merge" option, so the proposed log message needs to be
> revamped to match.
My feeling is that this feature will drift far away from what I initially
needed. I have never used 'checkout -m' but stash->checkout-unstash is a
pattern I use a lot.
I wonder with how complicated this might turn out, if it would be better to
have this as a separate feature called autostash (mirroring the same
feature from git pull rebase which I have enabled).
Harald
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-13 19:33 ` [PATCH] " Harald Nordgren
@ 2026-03-13 20:30 ` Junio C Hamano
0 siblings, 0 replies; 29+ messages in thread
From: Junio C Hamano @ 2026-03-13 20:30 UTC (permalink / raw)
To: Harald Nordgren; +Cc: git, gitgitgadget
Harald Nordgren <haraldnordgren@gmail.com> writes:
>> Almost all of the above is now stale, as we no longer call this the
>> "autostash" feature. It turned into a project to vastly improve the
>> "checkout --merge" option, so the proposed log message needs to be
>> revamped to match.
>
> My feeling is that this feature will drift far away from what I initially
> needed. I have never used 'checkout -m' but stash->checkout-unstash is a
> pattern I use a lot.
Perhaps. But things like "checkout --autostash", which can entirely
be done by the end user via a wrapper script or an alias, is not
interesting enough from my point of view.
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v4] checkout: -m (--merge) uses autostash when switching branches
2026-03-13 9:23 ` [PATCH v3] " Harald Nordgren via GitGitGadget
2026-03-13 17:16 ` Junio C Hamano
@ 2026-03-14 9:59 ` Harald Nordgren via GitGitGadget
2026-03-15 2:25 ` Junio C Hamano
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
1 sibling, 2 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-14 9:59 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches with "git checkout -m", local modifications
can block the switch. Teach the -m flow to create a temporary stash
before switching and reapply it after. On success, only "Applied
autostash." is shown. If reapplying causes conflicts, the stash is
kept and the user is told they can resolve and run "git stash drop",
or run "git reset --hard" and later "git stash pop" to recover their
changes.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
checkout: 'autostash' for branch switching
cc: Phillip Wood phillip.wood123@gmail.com
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v4
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v4
Pull-Request: https://github.com/git/git/pull/2234
Range-diff vs v3:
1: 05f1e53163 ! 1: 5d49c0031a checkout: add --autostash option for branch switching
@@ Metadata
Author: Harald Nordgren <haraldnordgren@gmail.com>
## Commit message ##
- checkout: add --autostash option for branch switching
+ checkout: -m (--merge) uses autostash when switching branches
- When switching branches, local modifications in the working tree can
- prevent the checkout from succeeding. While "git rebase" and "git
- merge" already support --autostash to handle this case automatically,
- "git checkout" and "git switch" require users to manually stash and
- unstash their changes.
-
- Teach "git checkout" and "git switch" to accept --autostash and
- --no-autostash options that automatically create a temporary stash
- entry before the branch switch begins and apply it after the switch
- completes. If the stash application results in conflicts, the stash
- entry is saved to the stash list so the user can resolve them later.
-
- Also add a checkout.autoStash configuration option that enables this
- behavior by default, which can be overridden with --no-autostash on
- the command line.
+ When switching branches with "git checkout -m", local modifications
+ can block the switch. Teach the -m flow to create a temporary stash
+ before switching and reapply it after. On success, only "Applied
+ autostash." is shown. If reapplying causes conflicts, the stash is
+ kept and the user is told they can resolve and run "git stash drop",
+ or run "git reset --hard" and later "git stash pop" to recover their
+ changes.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
@@ Documentation/git-checkout.adoc: $ git checkout mytopic
------------
$ git checkout -m mytopic
-Auto-merging frotz
-+Created autostash: 7a9afa3
+Applied autostash.
------------
@@ Documentation/git-checkout.adoc: $ git checkout mytopic
-When a merge conflict happens during switching branches with
-the `-m` option, you would see something like this:
-+When the locally modified files overlap with files that need to be
-+updated by the branch switch, the changes are stashed and reapplied
-+after the switch. If the stash application results in conflicts,
-+they are not resolved and the stash is saved to the stash list:
++When the `--merge` (`-m`) option is in effect and the locally
++modified files overlap with files that need to be updated by the
++branch switch, the changes are stashed and reapplied after the
++switch. If the stash application results in conflicts, they are not
++resolved and the stash is saved to the stash list:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-ERROR: Merge conflict in frotz
-fatal: merge program failed
-+Created autostash: 7a9afa3
-+Applying autostash resulted in conflicts.
-+Your changes are safe in the stash.
-+You can run "git stash pop" or "git stash drop" at any time.
- ------------
+-------------
++Your local changes are stashed, however, applying it to carry
++forward your local changes resulted in conflicts:
-At this point, `git diff` shows the changes cleanly merged as in
-the previous example, as well as the changes in the conflicted
-files. Edit and resolve the conflict and mark it resolved with
-`git add` as usual:
--
--------------
++ - You can try resolving them now. If you resolved them
++ successfully, discard the stash entry with "git stash drop".
+
++ - Alternatively you can "git reset --hard" if you do not want
++ to deal with them right now, and later "git stash pop" to
++ recover your local changes.
+ ------------
-$ edit frotz
-$ git add frotz
-------------
-+At this point, `git stash pop` can be used to recover and resolve
-+the conflicts, and `git stash drop` to discard the stash when done.
++
++You can try resolving the conflicts now. Edit the conflicting files
++and mark them resolved with `git add` as usual, then run `git stash
++drop` to discard the stash entry. Alternatively, you can clear the
++working tree with `git reset --hard` and recover your local changes
++later with `git stash pop`.
CONFIGURATION
-------------
@@ builtin/checkout.c: static void orphaned_commit_warning(struct commit *old_commi
static int switch_branches(const struct checkout_opts *opts,
struct branch_info *new_branch_info)
{
+@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ struct object_id rev;
+ int flag, writeout_error = 0;
+ int do_merge = 1;
++ struct strbuf old_commit_shortname = STRBUF_INIT;
++ const char *stash_label_ancestor = NULL;
+
+ trace2_cmd_mode("branch");
+
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
++ if (old_branch_info.name)
++ stash_label_ancestor = old_branch_info.name;
++ else if (old_branch_info.commit) {
++ strbuf_add_unique_abbrev(&old_commit_shortname,
++ &old_branch_info.commit->object.oid,
++ DEFAULT_ABBREV);
++ stash_label_ancestor = old_commit_shortname.buf;
++ }
++
+ if (opts->merge) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info))
-+ create_autostash_ref(the_repository,
-+ "CHECKOUT_AUTOSTASH");
++ create_autostash_ref_silent(the_repository,
++ "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
-+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
++ apply_autostash_ref_with_labels(the_repository,
++ "CHECKOUT_AUTOSTASH",
++ new_branch_info->name,
++ "local",
++ stash_label_ancestor);
branch_info_release(&old_branch_info);
++ strbuf_release(&old_commit_shortname);
return ret;
}
+ }
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ git_config_push_parameter(cfg.buf);
+ strbuf_release(&cfg);
+ }
-+ apply_autostash_ref(the_repository, "CHECKOUT_AUTOSTASH");
++ apply_autostash_ref_with_labels(the_repository, "CHECKOUT_AUTOSTASH",
++ new_branch_info->name, "local",
++ stash_label_ancestor);
+
+ discard_index(the_repository->index);
+ if (repo_read_index(the_repository) < 0)
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
++ strbuf_release(&old_commit_shortname);
+
+ return ret || writeout_error;
+ }
+
+ ## builtin/stash.c ##
+@@ builtin/stash.c: static void unstage_changes_unless_new(struct object_id *orig_tree)
+ die(_("could not write index"));
+ }
+
+-static int do_apply_stash(const char *prefix, struct stash_info *info,
+- int index, int quiet)
++static int do_apply_stash_with_labels(const char *prefix,
++ struct stash_info *info,
++ int index, int quiet,
++ const char *label1, const char *label2,
++ const char *label_ancestor)
+ {
+ int clean, ret;
+ int has_index = index;
+@@ builtin/stash.c: static int do_apply_stash(const char *prefix, struct stash_info *info,
+ init_ui_merge_options(&o, the_repository);
+
+- o.branch1 = "Updated upstream";
+- o.branch2 = "Stashed changes";
+- o.ancestor = "Stash base";
++ o.branch1 = label1 ? label1 : "Updated upstream";
++ o.branch2 = label2 ? label2 : "Stashed changes";
++ o.ancestor = label_ancestor ? label_ancestor : "Stash base";
+
+ if (oideq(&info->b_tree, &c_tree))
+ o.branch1 = "Version stash was based on";
+@@ builtin/stash.c: restore_untracked:
+ return ret;
+ }
+
++static int do_apply_stash(const char *prefix, struct stash_info *info,
++ int index, int quiet)
++{
++ return do_apply_stash_with_labels(prefix, info, index, quiet,
++ NULL, NULL, NULL);
++}
++
+ static int apply_stash(int argc, const char **argv, const char *prefix,
+ struct repository *repo UNUSED)
+ {
+ int ret = -1;
+ int quiet = 0;
+ int index = use_index;
++ const char *label1 = NULL, *label2 = NULL, *label_ancestor = NULL;
+ struct stash_info info = STASH_INFO_INIT;
+ struct option options[] = {
+ OPT__QUIET(&quiet, N_("be quiet, only report errors")),
+ OPT_BOOL(0, "index", &index,
+ N_("attempt to recreate the index")),
++ OPT_STRING(0, "ours-label", &label1, N_("label"),
++ N_("label for the upstream side in conflict markers")),
++ OPT_STRING(0, "theirs-label", &label2, N_("label"),
++ N_("label for the stashed side in conflict markers")),
++ OPT_STRING(0, "base-label", &label_ancestor, N_("label"),
++ N_("label for the base in diff3 conflict markers")),
+ OPT_END()
+ };
+
+@@ builtin/stash.c: static int apply_stash(int argc, const char **argv, const char *prefix,
+ if (get_stash_info(&info, argc, argv))
+ goto cleanup;
+
+- ret = do_apply_stash(prefix, &info, index, quiet);
++ ret = do_apply_stash_with_labels(prefix, &info, index, quiet,
++ label1, label2, label_ancestor);
+ cleanup:
+ free_stash_info(&info);
+ return ret;
## sequencer.c ##
+@@ sequencer.c: static enum todo_command peek_command(struct todo_list *todo_list, int offset)
+
+ static void create_autostash_internal(struct repository *r,
+ const char *path,
+- const char *refname)
++ const char *refname,
++ int silent)
+ {
+ struct strbuf buf = STRBUF_INIT;
+ struct lock_file lock_file = LOCK_INIT;
@@ sequencer.c: static void create_autostash_internal(struct repository *r,
&oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
}
- printf(_("Created autostash: %s\n"), buf.buf);
-+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
++ if (!silent)
++ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
if (reset_head(r, &ropts) < 0)
die(_("could not reset --hard"));
discard_index(r->index);
+@@ sequencer.c: static void create_autostash_internal(struct repository *r,
+
+ void create_autostash(struct repository *r, const char *path)
+ {
+- create_autostash_internal(r, path, NULL);
++ create_autostash_internal(r, path, NULL, 0);
+ }
+
+ void create_autostash_ref(struct repository *r, const char *refname)
+ {
+- create_autostash_internal(r, NULL, refname);
++ create_autostash_internal(r, NULL, refname, 0);
+ }
+
+-static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
++void create_autostash_ref_silent(struct repository *r, const char *refname)
++{
++ create_autostash_internal(r, NULL, refname, 1);
++}
++
++static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
++ const char *label1, const char *label2,
++ const char *label_ancestor)
+ {
+ struct child_process child = CHILD_PROCESS_INIT;
+ int ret = 0;
+@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+ child.no_stderr = 1;
+ strvec_push(&child.args, "stash");
+ strvec_push(&child.args, "apply");
++ if (label1)
++ strvec_pushf(&child.args, "--ours-label=%s", label1);
++ if (label2)
++ strvec_pushf(&child.args, "--theirs-label=%s", label2);
++ if (label_ancestor)
++ strvec_pushf(&child.args, "--base-label=%s", label_ancestor);
+ strvec_push(&child.args, stash_oid);
+ ret = run_command(&child);
+ }
+@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+ strvec_push(&store.args, stash_oid);
+ if (run_command(&store))
+ ret = error(_("cannot store %s"), stash_oid);
++ else if (attempt_apply)
++ fprintf(stderr,
++ _("Your local changes are stashed, however, applying it to carry\n"
++ "forward your local changes resulted in conflicts:\n"
++ "\n"
++ " - You can try resolving them now. If you resolved them\n"
++ " successfully, discard the stash entry with \"git stash drop\".\n"
++ "\n"
++ " - Alternatively you can \"git reset --hard\" if you do not want\n"
++ " to deal with them right now, and later \"git stash pop\" to\n"
++ " recover your local changes.\n"));
+ else
+ fprintf(stderr,
+- _("%s\n"
++ _("Autostash exists; creating a new stash entry.\n"
+ "Your changes are safe in the stash.\n"
+ "You can run \"git stash pop\" or"
+- " \"git stash drop\" at any time.\n"),
+- attempt_apply ?
+- _("Applying autostash resulted in conflicts.") :
+- _("Autostash exists; creating a new stash entry."));
++ " \"git stash drop\" at any time.\n"));
+ }
+
+ return ret;
+@@ sequencer.c: static int apply_save_autostash(const char *path, int attempt_apply)
+ }
+ strbuf_trim(&stash_oid);
+
+- ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
++ ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
++ NULL, NULL, NULL);
+
+ unlink(path);
+ strbuf_release(&stash_oid);
+@@ sequencer.c: int apply_autostash(const char *path)
+
+ int apply_autostash_oid(const char *stash_oid)
+ {
+- return apply_save_autostash_oid(stash_oid, 1);
++ return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
+ }
+
+ static int apply_save_autostash_ref(struct repository *r, const char *refname,
+- int attempt_apply)
++ int attempt_apply,
++ const char *label1, const char *label2,
++ const char *label_ancestor)
+ {
+ struct object_id stash_oid;
+ char stash_oid_hex[GIT_MAX_HEXSZ + 1];
+@@ sequencer.c: static int apply_save_autostash_ref(struct repository *r, const char *refname,
+ return error(_("autostash reference is a symref"));
+
+ oid_to_hex_r(stash_oid_hex, &stash_oid);
+- ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
++ ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
++ label1, label2, label_ancestor);
+
+ refs_delete_ref(get_main_ref_store(r), "", refname,
+ &stash_oid, REF_NO_DEREF);
+@@ sequencer.c: static int apply_save_autostash_ref(struct repository *r, const char *refname,
+
+ int save_autostash_ref(struct repository *r, const char *refname)
+ {
+- return apply_save_autostash_ref(r, refname, 0);
++ return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
+ }
+
+ int apply_autostash_ref(struct repository *r, const char *refname)
+ {
+- return apply_save_autostash_ref(r, refname, 1);
++ return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
++}
++
++int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
++ const char *label1, const char *label2,
++ const char *label_ancestor)
++{
++ return apply_save_autostash_ref(r, refname, 1,
++ label1, label2, label_ancestor);
+ }
+
+ static int checkout_onto(struct repository *r, struct replay_opts *opts,
- ## t/t7201-co.sh ##
-@@ t/t7201-co.sh: test_expect_success 'format of merge conflict from checkout -m' '
- test_cmp expect current &&
-
- cat <<-EOF >expect &&
-- <<<<<<< simple
-+ <<<<<<< Updated upstream
- a
- c
- e
- =======
- b
- d
-- >>>>>>> local
-+ >>>>>>> Stashed changes
+ ## sequencer.h ##
+@@ sequencer.h: void commit_post_rewrite(struct repository *r,
+
+ void create_autostash(struct repository *r, const char *path);
+ void create_autostash_ref(struct repository *r, const char *refname);
++void create_autostash_ref_silent(struct repository *r, const char *refname);
+ int save_autostash(const char *path);
+ int save_autostash_ref(struct repository *r, const char *refname);
+ int apply_autostash(const char *path);
+ int apply_autostash_oid(const char *stash_oid);
+ int apply_autostash_ref(struct repository *r, const char *refname);
++int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
++ const char *label1, const char *label2,
++ const char *label_ancestor);
+
+ #define SUMMARY_INITIAL_COMMIT (1 << 0)
+ #define SUMMARY_SHOW_AUTHOR_DATE (1 << 1)
+
+ ## t/t3420-rebase-autostash.sh ##
+@@ t/t3420-rebase-autostash.sh: create_expected_failure_apply () {
+ First, rewinding head to replay your work on top of it...
+ Applying: second commit
+ Applying: third commit
+- Applying autostash resulted in conflicts.
+- Your changes are safe in the stash.
+- You can run "git stash pop" or "git stash drop" at any time.
++ Your local changes are stashed, however, applying it to carry
++ forward your local changes resulted in conflicts:
++
++ - You can try resolving them now. If you resolved them
++ successfully, discard the stash entry with "git stash drop".
++
++ - Alternatively you can "git reset --hard" if you do not want
++ to deal with them right now, and later "git stash pop" to
++ recover your local changes.
EOF
- test_cmp expect two
- '
-@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
- git checkout --merge --conflict=diff3 simple &&
-
- cat <<-EOF >expect &&
-- <<<<<<< simple
-+ <<<<<<< Updated upstream
- a
- c
- e
-- ||||||| main
-+ ||||||| Stash base
- a
- b
- c
-@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
- =======
- b
- d
-- >>>>>>> local
-+ >>>>>>> Stashed changes
+ }
+
+ create_expected_failure_merge () {
+ cat >expected <<-EOF
+ $(grep "^Created autostash: [0-9a-f][0-9a-f]*\$" actual)
+- Applying autostash resulted in conflicts.
+- Your changes are safe in the stash.
+- You can run "git stash pop" or "git stash drop" at any time.
++ Your local changes are stashed, however, applying it to carry
++ forward your local changes resulted in conflicts:
++
++ - You can try resolving them now. If you resolved them
++ successfully, discard the stash entry with "git stash drop".
++
++ - Alternatively you can "git reset --hard" if you do not want
++ to deal with them right now, and later "git stash pop" to
++ recover your local changes.
+ Successfully rebased and updated refs/heads/rebased-feature-branch.
EOF
+ }
+
+ ## t/t7201-co.sh ##
+@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
test_cmp expect two
'
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+
+ cat <<-EOF >expect &&
+ a
-+ <<<<<<< Updated upstream
++ <<<<<<< simple
+ c
-+ ||||||| Stash base
++ ||||||| main
+ b
+ c
+ d
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ b
+ X
+ d
-+ >>>>>>> Stashed changes
++ >>>>>>> local
+ e
+ EOF
+ test_cmp expect two
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ git checkout -m simple &&
+
+ cat <<-EOF >expect &&
-+ <<<<<<< Updated upstream
++ <<<<<<< simple
+ a
+ c
+ e
-+ ||||||| Stash base
++ ||||||| main
+ a
+ b
+ c
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ =======
+ b
+ d
-+ >>>>>>> Stashed changes
++ >>>>>>> local
+ EOF
+ test_cmp expect two
+'
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ git add same &&
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
++ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git checkout -m side >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applying autostash resulted in conflicts" actual &&
-+ test_grep "Your changes are safe in the stash" actual &&
++ test_grep ! "Created autostash" actual &&
++ test_grep "resulted in conflicts" actual &&
++ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+
+ fill 1 2 3 4 5 >one &&
+ git checkout -m side >actual 2>&1 &&
-+ test_grep "Your changes are safe in the stash" actual &&
++ test_grep "recover your local changes" actual &&
+ git checkout -f main &&
+ git stash pop &&
+ fill 1 2 3 4 5 >expect &&
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
+ fill 1 2 3 4 5 >one &&
+ git add one &&
+ git checkout -m side >actual 2>&1 &&
-+ test_grep "Created autostash" actual &&
-+ test_grep "Applying autostash resulted in conflicts" actual &&
-+ test_grep "Your changes are safe in the stash" actual &&
++ test_grep ! "Created autostash" actual &&
++ test_grep "resulted in conflicts" actual &&
++ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
git checkout -f main &&
git reset --hard &&
+ ## t/t7600-merge.sh ##
+@@ t/t7600-merge.sh: test_expect_success 'merge --squash --autostash conflict does not attempt to app
+ >unrelated &&
+ git add unrelated &&
+ test_must_fail git merge --squash c7 --autostash >out 2>err &&
+- ! grep "Applying autostash resulted in conflicts." err &&
++ ! grep "resulted in conflicts" err &&
+ grep "When finished, apply stashed changes with \`git stash pop\`" out
+ '
+
+@@ t/t7600-merge.sh: test_expect_success 'merge with conflicted --autostash changes' '
+ git diff >expect &&
+ test_when_finished "test_might_fail git stash drop" &&
+ git merge --autostash c3 2>err &&
+- test_grep "Applying autostash resulted in conflicts." err &&
++ test_grep "resulted in conflicts" err &&
+ git show HEAD:file >merge-result &&
+ test_cmp result.1-9 merge-result &&
+ git stash show -p >actual &&
+
## xdiff-interface.c ##
@@ xdiff-interface.c: int parse_conflict_style_name(const char *value)
return -1;
Documentation/git-checkout.adoc | 56 +++++-----
Documentation/git-switch.adoc | 26 ++---
builtin/checkout.c | 179 +++++++++++++++++---------------
builtin/stash.c | 30 ++++--
sequencer.c | 67 +++++++++---
sequencer.h | 4 +
t/t3420-rebase-autostash.sh | 24 +++--
t/t7201-co.sh | 160 ++++++++++++++++++++++++++++
t/t7600-merge.sh | 4 +-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
11 files changed, 412 insertions(+), 151 deletions(-)
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..9d5f5c51ae 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -251,20 +251,17 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
are different between the current branch and the branch to
which you are switching, the command refuses to switch
branches in order to preserve your modifications in context.
- However, with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch
- is done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ With this option, the conflicting local changes are
+ automatically stashed before the switch and reapplied
+ afterwards. If the local changes do not overlap with the
+ differences between branches, the switch proceeds without
+ stashing. If reapplying the stash results in conflicts, the
+ entry is saved to the stash list so you can use `git stash
+ pop` to recover and `git stash drop` when done.
+
When checking out paths from the index, this option lets you recreate
the conflicted merge in the specified paths. This option cannot be
used when checking out paths from a tree-ish.
-+
-When switching branches with `--merge`, staged changes may be lost.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -578,39 +575,44 @@ $ git checkout mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a
-three-way merge:
+You can give the `-m` flag to the command, which would save the local
+changes in a stash entry and reset the working tree to allow switching:
------------
$ git checkout -m mytopic
-Auto-merging frotz
+Applied autostash.
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
=== 3. Merge conflict
-When a merge conflict happens during switching branches with
-the `-m` option, you would see something like this:
+When the `--merge` (`-m`) option is in effect and the locally
+modified files overlap with files that need to be updated by the
+branch switch, the changes are stashed and reapplied after the
+switch. If the stash application results in conflicts, they are not
+resolved and the stash is saved to the stash list:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-ERROR: Merge conflict in frotz
-fatal: merge program failed
-------------
+Your local changes are stashed, however, applying it to carry
+forward your local changes resulted in conflicts:
-At this point, `git diff` shows the changes cleanly merged as in
-the previous example, as well as the changes in the conflicted
-files. Edit and resolve the conflict and mark it resolved with
-`git add` as usual:
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
------------
-$ edit frotz
-$ git add frotz
-------------
+
+You can try resolving the conflicts now. Edit the conflicting files
+and mark them resolved with `git add` as usual, then run `git stash
+drop` to discard the stash entry. Alternatively, you can clear the
+working tree with `git reset --hard` and recover your local changes
+later with `git stash pop`.
CONFIGURATION
-------------
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..350e760fd2 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -126,15 +126,14 @@ variable.
If you have local modifications to one or more files that are
different between the current branch and the branch to which
you are switching, the command refuses to switch branches in
- order to preserve your modifications in context. However,
- with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch is
- done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ order to preserve your modifications in context. With this
+ option, the conflicting local changes are automatically
+ stashed before the switch and reapplied afterwards. If the
+ local changes do not overlap with the differences between
+ branches, the switch proceeds without stashing. If
+ reapplying the stash results in conflicts, the entry is
+ saved to the stash list so you can use `git stash pop` to
+ recover and `git stash drop` when done.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -217,15 +216,16 @@ $ git switch mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a three-way
-merge:
+You can give the `-m` flag to the command, which would save the local
+changes in a stash entry and reset the working tree to allow switching:
------------
$ git switch -m mytopic
-Auto-merging frotz
+Created autostash: 7a9afa3
+Applied autostash.
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..1968e46ae9 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -17,7 +17,6 @@
#include "merge-ll.h"
#include "lockfile.h"
#include "mem-pool.h"
-#include "merge-ort-wrappers.h"
#include "object-file.h"
#include "object-name.h"
#include "odb.h"
@@ -30,6 +29,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -845,83 +845,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
ret = unpack_trees(2, trees, &topts);
clear_unpack_trees_porcelain(&topts);
- if (ret == -1) {
- /*
- * Unpack couldn't do a trivial merge; either
- * give up or do a real merge, depending on
- * whether the merge flag was used.
- */
- struct tree *work;
- struct tree *old_tree;
- struct merge_options o;
- struct strbuf sb = STRBUF_INIT;
- struct strbuf old_commit_shortname = STRBUF_INIT;
-
- if (!opts->merge)
- return 1;
-
- /*
- * Without old_branch_info->commit, the below is the same as
- * the two-tree unpack we already tried and failed.
- */
- if (!old_branch_info->commit)
- return 1;
- old_tree = repo_get_commit_tree(the_repository,
- old_branch_info->commit);
-
- if (repo_index_has_changes(the_repository, old_tree, &sb))
- die(_("cannot continue with staged changes in "
- "the following files:\n%s"), sb.buf);
- strbuf_release(&sb);
-
- /* Do more real merge */
-
- /*
- * We update the index fully, then write the
- * tree from the index, then merge the new
- * branch with the current tree, with the old
- * branch as the base. Then we reset the index
- * (but not the working tree) to the new
- * branch, leaving the working tree as the
- * merged version, but skipping unmerged
- * entries in the index.
- */
-
- add_files_to_cache(the_repository, NULL, NULL, NULL, 0,
- 0, 0);
- init_ui_merge_options(&o, the_repository);
- o.verbosity = 0;
- work = write_in_core_index_as_tree(the_repository);
-
- ret = reset_tree(new_tree,
- opts, 1,
- writeout_error, new_branch_info);
- if (ret)
- return ret;
- o.ancestor = old_branch_info->name;
- if (!old_branch_info->name) {
- strbuf_add_unique_abbrev(&old_commit_shortname,
- &old_branch_info->commit->object.oid,
- DEFAULT_ABBREV);
- o.ancestor = old_commit_shortname.buf;
- }
- o.branch1 = new_branch_info->name;
- o.branch2 = "local";
- o.conflict_style = opts->conflict_style;
- ret = merge_ort_nonrecursive(&o,
- new_tree,
- work,
- old_tree);
- if (ret < 0)
- die(NULL);
- ret = reset_tree(new_tree,
- opts, 0,
- writeout_error, new_branch_info);
- strbuf_release(&o.obuf);
- strbuf_release(&old_commit_shortname);
- if (ret)
- return ret;
- }
+ if (ret == -1)
+ return 1;
}
if (!cache_tree_fully_valid(the_repository->index->cache_tree))
@@ -930,9 +855,6 @@ static int merge_working_tree(const struct checkout_opts *opts,
if (write_locked_index(the_repository->index, &lock_file, COMMIT_LOCK))
die(_("unable to write new index file"));
- if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
- show_local_changes(&new_branch_info->commit->object, &opts->diff_options);
-
return 0;
}
@@ -1157,6 +1079,55 @@ static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
+static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
+ struct branch_info *new_branch_info)
+{
+ struct tree_desc trees[2];
+ struct tree *old_tree, *new_tree;
+ struct unpack_trees_options topts;
+ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
+ const struct object_id *old_commit_oid;
+ int ret;
+
+ if (!new_branch_info->commit)
+ return 0;
+
+ old_commit_oid = old_branch_info->commit ?
+ &old_branch_info->commit->object.oid :
+ the_hash_algo->empty_tree;
+ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
+ if (!old_tree)
+ return 0;
+
+ new_tree = repo_get_commit_tree(the_repository,
+ new_branch_info->commit);
+ if (!new_tree)
+ return 0;
+ if (repo_parse_tree(the_repository, new_tree) < 0)
+ return 0;
+
+ memset(&topts, 0, sizeof(topts));
+ topts.head_idx = -1;
+ topts.src_index = the_repository->index;
+ topts.dst_index = &tmp_index;
+ topts.initial_checkout = is_index_unborn(the_repository->index);
+ topts.merge = 1;
+ topts.update = 1;
+ topts.dry_run = 1;
+ topts.quiet = 1;
+ topts.fn = twoway_merge;
+
+ init_tree_desc(&trees[0], &old_tree->object.oid,
+ old_tree->buffer, old_tree->size);
+ init_tree_desc(&trees[1], &new_tree->object.oid,
+ new_tree->buffer, new_tree->size);
+
+ ret = unpack_trees(2, trees, &topts);
+ discard_index(&tmp_index);
+
+ return ret != 0;
+}
+
static int switch_branches(const struct checkout_opts *opts,
struct branch_info *new_branch_info)
{
@@ -1165,6 +1136,8 @@ static int switch_branches(const struct checkout_opts *opts,
struct object_id rev;
int flag, writeout_error = 0;
int do_merge = 1;
+ struct strbuf old_commit_shortname = STRBUF_INIT;
+ const char *stash_label_ancestor = NULL;
trace2_cmd_mode("branch");
@@ -1202,10 +1175,34 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (old_branch_info.name)
+ stash_label_ancestor = old_branch_info.name;
+ else if (old_branch_info.commit) {
+ strbuf_add_unique_abbrev(&old_commit_shortname,
+ &old_branch_info.commit->object.oid,
+ DEFAULT_ABBREV);
+ stash_label_ancestor = old_commit_shortname.buf;
+ }
+
+ if (opts->merge) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info))
+ create_autostash_ref_silent(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref_with_labels(the_repository,
+ "CHECKOUT_AUTOSTASH",
+ new_branch_info->name,
+ "local",
+ stash_label_ancestor);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret;
}
}
@@ -1215,8 +1212,28 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ if (opts->conflict_style >= 0) {
+ struct strbuf cfg = STRBUF_INIT;
+ strbuf_addf(&cfg, "merge.conflictStyle=%s",
+ conflict_style_name(opts->conflict_style));
+ git_config_push_parameter(cfg.buf);
+ strbuf_release(&cfg);
+ }
+ apply_autostash_ref_with_labels(the_repository, "CHECKOUT_AUTOSTASH",
+ new_branch_info->name, "local",
+ stash_label_ancestor);
+
+ discard_index(the_repository->index);
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+
+ if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
+ show_local_changes(&new_branch_info->commit->object,
+ &opts->diff_options);
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret || writeout_error;
}
diff --git a/builtin/stash.c b/builtin/stash.c
index e79d612e57..1016d88e52 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -590,8 +590,11 @@ static void unstage_changes_unless_new(struct object_id *orig_tree)
die(_("could not write index"));
}
-static int do_apply_stash(const char *prefix, struct stash_info *info,
- int index, int quiet)
+static int do_apply_stash_with_labels(const char *prefix,
+ struct stash_info *info,
+ int index, int quiet,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
int clean, ret;
int has_index = index;
@@ -643,9 +646,9 @@ static int do_apply_stash(const char *prefix, struct stash_info *info,
init_ui_merge_options(&o, the_repository);
- o.branch1 = "Updated upstream";
- o.branch2 = "Stashed changes";
- o.ancestor = "Stash base";
+ o.branch1 = label1 ? label1 : "Updated upstream";
+ o.branch2 = label2 ? label2 : "Stashed changes";
+ o.ancestor = label_ancestor ? label_ancestor : "Stash base";
if (oideq(&info->b_tree, &c_tree))
o.branch1 = "Version stash was based on";
@@ -717,17 +720,31 @@ restore_untracked:
return ret;
}
+static int do_apply_stash(const char *prefix, struct stash_info *info,
+ int index, int quiet)
+{
+ return do_apply_stash_with_labels(prefix, info, index, quiet,
+ NULL, NULL, NULL);
+}
+
static int apply_stash(int argc, const char **argv, const char *prefix,
struct repository *repo UNUSED)
{
int ret = -1;
int quiet = 0;
int index = use_index;
+ const char *label1 = NULL, *label2 = NULL, *label_ancestor = NULL;
struct stash_info info = STASH_INFO_INIT;
struct option options[] = {
OPT__QUIET(&quiet, N_("be quiet, only report errors")),
OPT_BOOL(0, "index", &index,
N_("attempt to recreate the index")),
+ OPT_STRING(0, "ours-label", &label1, N_("label"),
+ N_("label for the upstream side in conflict markers")),
+ OPT_STRING(0, "theirs-label", &label2, N_("label"),
+ N_("label for the stashed side in conflict markers")),
+ OPT_STRING(0, "base-label", &label_ancestor, N_("label"),
+ N_("label for the base in diff3 conflict markers")),
OPT_END()
};
@@ -737,7 +754,8 @@ static int apply_stash(int argc, const char **argv, const char *prefix,
if (get_stash_info(&info, argc, argv))
goto cleanup;
- ret = do_apply_stash(prefix, &info, index, quiet);
+ ret = do_apply_stash_with_labels(prefix, &info, index, quiet,
+ label1, label2, label_ancestor);
cleanup:
free_stash_info(&info);
return ret;
diff --git a/sequencer.c b/sequencer.c
index aafd0bc959..53e04d8a94 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4632,7 +4632,8 @@ static enum todo_command peek_command(struct todo_list *todo_list, int offset)
static void create_autostash_internal(struct repository *r,
const char *path,
- const char *refname)
+ const char *refname,
+ int silent)
{
struct strbuf buf = STRBUF_INIT;
struct lock_file lock_file = LOCK_INIT;
@@ -4677,7 +4678,8 @@ static void create_autostash_internal(struct repository *r,
&oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
}
- printf(_("Created autostash: %s\n"), buf.buf);
+ if (!silent)
+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
if (reset_head(r, &ropts) < 0)
die(_("could not reset --hard"));
discard_index(r->index);
@@ -4689,15 +4691,22 @@ static void create_autostash_internal(struct repository *r,
void create_autostash(struct repository *r, const char *path)
{
- create_autostash_internal(r, path, NULL);
+ create_autostash_internal(r, path, NULL, 0);
}
void create_autostash_ref(struct repository *r, const char *refname)
{
- create_autostash_internal(r, NULL, refname);
+ create_autostash_internal(r, NULL, refname, 0);
}
-static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+void create_autostash_ref_silent(struct repository *r, const char *refname)
+{
+ create_autostash_internal(r, NULL, refname, 1);
+}
+
+static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct child_process child = CHILD_PROCESS_INIT;
int ret = 0;
@@ -4708,6 +4717,12 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
child.no_stderr = 1;
strvec_push(&child.args, "stash");
strvec_push(&child.args, "apply");
+ if (label1)
+ strvec_pushf(&child.args, "--ours-label=%s", label1);
+ if (label2)
+ strvec_pushf(&child.args, "--theirs-label=%s", label2);
+ if (label_ancestor)
+ strvec_pushf(&child.args, "--base-label=%s", label_ancestor);
strvec_push(&child.args, stash_oid);
ret = run_command(&child);
}
@@ -4726,15 +4741,23 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
strvec_push(&store.args, stash_oid);
if (run_command(&store))
ret = error(_("cannot store %s"), stash_oid);
+ else if (attempt_apply)
+ fprintf(stderr,
+ _("Your local changes are stashed, however, applying it to carry\n"
+ "forward your local changes resulted in conflicts:\n"
+ "\n"
+ " - You can try resolving them now. If you resolved them\n"
+ " successfully, discard the stash entry with \"git stash drop\".\n"
+ "\n"
+ " - Alternatively you can \"git reset --hard\" if you do not want\n"
+ " to deal with them right now, and later \"git stash pop\" to\n"
+ " recover your local changes.\n"));
else
fprintf(stderr,
- _("%s\n"
+ _("Autostash exists; creating a new stash entry.\n"
"Your changes are safe in the stash.\n"
"You can run \"git stash pop\" or"
- " \"git stash drop\" at any time.\n"),
- attempt_apply ?
- _("Applying autostash resulted in conflicts.") :
- _("Autostash exists; creating a new stash entry."));
+ " \"git stash drop\" at any time.\n"));
}
return ret;
@@ -4752,7 +4775,8 @@ static int apply_save_autostash(const char *path, int attempt_apply)
}
strbuf_trim(&stash_oid);
- ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
+ NULL, NULL, NULL);
unlink(path);
strbuf_release(&stash_oid);
@@ -4771,11 +4795,13 @@ int apply_autostash(const char *path)
int apply_autostash_oid(const char *stash_oid)
{
- return apply_save_autostash_oid(stash_oid, 1);
+ return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
}
static int apply_save_autostash_ref(struct repository *r, const char *refname,
- int attempt_apply)
+ int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct object_id stash_oid;
char stash_oid_hex[GIT_MAX_HEXSZ + 1];
@@ -4791,7 +4817,8 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
return error(_("autostash reference is a symref"));
oid_to_hex_r(stash_oid_hex, &stash_oid);
- ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
+ label1, label2, label_ancestor);
refs_delete_ref(get_main_ref_store(r), "", refname,
&stash_oid, REF_NO_DEREF);
@@ -4801,12 +4828,20 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
int save_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 0);
+ return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
}
int apply_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 1);
+ return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
+}
+
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
+{
+ return apply_save_autostash_ref(r, refname, 1,
+ label1, label2, label_ancestor);
}
static int checkout_onto(struct repository *r, struct replay_opts *opts,
diff --git a/sequencer.h b/sequencer.h
index 719684c8a9..68b94d86e3 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -227,11 +227,15 @@ void commit_post_rewrite(struct repository *r,
void create_autostash(struct repository *r, const char *path);
void create_autostash_ref(struct repository *r, const char *refname);
+void create_autostash_ref_silent(struct repository *r, const char *refname);
int save_autostash(const char *path);
int save_autostash_ref(struct repository *r, const char *refname);
int apply_autostash(const char *path);
int apply_autostash_oid(const char *stash_oid);
int apply_autostash_ref(struct repository *r, const char *refname);
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor);
#define SUMMARY_INITIAL_COMMIT (1 << 0)
#define SUMMARY_SHOW_AUTHOR_DATE (1 << 1)
diff --git a/t/t3420-rebase-autostash.sh b/t/t3420-rebase-autostash.sh
index ad3ba6a984..e4e2cb19ce 100755
--- a/t/t3420-rebase-autostash.sh
+++ b/t/t3420-rebase-autostash.sh
@@ -61,18 +61,30 @@ create_expected_failure_apply () {
First, rewinding head to replay your work on top of it...
Applying: second commit
Applying: third commit
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
EOF
}
create_expected_failure_merge () {
cat >expected <<-EOF
$(grep "^Created autostash: [0-9a-f][0-9a-f]*\$" actual)
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
Successfully rebased and updated refs/heads/rebased-feature-branch.
EOF
}
diff --git a/t/t7201-co.sh b/t/t7201-co.sh
index 9bcf7c0b40..f4fea7bb7a 100755
--- a/t/t7201-co.sh
+++ b/t/t7201-co.sh
@@ -210,6 +210,166 @@ test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
test_cmp expect two
'
+test_expect_success 'checkout --merge --conflict=zdiff3 <branch>' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ fill a b X d e >two &&
+ git checkout --merge --conflict=zdiff3 simple &&
+
+ cat <<-EOF >expect &&
+ a
+ <<<<<<< simple
+ c
+ ||||||| main
+ b
+ c
+ d
+ =======
+ b
+ X
+ d
+ >>>>>>> local
+ e
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m respects merge.conflictStyle config' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ test_config merge.conflictStyle diff3 &&
+ fill b d >two &&
+ git checkout -m simple &&
+
+ cat <<-EOF >expect &&
+ <<<<<<< simple
+ a
+ c
+ e
+ ||||||| main
+ a
+ b
+ c
+ d
+ e
+ =======
+ b
+ d
+ >>>>>>> local
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m skips stash when no conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m skips stash with non-conflicting dirty index' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m stashes and applies on conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m with mixed staged and unstaged changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on truly conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m produces usable stash on conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "recover your local changes" actual &&
+ git checkout -f main &&
+ git stash pop &&
+ fill 1 2 3 4 5 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on staged conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git add one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m -b skips stash with dirty tree' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m -b newbranch >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ git checkout main &&
+ git branch -D newbranch
+'
+
test_expect_success 'switch to another branch while carrying a deletion' '
git checkout -f main &&
git reset --hard &&
diff --git a/t/t7600-merge.sh b/t/t7600-merge.sh
index 9838094b66..b061cc1b4a 100755
--- a/t/t7600-merge.sh
+++ b/t/t7600-merge.sh
@@ -342,7 +342,7 @@ test_expect_success 'merge --squash --autostash conflict does not attempt to app
>unrelated &&
git add unrelated &&
test_must_fail git merge --squash c7 --autostash >out 2>err &&
- ! grep "Applying autostash resulted in conflicts." err &&
+ ! grep "resulted in conflicts" err &&
grep "When finished, apply stashed changes with \`git stash pop\`" out
'
@@ -914,7 +914,7 @@ test_expect_success 'merge with conflicted --autostash changes' '
git diff >expect &&
test_when_finished "test_might_fail git stash drop" &&
git merge --autostash c3 2>err &&
- test_grep "Applying autostash resulted in conflicts." err &&
+ test_grep "resulted in conflicts" err &&
git show HEAD:file >merge-result &&
test_cmp result.1-9 merge-result &&
git stash show -p >actual &&
diff --git a/xdiff-interface.c b/xdiff-interface.c
index f043330f2a..5ee2b96d0a 100644
--- a/xdiff-interface.c
+++ b/xdiff-interface.c
@@ -325,6 +325,18 @@ int parse_conflict_style_name(const char *value)
return -1;
}
+const char *conflict_style_name(int style)
+{
+ switch (style) {
+ case XDL_MERGE_DIFF3:
+ return "diff3";
+ case XDL_MERGE_ZEALOUS_DIFF3:
+ return "zdiff3";
+ default:
+ return "merge";
+ }
+}
+
int git_xmerge_style = -1;
int git_xmerge_config(const char *var, const char *value,
diff --git a/xdiff-interface.h b/xdiff-interface.h
index fbc4ceec40..ce54e1c0e0 100644
--- a/xdiff-interface.h
+++ b/xdiff-interface.h
@@ -55,6 +55,7 @@ void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
void xdiff_clear_find_func(xdemitconf_t *xecfg);
struct config_context;
int parse_conflict_style_name(const char *value);
+const char *conflict_style_name(int style);
int git_xmerge_config(const char *var, const char *value,
const struct config_context *ctx, void *cb);
extern int git_xmerge_style;
base-commit: dc6ecd5354dca88d51b6d6562777fc8fc10d77e1
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-13 14:29 ` [PATCH] checkout: add --autostash option for branch switching Phillip Wood
@ 2026-03-14 17:17 ` Junio C Hamano
2026-03-16 16:36 ` Phillip Wood
0 siblings, 1 reply; 29+ messages in thread
From: Junio C Hamano @ 2026-03-14 17:17 UTC (permalink / raw)
To: Phillip Wood; +Cc: Harald Nordgren via GitGitGadget, git, Harald Nordgren
Phillip Wood <phillip.wood123@gmail.com> writes:
> On 12/03/2026 14:40, Junio C Hamano wrote:
>>
>> Perhaps doing it this way would make it more worth doing?
>>
>> - At the beginning of branch switching, ask a new helper function
>> that takes the branch we are switching to as an argument this
>> question:
>>
>> Do any paths that are different between the current branch and
>> the branch we are switching to have local (i.e., either in the
>> index or in the working tree) change [Yes/No]?
>>
>> - When the answer is "yes", save the local changes to a new stash
>> entry, and clear the local changes from the index and from the
>> working tree. If not, do not bother with stash at all.
>
> Can we avoid the extra check and stash if the user passed "--autostash"
> and unpack_trees() fails because it would overwrite local changes in
> merge_working_tree()?
Sorry, but I couldn't quite figure out what you are saying here.
My guess on one part of what it says is that an explicit
"--autostash", we should stash without second guessing the user
(i.e., avoid chedk and stash). But then the latter part of the
sentence "and unpack_trees() fails ..." do not quite parse.
If the user gave "--autostash" and we check with unpack_trees()
dry-run and find out that a normal branch switch will be interfered
by the local changes, then we would stash, but that check made by a
dry-run unpack_trees() is not an "extra" check, so, that does not
work as a guess of what you are saying, either.
>> If we can sell it as an improved implementation of "-m", we probably
>> can lose some code that the current "-m" implementation uses to do
>> its merge; we'd be instead using the "unstash" code paths.
>
> That would be nice but I think "git checkout --recurse-submodules -m
> <branch>" currently updates submodules whereas "git stash" does not know
> how to recurse submodules.
Hmph, I do not do submodules outside what we already have, and I
certainly do not do "checkout --recurse-submodules" with or without
"-m" with local changes in our submodule.
But does "git stash" even need to know about recursing into
submodules for this? When checkout recurses into a submodule, that
checkout that is working in the repository of the submodule can
handle "-m" itself, which may stash the local changes made in the
submodule, no?
> It would be nice to teach "git stash" to recurse submodules but I don't
> think it is completly straight forward as we'd need to store the object
> id of the submodule's stash commit in the parent stash.
No, let's not add more commands that take "--recurse-submodules", if
we do not have to.
Thanks.
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v4] checkout: -m (--merge) uses autostash when switching branches
2026-03-14 9:59 ` [PATCH v4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
@ 2026-03-15 2:25 ` Junio C Hamano
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
1 sibling, 0 replies; 29+ messages in thread
From: Junio C Hamano @ 2026-03-15 2:25 UTC (permalink / raw)
To: Harald Nordgren via GitGitGadget, Phillip Wood; +Cc: git, Harald Nordgren
"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Harald Nordgren <haraldnordgren@gmail.com>
>
> When switching branches with "git checkout -m", local modifications
> can block the switch. Teach the -m flow to create a temporary stash
> before switching and reapply it after. On success, only "Applied
> autostash." is shown. If reapplying causes conflicts, the stash is
> kept and the user is told they can resolve and run "git stash drop",
> or run "git reset --hard" and later "git stash pop" to recover their
> changes.
>
> Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
> ---
> checkout: 'autostash' for branch switching
>
> cc: Phillip Wood phillip.wood123@gmail.com
I very much appreciate this, but somehow this "cc:" seems to have
been ignored.
> Documentation/git-checkout.adoc | 56 +++++-----
> Documentation/git-switch.adoc | 26 ++---
> builtin/checkout.c | 179 +++++++++++++++++---------------
> builtin/stash.c | 30 ++++--
> sequencer.c | 67 +++++++++---
> sequencer.h | 4 +
> t/t3420-rebase-autostash.sh | 24 +++--
> t/t7201-co.sh | 160 ++++++++++++++++++++++++++++
> t/t7600-merge.sh | 4 +-
> xdiff-interface.c | 12 +++
> xdiff-interface.h | 1 +
> 11 files changed, 412 insertions(+), 151 deletions(-)
>
> diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
> index 43ccf47cf6..9d5f5c51ae 100644
> --- a/Documentation/git-checkout.adoc
> +++ b/Documentation/git-checkout.adoc
> @@ -251,20 +251,17 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
> are different between the current branch and the branch to
> which you are switching, the command refuses to switch
> branches in order to preserve your modifications in context.
> - However, with this option, a three-way merge between the current
> - branch, your working tree contents, and the new branch
> - is done, and you will be on the new branch.
> -+
> -When a merge conflict happens, the index entries for conflicting
> -paths are left unmerged, and you need to resolve the conflicts
> -and mark the resolved paths with `git add` (or `git rm` if the merge
> -should result in deletion of the path).
> + With this option, the conflicting local changes are
> + automatically stashed before the switch and reapplied
> + afterwards. If the local changes do not overlap with the
> + differences between branches, the switch proceeds without
> + stashing. If reapplying the stash results in conflicts, the
> + entry is saved to the stash list so you can use `git stash
> + pop` to recover and `git stash drop` when done.
The new conflict message in sequencer.c correctly points out that,
if the user wants to use `git stash pop`, they would first need to
clear the working tree (e.g. `git reset --hard`) because the
conflicted changes are already there.
The documentation here is a bit too simplified and may give the
impression that `git stash pop` is the immediate next step when
conflicts are present in the worktree, which is probably something
we want to avoid.
> ------------
> $ git checkout -m mytopic
> -Auto-merging frotz
> +Applied autostash.
> ------------
I must be utterly confused, but the documentation _before_ this
patch does not seem to match the reality. In the following
illustration, I am switching to another branch that has contents in
transport.c that is different from the version in HEAD:
$ git checkout master
$ echo >>transport.c
$ git checkout jk/transport-color-leakfix
error: Your local changes to the following files would be overwritten...
transport.c
Please commit your changes or stash ...
Aborting
$ git checkout -m jk/transport-color-leakfix
M transport.c
Switched to branch 'jk/transport-color-leakfix'
Hmmmm.
The new behavior is silent about individual files, and instead
reports "Applied autostash."; even though it is consistent with how
other commands may behave when given the "--autostash" option, I am
not sure if it is even necessary.
Even in the documented version, the "Auto-merging" message is
presumably given only to the paths that needed real content-level
merges, and the paths that went through the trivial tree-level
merges (i.e., paths that are different from HEAD and mytopic but
without local changes) are not reported. The behaviour I am seeing
does not say anything to either trivially merged or really merged
paths while branch switching is happening, and shows the local
changes at the end of merge_working_tree() by calling
show_local_changes(), which runs "git diff --name-status"
in-process.
In either case, the stashing is a mere implementation detail of
carrying the local changes with you when you move to a different
branch, and moving with your local changes is what "Switching"
branch is about in this system, so I am not sure if we need to say
anything more than the current "Switched to branch 'mytopic'" in the
successful case.
> diff --git a/builtin/checkout.c b/builtin/checkout.c
> index 1d1667fa4c..1968e46ae9 100644
> --- a/builtin/checkout.c
> +++ b/builtin/checkout.c
> @@ -17,7 +17,6 @@
> #include "merge-ll.h"
> #include "lockfile.h"
> #include "mem-pool.h"
> -#include "merge-ort-wrappers.h"
> #include "object-file.h"
> #include "object-name.h"
> #include "odb.h"
> @@ -30,6 +29,7 @@
> #include "repo-settings.h"
> #include "resolve-undo.h"
> #include "revision.h"
> +#include "sequencer.h"
> #include "setup.h"
> #include "submodule.h"
> #include "symlinks.h"
> @@ -845,83 +845,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
>
> ret = unpack_trees(2, trees, &topts);
> clear_unpack_trees_porcelain(&topts);
> - if (ret == -1) {
> -...
> - }
> + if (ret == -1)
> + return 1;
> }
>
> if (!cache_tree_fully_valid(the_repository->index->cache_tree))
> @@ -930,9 +855,6 @@ static int merge_working_tree(const struct checkout_opts *opts,
> if (write_locked_index(the_repository->index, &lock_file, COMMIT_LOCK))
> die(_("unable to write new index file"));
>
> - if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
> - show_local_changes(&new_branch_info->commit->object, &opts->diff_options);
Ah, this is what you lost. I do not see a reason why we would want
to lose it.
> +static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
> + struct branch_info *new_branch_info)
> +{
> +...
> + init_tree_desc(&trees[0], &old_tree->object.oid,
> + old_tree->buffer, old_tree->size);
> + init_tree_desc(&trees[1], &new_tree->object.oid,
> + new_tree->buffer, new_tree->size);
> +
> + ret = unpack_trees(2, trees, &topts);
> + discard_index(&tmp_index);
In my original outline, I did suggest
- check if the local changes overlap with paths that differ between
HEAD and the new branch, and if not, we need no stashing;
otherwise stash.
- then switch branches with two-way merge
- then unstash.
But my untold assumption was that the initial check was done by
running a moral equivalent of "git diff --name-only HEAD <branch>"
to obtain the list of paths that differ between branches, and
a "git diff HEAD" to obtain the list of paths with local changes.
If you are using the real branch switching machinery in the
"dry-run" mode, then it is a waste to discard when we find out that
there is no overlap. So if I knew you would use the unpack-trees in
the dry-run mode, instead of two diffs, the outline would have been
more like
- Try unpack-trees in dry-run mode to see if there would get any
failure from overlap.
- If unpack-trees would succeed, we already have the right contents
in the index after the above dry-run. We did not touch the
working tree so far, but if we kept track of which paths we would
have written out to the working tree during our dry-run, we can
write them out of the index at this point and we are done (we of
course need to update HEAD to point at the new branch, give the
"Switched to branch 'foo'" message, etc., but the real work of
switching branches is done).
- If not, discard the dry-run index, stash, switch branches the
usual way, and unstash.
which might be more performant in a large tree.
Having said all that, what we see above, while running the same
unpack-trees once in dry-run mode and another for real, may be
performant enough and complicating the dry-run code might not be
worth it.
> @@ -1165,6 +1136,8 @@ static int switch_branches(const struct checkout_opts *opts,
> struct object_id rev;
> int flag, writeout_error = 0;
> int do_merge = 1;
> + struct strbuf old_commit_shortname = STRBUF_INIT;
> + const char *stash_label_ancestor = NULL;
>
> trace2_cmd_mode("branch");
>
> @@ -1202,10 +1175,34 @@ static int switch_branches(const struct checkout_opts *opts,
> do_merge = 0;
> }
>
> + if (old_branch_info.name)
> + stash_label_ancestor = old_branch_info.name;
> + else if (old_branch_info.commit) {
> + strbuf_add_unique_abbrev(&old_commit_shortname,
> + &old_branch_info.commit->object.oid,
> + DEFAULT_ABBREV);
> + stash_label_ancestor = old_commit_shortname.buf;
> + }
> +
> + if (opts->merge) {
> + if (repo_read_index(the_repository) < 0)
> + die(_("index file corrupt"));
> + if (checkout_would_clobber_changes(&old_branch_info,
> + new_branch_info))
> + create_autostash_ref_silent(the_repository,
> + "CHECKOUT_AUTOSTASH");
> + }
> +
> if (do_merge) {
> ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
> if (ret) {
> + apply_autostash_ref_with_labels(the_repository,
> + "CHECKOUT_AUTOSTASH",
> + new_branch_info->name,
> + "local",
> + stash_label_ancestor);
> branch_info_release(&old_branch_info);
> + strbuf_release(&old_commit_shortname);
> return ret;
> }
> }
Nice changes to preserve the conflict marker labels.
> diff --git a/builtin/stash.c b/builtin/stash.c
> index e79d612e57..1016d88e52 100644
> --- a/builtin/stash.c
> +++ b/builtin/stash.c
This can and should become a separate patch that is a preliminary
preparation step, [1/2], with its own documentation and test, only
about teaching "git stash" command a set of new options to use
custom conflict labels, I think. And on top of it, patch [2/2] that
updates "checkout -m" to use the improved "git stash" with custom
labels support is written. It would give us a clean logical
progression of the series.
> diff --git a/sequencer.c b/sequencer.c
> index aafd0bc959..53e04d8a94 100644
> --- a/sequencer.c
> +++ b/sequencer.c
> @@ -4632,7 +4632,8 @@ static enum todo_command peek_command(struct todo_list *todo_list, int offset)
>
> static void create_autostash_internal(struct repository *r,
> const char *path,
> - const char *refname)
> + const char *refname,
> + int silent)
> {
> struct strbuf buf = STRBUF_INIT;
> struct lock_file lock_file = LOCK_INIT;
> @@ -4677,7 +4678,8 @@ static void create_autostash_internal(struct repository *r,
> &oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
> }
>
> - printf(_("Created autostash: %s\n"), buf.buf);
> + if (!silent)
> + fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
> if (reset_head(r, &ropts) < 0)
> die(_("could not reset --hard"));
> discard_index(r->index);
> @@ -4689,15 +4691,22 @@ static void create_autostash_internal(struct repository *r,
>
> void create_autostash(struct repository *r, const char *path)
> {
> - create_autostash_internal(r, path, NULL);
> + create_autostash_internal(r, path, NULL, 0);
> }
>
> void create_autostash_ref(struct repository *r, const char *refname)
> {
> - create_autostash_internal(r, NULL, refname);
> + create_autostash_internal(r, NULL, refname, 0);
> }
>
> -static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
> +void create_autostash_ref_silent(struct repository *r, const char *refname)
> +{
> + create_autostash_internal(r, NULL, refname, 1);
> +}
Changes around here to optionally squelch message(s) is better
presented as yet another preliminary preparation step, I think.
> @@ -4752,7 +4775,8 @@ static int apply_save_autostash(const char *path, int attempt_apply)
> }
> strbuf_trim(&stash_oid);
>
> - ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
> + ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
> + NULL, NULL, NULL);
>
> unlink(path);
> strbuf_release(&stash_oid);
> @@ -4771,11 +4795,13 @@ int apply_autostash(const char *path)
>
> int apply_autostash_oid(const char *stash_oid)
> {
> - return apply_save_autostash_oid(stash_oid, 1);
> + return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
> }
>
> static int apply_save_autostash_ref(struct repository *r, const char *refname,
> - int attempt_apply)
> + int attempt_apply,
> + const char *label1, const char *label2,
> + const char *label_ancestor)
> {
> struct object_id stash_oid;
> char stash_oid_hex[GIT_MAX_HEXSZ + 1];
> @@ -4791,7 +4817,8 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
> return error(_("autostash reference is a symref"));
>
> oid_to_hex_r(stash_oid_hex, &stash_oid);
> - ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
> + ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
> + label1, label2, label_ancestor);
>
> refs_delete_ref(get_main_ref_store(r), "", refname,
> &stash_oid, REF_NO_DEREF);
>
> @@ -4801,12 +4828,20 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
>
> int save_autostash_ref(struct repository *r, const char *refname)
> {
> - return apply_save_autostash_ref(r, refname, 0);
> + return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
> }
>
> int apply_autostash_ref(struct repository *r, const char *refname)
> {
> - return apply_save_autostash_ref(r, refname, 1);
> + return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
> +}
> +
> +int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
> + const char *label1, const char *label2,
> + const char *label_ancestor)
> +{
> + return apply_save_autostash_ref(r, refname, 1,
> + label1, label2, label_ancestor);
> }
>
> static int checkout_onto(struct repository *r, struct replay_opts *opts,
So are these, to teach the machinery to take optional conflict
marker labels.
Thanks.
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v5 0/4] checkout: 'autostash' for branch switching
2026-03-14 9:59 ` [PATCH v4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
2026-03-15 2:25 ` Junio C Hamano
@ 2026-03-15 11:19 ` Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
` (4 more replies)
1 sibling, 5 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-15 11:19 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren
cc: Phillip Wood phillip.wood123@gmail.com
Harald Nordgren (4):
stash: add --ours-label, --theirs-label, --base-label for apply
sequencer: allow create_autostash to run silently
sequencer: teach autostash apply to take optional conflict marker
labels
checkout: -m (--merge) uses autostash when switching branches
Documentation/git-checkout.adoc | 58 +++++-----
Documentation/git-stash.adoc | 11 +-
Documentation/git-switch.adoc | 27 ++---
builtin/checkout.c | 180 ++++++++++++++++++--------------
builtin/stash.c | 32 ++++--
sequencer.c | 67 +++++++++---
sequencer.h | 4 +
t/t3420-rebase-autostash.sh | 24 +++--
t/t3903-stash.sh | 18 ++++
t/t7201-co.sh | 160 ++++++++++++++++++++++++++++
t/t7600-merge.sh | 2 +-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
13 files changed, 447 insertions(+), 149 deletions(-)
base-commit: dc6ecd5354dca88d51b6d6562777fc8fc10d77e1
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v5
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v5
Pull-Request: https://github.com/git/git/pull/2234
Range-diff vs v4:
-: ---------- > 1: 00d8920498 stash: add --ours-label, --theirs-label, --base-label for apply
-: ---------- > 2: 5d176f1700 sequencer: allow create_autostash to run silently
-: ---------- > 3: 3d6829438a sequencer: teach autostash apply to take optional conflict marker labels
1: 5d49c0031a ! 4: 7f3735d40e checkout: -m (--merge) uses autostash when switching branches
@@ Documentation/git-checkout.adoc: working tree, by copying them from elsewhere, e
+ afterwards. If the local changes do not overlap with the
+ differences between branches, the switch proceeds without
+ stashing. If reapplying the stash results in conflicts, the
-+ entry is saved to the stash list so you can use `git stash
-+ pop` to recover and `git stash drop` when done.
++ entry is saved to the stash list. Resolve the conflicts
++ and run `git stash drop` when done, or clear the working
++ tree (e.g. with `git reset --hard`) before running `git stash
++ pop` later to re-apply your changes.
+
When checking out paths from the index, this option lets you recreate
the conflicted merge in the specified paths. This option cannot be
@@ Documentation/git-checkout.adoc: $ git checkout mytopic
-You can give the `-m` flag to the command, which would try a
-three-way merge:
-+You can give the `-m` flag to the command, which would save the local
-+changes in a stash entry and reset the working tree to allow switching:
++You can give the `-m` flag to the command, which would carry your local
++changes to the new branch:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-+Applied autostash.
++Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
@@ Documentation/git-switch.adoc: variable.
+ local changes do not overlap with the differences between
+ branches, the switch proceeds without stashing. If
+ reapplying the stash results in conflicts, the entry is
-+ saved to the stash list so you can use `git stash pop` to
-+ recover and `git stash drop` when done.
++ saved to the stash list. Resolve the conflicts and run
++ `git stash drop` when done, or clear the working tree
++ (e.g. with `git reset --hard`) before running `git stash pop`
++ later to re-apply your changes.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ Documentation/git-switch.adoc: $ git switch mytopic
-You can give the `-m` flag to the command, which would try a three-way
-merge:
-+You can give the `-m` flag to the command, which would save the local
-+changes in a stash entry and reset the working tree to allow switching:
++You can give the `-m` flag to the command, which would carry your local
++changes to the new branch:
------------
$ git switch -m mytopic
-Auto-merging frotz
-+Created autostash: 7a9afa3
-+Applied autostash.
++Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *op
}
if (!cache_tree_fully_valid(the_repository->index->cache_tree))
-@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
- if (write_locked_index(the_repository->index, &lock_file, COMMIT_LOCK))
- die(_("unable to write new index file"));
-
-- if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
-- show_local_changes(&new_branch_info->commit->object, &opts->diff_options);
--
- return 0;
- }
-
@@ builtin/checkout.c: static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
struct object_id rev;
int flag, writeout_error = 0;
int do_merge = 1;
++ int created_autostash = 0;
+ struct strbuf old_commit_shortname = STRBUF_INIT;
+ const char *stash_label_ancestor = NULL;
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (checkout_would_clobber_changes(&old_branch_info,
-+ new_branch_info))
++ new_branch_info)) {
+ create_autostash_ref_silent(the_repository,
+ "CHECKOUT_AUTOSTASH");
++ created_autostash = 1;
++ }
+ }
+
if (do_merge) {
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+
-+ if (!opts->discard_changes && !opts->quiet && new_branch_info->commit)
++ if (created_autostash && !opts->discard_changes && !opts->quiet &&
++ new_branch_info->commit)
+ show_local_changes(&new_branch_info->commit->object,
+ &opts->diff_options);
+
@@ builtin/stash.c: static int apply_stash(int argc, const char **argv, const char
return ret;
## sequencer.c ##
-@@ sequencer.c: static enum todo_command peek_command(struct todo_list *todo_list, int offset)
-
- static void create_autostash_internal(struct repository *r,
- const char *path,
-- const char *refname)
-+ const char *refname,
-+ int silent)
- {
- struct strbuf buf = STRBUF_INIT;
- struct lock_file lock_file = LOCK_INIT;
-@@ sequencer.c: static void create_autostash_internal(struct repository *r,
- &oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
- }
-
-- printf(_("Created autostash: %s\n"), buf.buf);
-+ if (!silent)
-+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
- if (reset_head(r, &ropts) < 0)
- die(_("could not reset --hard"));
- discard_index(r->index);
-@@ sequencer.c: static void create_autostash_internal(struct repository *r,
-
- void create_autostash(struct repository *r, const char *path)
- {
-- create_autostash_internal(r, path, NULL);
-+ create_autostash_internal(r, path, NULL, 0);
- }
-
- void create_autostash_ref(struct repository *r, const char *refname)
- {
-- create_autostash_internal(r, NULL, refname);
-+ create_autostash_internal(r, NULL, refname, 0);
- }
-
--static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
-+void create_autostash_ref_silent(struct repository *r, const char *refname)
-+{
-+ create_autostash_internal(r, NULL, refname, 1);
-+}
-+
-+static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
-+ const char *label1, const char *label2,
-+ const char *label_ancestor)
- {
- struct child_process child = CHILD_PROCESS_INIT;
- int ret = 0;
-@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
- child.no_stderr = 1;
- strvec_push(&child.args, "stash");
- strvec_push(&child.args, "apply");
-+ if (label1)
-+ strvec_pushf(&child.args, "--ours-label=%s", label1);
-+ if (label2)
-+ strvec_pushf(&child.args, "--theirs-label=%s", label2);
-+ if (label_ancestor)
-+ strvec_pushf(&child.args, "--base-label=%s", label_ancestor);
- strvec_push(&child.args, stash_oid);
- ret = run_command(&child);
- }
-@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
strvec_push(&store.args, stash_oid);
if (run_command(&store))
ret = error(_("cannot store %s"), stash_oid);
@@ sequencer.c: static int apply_save_autostash_oid(const char *stash_oid, int atte
}
return ret;
-@@ sequencer.c: static int apply_save_autostash(const char *path, int attempt_apply)
- }
- strbuf_trim(&stash_oid);
-
-- ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
-+ ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
-+ NULL, NULL, NULL);
-
- unlink(path);
- strbuf_release(&stash_oid);
-@@ sequencer.c: int apply_autostash(const char *path)
-
- int apply_autostash_oid(const char *stash_oid)
- {
-- return apply_save_autostash_oid(stash_oid, 1);
-+ return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
- }
-
- static int apply_save_autostash_ref(struct repository *r, const char *refname,
-- int attempt_apply)
-+ int attempt_apply,
-+ const char *label1, const char *label2,
-+ const char *label_ancestor)
- {
- struct object_id stash_oid;
- char stash_oid_hex[GIT_MAX_HEXSZ + 1];
-@@ sequencer.c: static int apply_save_autostash_ref(struct repository *r, const char *refname,
- return error(_("autostash reference is a symref"));
-
- oid_to_hex_r(stash_oid_hex, &stash_oid);
-- ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
-+ ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
-+ label1, label2, label_ancestor);
-
- refs_delete_ref(get_main_ref_store(r), "", refname,
- &stash_oid, REF_NO_DEREF);
-@@ sequencer.c: static int apply_save_autostash_ref(struct repository *r, const char *refname,
-
- int save_autostash_ref(struct repository *r, const char *refname)
- {
-- return apply_save_autostash_ref(r, refname, 0);
-+ return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
- }
-
- int apply_autostash_ref(struct repository *r, const char *refname)
- {
-- return apply_save_autostash_ref(r, refname, 1);
-+ return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
-+}
-+
-+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
-+ const char *label1, const char *label2,
-+ const char *label_ancestor)
-+{
-+ return apply_save_autostash_ref(r, refname, 1,
-+ label1, label2, label_ancestor);
- }
-
- static int checkout_onto(struct repository *r, struct replay_opts *opts,
-
- ## sequencer.h ##
-@@ sequencer.h: void commit_post_rewrite(struct repository *r,
-
- void create_autostash(struct repository *r, const char *path);
- void create_autostash_ref(struct repository *r, const char *refname);
-+void create_autostash_ref_silent(struct repository *r, const char *refname);
- int save_autostash(const char *path);
- int save_autostash_ref(struct repository *r, const char *refname);
- int apply_autostash(const char *path);
- int apply_autostash_oid(const char *stash_oid);
- int apply_autostash_ref(struct repository *r, const char *refname);
-+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
-+ const char *label1, const char *label2,
-+ const char *label_ancestor);
-
- #define SUMMARY_INITIAL_COMMIT (1 << 0)
- #define SUMMARY_SHOW_AUTHOR_DATE (1 << 1)
## t/t3420-rebase-autostash.sh ##
@@ t/t3420-rebase-autostash.sh: create_expected_failure_apply () {
@@ t/t7201-co.sh: test_expect_success 'checkout --merge --conflict=diff3 <branch>'
git reset --hard &&
## t/t7600-merge.sh ##
-@@ t/t7600-merge.sh: test_expect_success 'merge --squash --autostash conflict does not attempt to app
- >unrelated &&
- git add unrelated &&
- test_must_fail git merge --squash c7 --autostash >out 2>err &&
-- ! grep "Applying autostash resulted in conflicts." err &&
-+ ! grep "resulted in conflicts" err &&
- grep "When finished, apply stashed changes with \`git stash pop\`" out
- '
-
@@ t/t7600-merge.sh: test_expect_success 'merge with conflicted --autostash changes' '
git diff >expect &&
test_when_finished "test_might_fail git stash drop" &&
git merge --autostash c3 2>err &&
- test_grep "Applying autostash resulted in conflicts." err &&
-+ test_grep "resulted in conflicts" err &&
++ test_grep "your local changes resulted in conflicts" err &&
git show HEAD:file >merge-result &&
test_cmp result.1-9 merge-result &&
git stash show -p >actual &&
--
gitgitgadget
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v5 1/4] stash: add --ours-label, --theirs-label, --base-label for apply
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
@ 2026-03-15 11:19 ` Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
` (3 subsequent siblings)
4 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-15 11:19 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Allow callers of "git stash apply" to pass custom labels for conflict
markers instead of the default "Updated upstream" and "Stashed changes".
Document the new options and add a test.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
Documentation/git-stash.adoc | 11 ++++++++++-
builtin/stash.c | 2 +-
t/t3903-stash.sh | 18 ++++++++++++++++++
3 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-stash.adoc b/Documentation/git-stash.adoc
index 235d57ddd8..43650a3c1a 100644
--- a/Documentation/git-stash.adoc
+++ b/Documentation/git-stash.adoc
@@ -12,7 +12,7 @@ git stash list [<log-options>]
git stash show [-u | --include-untracked | --only-untracked] [<diff-options>] [<stash>]
git stash drop [-q | --quiet] [<stash>]
git stash pop [--index] [-q | --quiet] [<stash>]
-git stash apply [--index] [-q | --quiet] [<stash>]
+git stash apply [--index] [-q | --quiet] [--ours-label=<label>] [--theirs-label=<label>] [--base-label=<label>] [<stash>]
git stash branch <branchname> [<stash>]
git stash [push [-p | --patch] [-S | --staged] [-k | --[no-]keep-index] [-q | --quiet]
[-u | --include-untracked] [-a | --all] [(-m | --message) <message>]
@@ -197,6 +197,15 @@ the index's ones. However, this can fail, when you have conflicts
(which are stored in the index, where you therefore can no longer
apply the changes as they were originally).
+`--ours-label=<label>`::
+`--theirs-label=<label>`::
+`--base-label=<label>`::
+ These options are only valid for the `apply` command.
++
+Use the given labels in conflict markers instead of the default
+"Updated upstream", "Stashed changes", and "Stash base".
+`--base-label` only has an effect with merge.conflictStyle=diff3.
+
`-k`::
`--keep-index`::
`--no-keep-index`::
diff --git a/builtin/stash.c b/builtin/stash.c
index e79d612e57..252e4df3a9 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -44,7 +44,7 @@
#define BUILTIN_STASH_POP_USAGE \
N_("git stash pop [--index] [-q | --quiet] [<stash>]")
#define BUILTIN_STASH_APPLY_USAGE \
- N_("git stash apply [--index] [-q | --quiet] [<stash>]")
+ N_("git stash apply [--index] [-q | --quiet] [--ours-label=<label>] [--theirs-label=<label>] [--base-label=<label>] [<stash>]")
#define BUILTIN_STASH_BRANCH_USAGE \
N_("git stash branch <branchname> [<stash>]")
#define BUILTIN_STASH_STORE_USAGE \
diff --git a/t/t3903-stash.sh b/t/t3903-stash.sh
index 70879941c2..dd47c1322a 100755
--- a/t/t3903-stash.sh
+++ b/t/t3903-stash.sh
@@ -1666,6 +1666,24 @@ test_expect_success 'restore untracked files even when we hit conflicts' '
)
'
+test_expect_success 'apply with custom conflict labels' '
+ git init conflict_labels &&
+ (
+ cd conflict_labels &&
+ echo base >file &&
+ git add file &&
+ git commit -m base &&
+ echo stashed >file &&
+ git stash push -m "stashed" &&
+ echo upstream >file &&
+ git add file &&
+ git commit -m upstream &&
+ test_must_fail git stash apply --ours-label=UP --theirs-label=STASH &&
+ grep "^<<<<<<< UP" file &&
+ grep "^>>>>>>> STASH" file
+ )
+'
+
test_expect_success 'stash create reports a locked index' '
test_when_finished "rm -rf repo" &&
git init repo &&
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v5 2/4] sequencer: allow create_autostash to run silently
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
@ 2026-03-15 11:19 ` Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
` (2 subsequent siblings)
4 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-15 11:19 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Add a silent parameter to create_autostash_internal and introduce
create_autostash_ref_silent so that callers can create an autostash
without printing the "Created autostash" message. Use stderr for
the message when not silent.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
sequencer.c | 15 +++++++++++----
sequencer.h | 1 +
2 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index aafd0bc959..eebefd731b 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4632,7 +4632,8 @@ static enum todo_command peek_command(struct todo_list *todo_list, int offset)
static void create_autostash_internal(struct repository *r,
const char *path,
- const char *refname)
+ const char *refname,
+ int silent)
{
struct strbuf buf = STRBUF_INIT;
struct lock_file lock_file = LOCK_INIT;
@@ -4677,7 +4678,8 @@ static void create_autostash_internal(struct repository *r,
&oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
}
- printf(_("Created autostash: %s\n"), buf.buf);
+ if (!silent)
+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
if (reset_head(r, &ropts) < 0)
die(_("could not reset --hard"));
discard_index(r->index);
@@ -4689,12 +4691,17 @@ static void create_autostash_internal(struct repository *r,
void create_autostash(struct repository *r, const char *path)
{
- create_autostash_internal(r, path, NULL);
+ create_autostash_internal(r, path, NULL, 0);
}
void create_autostash_ref(struct repository *r, const char *refname)
{
- create_autostash_internal(r, NULL, refname);
+ create_autostash_internal(r, NULL, refname, 0);
+}
+
+void create_autostash_ref_silent(struct repository *r, const char *refname)
+{
+ create_autostash_internal(r, NULL, refname, 1);
}
static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
diff --git a/sequencer.h b/sequencer.h
index 719684c8a9..0b09d6799b 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -227,6 +227,7 @@ void commit_post_rewrite(struct repository *r,
void create_autostash(struct repository *r, const char *path);
void create_autostash_ref(struct repository *r, const char *refname);
+void create_autostash_ref_silent(struct repository *r, const char *refname);
int save_autostash(const char *path);
int save_autostash_ref(struct repository *r, const char *refname);
int apply_autostash(const char *path);
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v5 3/4] sequencer: teach autostash apply to take optional conflict marker labels
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
@ 2026-03-15 11:19 ` Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
4 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-15 11:19 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Add label1, label2, and label_ancestor parameters to the autostash
apply machinery so callers can pass custom conflict marker labels
through to "git stash apply --ours-label/--theirs-label/--base-label".
Introduce apply_autostash_ref_with_labels() for callers that want
to pass labels.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
sequencer.c | 34 +++++++++++++++++++++++++++-------
sequencer.h | 3 +++
2 files changed, 30 insertions(+), 7 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index eebefd731b..080a25820a 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4704,7 +4704,9 @@ void create_autostash_ref_silent(struct repository *r, const char *refname)
create_autostash_internal(r, NULL, refname, 1);
}
-static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct child_process child = CHILD_PROCESS_INIT;
int ret = 0;
@@ -4715,6 +4717,12 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
child.no_stderr = 1;
strvec_push(&child.args, "stash");
strvec_push(&child.args, "apply");
+ if (label1)
+ strvec_pushf(&child.args, "--ours-label=%s", label1);
+ if (label2)
+ strvec_pushf(&child.args, "--theirs-label=%s", label2);
+ if (label_ancestor)
+ strvec_pushf(&child.args, "--base-label=%s", label_ancestor);
strvec_push(&child.args, stash_oid);
ret = run_command(&child);
}
@@ -4759,7 +4767,8 @@ static int apply_save_autostash(const char *path, int attempt_apply)
}
strbuf_trim(&stash_oid);
- ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
+ NULL, NULL, NULL);
unlink(path);
strbuf_release(&stash_oid);
@@ -4778,11 +4787,13 @@ int apply_autostash(const char *path)
int apply_autostash_oid(const char *stash_oid)
{
- return apply_save_autostash_oid(stash_oid, 1);
+ return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
}
static int apply_save_autostash_ref(struct repository *r, const char *refname,
- int attempt_apply)
+ int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct object_id stash_oid;
char stash_oid_hex[GIT_MAX_HEXSZ + 1];
@@ -4798,7 +4809,8 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
return error(_("autostash reference is a symref"));
oid_to_hex_r(stash_oid_hex, &stash_oid);
- ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
+ label1, label2, label_ancestor);
refs_delete_ref(get_main_ref_store(r), "", refname,
&stash_oid, REF_NO_DEREF);
@@ -4808,12 +4820,20 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
int save_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 0);
+ return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
}
int apply_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 1);
+ return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
+}
+
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
+{
+ return apply_save_autostash_ref(r, refname, 1,
+ label1, label2, label_ancestor);
}
static int checkout_onto(struct repository *r, struct replay_opts *opts,
diff --git a/sequencer.h b/sequencer.h
index 0b09d6799b..68b94d86e3 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -233,6 +233,9 @@ int save_autostash_ref(struct repository *r, const char *refname);
int apply_autostash(const char *path);
int apply_autostash_oid(const char *stash_oid);
int apply_autostash_ref(struct repository *r, const char *refname);
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor);
#define SUMMARY_INITIAL_COMMIT (1 << 0)
#define SUMMARY_SHOW_AUTHOR_DATE (1 << 1)
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v5 4/4] checkout: -m (--merge) uses autostash when switching branches
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
` (2 preceding siblings ...)
2026-03-15 11:19 ` [PATCH v5 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
@ 2026-03-15 11:19 ` Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
4 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-15 11:19 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches with "git checkout -m", local modifications
can block the switch. Teach the -m flow to create a temporary stash
before switching and reapply it after. On success, only "Applied
autostash." is shown. If reapplying causes conflicts, the stash is
kept and the user is told they can resolve and run "git stash drop",
or run "git reset --hard" and later "git stash pop" to recover their
changes.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
Documentation/git-checkout.adoc | 58 +++++-----
Documentation/git-switch.adoc | 27 ++---
builtin/checkout.c | 180 ++++++++++++++++++--------------
builtin/stash.c | 30 ++++--
sequencer.c | 18 +++-
t/t3420-rebase-autostash.sh | 24 +++--
t/t7201-co.sh | 160 ++++++++++++++++++++++++++++
t/t7600-merge.sh | 2 +-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
10 files changed, 376 insertions(+), 136 deletions(-)
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..18022e610e 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -251,20 +251,19 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
are different between the current branch and the branch to
which you are switching, the command refuses to switch
branches in order to preserve your modifications in context.
- However, with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch
- is done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ With this option, the conflicting local changes are
+ automatically stashed before the switch and reapplied
+ afterwards. If the local changes do not overlap with the
+ differences between branches, the switch proceeds without
+ stashing. If reapplying the stash results in conflicts, the
+ entry is saved to the stash list. Resolve the conflicts
+ and run `git stash drop` when done, or clear the working
+ tree (e.g. with `git reset --hard`) before running `git stash
+ pop` later to re-apply your changes.
+
When checking out paths from the index, this option lets you recreate
the conflicted merge in the specified paths. This option cannot be
used when checking out paths from a tree-ish.
-+
-When switching branches with `--merge`, staged changes may be lost.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -578,39 +577,44 @@ $ git checkout mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a
-three-way merge:
+You can give the `-m` flag to the command, which would carry your local
+changes to the new branch:
------------
$ git checkout -m mytopic
-Auto-merging frotz
+Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
=== 3. Merge conflict
-When a merge conflict happens during switching branches with
-the `-m` option, you would see something like this:
+When the `--merge` (`-m`) option is in effect and the locally
+modified files overlap with files that need to be updated by the
+branch switch, the changes are stashed and reapplied after the
+switch. If the stash application results in conflicts, they are not
+resolved and the stash is saved to the stash list:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-ERROR: Merge conflict in frotz
-fatal: merge program failed
-------------
+Your local changes are stashed, however, applying it to carry
+forward your local changes resulted in conflicts:
-At this point, `git diff` shows the changes cleanly merged as in
-the previous example, as well as the changes in the conflicted
-files. Edit and resolve the conflict and mark it resolved with
-`git add` as usual:
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
------------
-$ edit frotz
-$ git add frotz
-------------
+
+You can try resolving the conflicts now. Edit the conflicting files
+and mark them resolved with `git add` as usual, then run `git stash
+drop` to discard the stash entry. Alternatively, you can clear the
+working tree with `git reset --hard` and recover your local changes
+later with `git stash pop`.
CONFIGURATION
-------------
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..3147023dc3 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -126,15 +126,16 @@ variable.
If you have local modifications to one or more files that are
different between the current branch and the branch to which
you are switching, the command refuses to switch branches in
- order to preserve your modifications in context. However,
- with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch is
- done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ order to preserve your modifications in context. With this
+ option, the conflicting local changes are automatically
+ stashed before the switch and reapplied afterwards. If the
+ local changes do not overlap with the differences between
+ branches, the switch proceeds without stashing. If
+ reapplying the stash results in conflicts, the entry is
+ saved to the stash list. Resolve the conflicts and run
+ `git stash drop` when done, or clear the working tree
+ (e.g. with `git reset --hard`) before running `git stash pop`
+ later to re-apply your changes.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -217,15 +218,15 @@ $ git switch mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a three-way
-merge:
+You can give the `-m` flag to the command, which would carry your local
+changes to the new branch:
------------
$ git switch -m mytopic
-Auto-merging frotz
+Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..db93eabd9b 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -17,7 +17,6 @@
#include "merge-ll.h"
#include "lockfile.h"
#include "mem-pool.h"
-#include "merge-ort-wrappers.h"
#include "object-file.h"
#include "object-name.h"
#include "odb.h"
@@ -30,6 +29,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -845,83 +845,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
ret = unpack_trees(2, trees, &topts);
clear_unpack_trees_porcelain(&topts);
- if (ret == -1) {
- /*
- * Unpack couldn't do a trivial merge; either
- * give up or do a real merge, depending on
- * whether the merge flag was used.
- */
- struct tree *work;
- struct tree *old_tree;
- struct merge_options o;
- struct strbuf sb = STRBUF_INIT;
- struct strbuf old_commit_shortname = STRBUF_INIT;
-
- if (!opts->merge)
- return 1;
-
- /*
- * Without old_branch_info->commit, the below is the same as
- * the two-tree unpack we already tried and failed.
- */
- if (!old_branch_info->commit)
- return 1;
- old_tree = repo_get_commit_tree(the_repository,
- old_branch_info->commit);
-
- if (repo_index_has_changes(the_repository, old_tree, &sb))
- die(_("cannot continue with staged changes in "
- "the following files:\n%s"), sb.buf);
- strbuf_release(&sb);
-
- /* Do more real merge */
-
- /*
- * We update the index fully, then write the
- * tree from the index, then merge the new
- * branch with the current tree, with the old
- * branch as the base. Then we reset the index
- * (but not the working tree) to the new
- * branch, leaving the working tree as the
- * merged version, but skipping unmerged
- * entries in the index.
- */
-
- add_files_to_cache(the_repository, NULL, NULL, NULL, 0,
- 0, 0);
- init_ui_merge_options(&o, the_repository);
- o.verbosity = 0;
- work = write_in_core_index_as_tree(the_repository);
-
- ret = reset_tree(new_tree,
- opts, 1,
- writeout_error, new_branch_info);
- if (ret)
- return ret;
- o.ancestor = old_branch_info->name;
- if (!old_branch_info->name) {
- strbuf_add_unique_abbrev(&old_commit_shortname,
- &old_branch_info->commit->object.oid,
- DEFAULT_ABBREV);
- o.ancestor = old_commit_shortname.buf;
- }
- o.branch1 = new_branch_info->name;
- o.branch2 = "local";
- o.conflict_style = opts->conflict_style;
- ret = merge_ort_nonrecursive(&o,
- new_tree,
- work,
- old_tree);
- if (ret < 0)
- die(NULL);
- ret = reset_tree(new_tree,
- opts, 0,
- writeout_error, new_branch_info);
- strbuf_release(&o.obuf);
- strbuf_release(&old_commit_shortname);
- if (ret)
- return ret;
- }
+ if (ret == -1)
+ return 1;
}
if (!cache_tree_fully_valid(the_repository->index->cache_tree))
@@ -1157,6 +1082,55 @@ static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
release_revisions(&revs);
}
+static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
+ struct branch_info *new_branch_info)
+{
+ struct tree_desc trees[2];
+ struct tree *old_tree, *new_tree;
+ struct unpack_trees_options topts;
+ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
+ const struct object_id *old_commit_oid;
+ int ret;
+
+ if (!new_branch_info->commit)
+ return 0;
+
+ old_commit_oid = old_branch_info->commit ?
+ &old_branch_info->commit->object.oid :
+ the_hash_algo->empty_tree;
+ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
+ if (!old_tree)
+ return 0;
+
+ new_tree = repo_get_commit_tree(the_repository,
+ new_branch_info->commit);
+ if (!new_tree)
+ return 0;
+ if (repo_parse_tree(the_repository, new_tree) < 0)
+ return 0;
+
+ memset(&topts, 0, sizeof(topts));
+ topts.head_idx = -1;
+ topts.src_index = the_repository->index;
+ topts.dst_index = &tmp_index;
+ topts.initial_checkout = is_index_unborn(the_repository->index);
+ topts.merge = 1;
+ topts.update = 1;
+ topts.dry_run = 1;
+ topts.quiet = 1;
+ topts.fn = twoway_merge;
+
+ init_tree_desc(&trees[0], &old_tree->object.oid,
+ old_tree->buffer, old_tree->size);
+ init_tree_desc(&trees[1], &new_tree->object.oid,
+ new_tree->buffer, new_tree->size);
+
+ ret = unpack_trees(2, trees, &topts);
+ discard_index(&tmp_index);
+
+ return ret != 0;
+}
+
static int switch_branches(const struct checkout_opts *opts,
struct branch_info *new_branch_info)
{
@@ -1165,6 +1139,9 @@ static int switch_branches(const struct checkout_opts *opts,
struct object_id rev;
int flag, writeout_error = 0;
int do_merge = 1;
+ int created_autostash = 0;
+ struct strbuf old_commit_shortname = STRBUF_INIT;
+ const char *stash_label_ancestor = NULL;
trace2_cmd_mode("branch");
@@ -1202,10 +1179,36 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (old_branch_info.name)
+ stash_label_ancestor = old_branch_info.name;
+ else if (old_branch_info.commit) {
+ strbuf_add_unique_abbrev(&old_commit_shortname,
+ &old_branch_info.commit->object.oid,
+ DEFAULT_ABBREV);
+ stash_label_ancestor = old_commit_shortname.buf;
+ }
+
+ if (opts->merge) {
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+ if (checkout_would_clobber_changes(&old_branch_info,
+ new_branch_info)) {
+ create_autostash_ref_silent(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ created_autostash = 1;
+ }
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref_with_labels(the_repository,
+ "CHECKOUT_AUTOSTASH",
+ new_branch_info->name,
+ "local",
+ stash_label_ancestor);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret;
}
}
@@ -1215,8 +1218,29 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ if (opts->conflict_style >= 0) {
+ struct strbuf cfg = STRBUF_INIT;
+ strbuf_addf(&cfg, "merge.conflictStyle=%s",
+ conflict_style_name(opts->conflict_style));
+ git_config_push_parameter(cfg.buf);
+ strbuf_release(&cfg);
+ }
+ apply_autostash_ref_with_labels(the_repository, "CHECKOUT_AUTOSTASH",
+ new_branch_info->name, "local",
+ stash_label_ancestor);
+
+ discard_index(the_repository->index);
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+
+ if (created_autostash && !opts->discard_changes && !opts->quiet &&
+ new_branch_info->commit)
+ show_local_changes(&new_branch_info->commit->object,
+ &opts->diff_options);
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret || writeout_error;
}
diff --git a/builtin/stash.c b/builtin/stash.c
index 252e4df3a9..ab9c1359b1 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -590,8 +590,11 @@ static void unstage_changes_unless_new(struct object_id *orig_tree)
die(_("could not write index"));
}
-static int do_apply_stash(const char *prefix, struct stash_info *info,
- int index, int quiet)
+static int do_apply_stash_with_labels(const char *prefix,
+ struct stash_info *info,
+ int index, int quiet,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
int clean, ret;
int has_index = index;
@@ -643,9 +646,9 @@ static int do_apply_stash(const char *prefix, struct stash_info *info,
init_ui_merge_options(&o, the_repository);
- o.branch1 = "Updated upstream";
- o.branch2 = "Stashed changes";
- o.ancestor = "Stash base";
+ o.branch1 = label1 ? label1 : "Updated upstream";
+ o.branch2 = label2 ? label2 : "Stashed changes";
+ o.ancestor = label_ancestor ? label_ancestor : "Stash base";
if (oideq(&info->b_tree, &c_tree))
o.branch1 = "Version stash was based on";
@@ -717,17 +720,31 @@ restore_untracked:
return ret;
}
+static int do_apply_stash(const char *prefix, struct stash_info *info,
+ int index, int quiet)
+{
+ return do_apply_stash_with_labels(prefix, info, index, quiet,
+ NULL, NULL, NULL);
+}
+
static int apply_stash(int argc, const char **argv, const char *prefix,
struct repository *repo UNUSED)
{
int ret = -1;
int quiet = 0;
int index = use_index;
+ const char *label1 = NULL, *label2 = NULL, *label_ancestor = NULL;
struct stash_info info = STASH_INFO_INIT;
struct option options[] = {
OPT__QUIET(&quiet, N_("be quiet, only report errors")),
OPT_BOOL(0, "index", &index,
N_("attempt to recreate the index")),
+ OPT_STRING(0, "ours-label", &label1, N_("label"),
+ N_("label for the upstream side in conflict markers")),
+ OPT_STRING(0, "theirs-label", &label2, N_("label"),
+ N_("label for the stashed side in conflict markers")),
+ OPT_STRING(0, "base-label", &label_ancestor, N_("label"),
+ N_("label for the base in diff3 conflict markers")),
OPT_END()
};
@@ -737,7 +754,8 @@ static int apply_stash(int argc, const char **argv, const char *prefix,
if (get_stash_info(&info, argc, argv))
goto cleanup;
- ret = do_apply_stash(prefix, &info, index, quiet);
+ ret = do_apply_stash_with_labels(prefix, &info, index, quiet,
+ label1, label2, label_ancestor);
cleanup:
free_stash_info(&info);
return ret;
diff --git a/sequencer.c b/sequencer.c
index 080a25820a..53e04d8a94 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4741,15 +4741,23 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
strvec_push(&store.args, stash_oid);
if (run_command(&store))
ret = error(_("cannot store %s"), stash_oid);
+ else if (attempt_apply)
+ fprintf(stderr,
+ _("Your local changes are stashed, however, applying it to carry\n"
+ "forward your local changes resulted in conflicts:\n"
+ "\n"
+ " - You can try resolving them now. If you resolved them\n"
+ " successfully, discard the stash entry with \"git stash drop\".\n"
+ "\n"
+ " - Alternatively you can \"git reset --hard\" if you do not want\n"
+ " to deal with them right now, and later \"git stash pop\" to\n"
+ " recover your local changes.\n"));
else
fprintf(stderr,
- _("%s\n"
+ _("Autostash exists; creating a new stash entry.\n"
"Your changes are safe in the stash.\n"
"You can run \"git stash pop\" or"
- " \"git stash drop\" at any time.\n"),
- attempt_apply ?
- _("Applying autostash resulted in conflicts.") :
- _("Autostash exists; creating a new stash entry."));
+ " \"git stash drop\" at any time.\n"));
}
return ret;
diff --git a/t/t3420-rebase-autostash.sh b/t/t3420-rebase-autostash.sh
index ad3ba6a984..e4e2cb19ce 100755
--- a/t/t3420-rebase-autostash.sh
+++ b/t/t3420-rebase-autostash.sh
@@ -61,18 +61,30 @@ create_expected_failure_apply () {
First, rewinding head to replay your work on top of it...
Applying: second commit
Applying: third commit
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
EOF
}
create_expected_failure_merge () {
cat >expected <<-EOF
$(grep "^Created autostash: [0-9a-f][0-9a-f]*\$" actual)
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
Successfully rebased and updated refs/heads/rebased-feature-branch.
EOF
}
diff --git a/t/t7201-co.sh b/t/t7201-co.sh
index 9bcf7c0b40..f4fea7bb7a 100755
--- a/t/t7201-co.sh
+++ b/t/t7201-co.sh
@@ -210,6 +210,166 @@ test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
test_cmp expect two
'
+test_expect_success 'checkout --merge --conflict=zdiff3 <branch>' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ fill a b X d e >two &&
+ git checkout --merge --conflict=zdiff3 simple &&
+
+ cat <<-EOF >expect &&
+ a
+ <<<<<<< simple
+ c
+ ||||||| main
+ b
+ c
+ d
+ =======
+ b
+ X
+ d
+ >>>>>>> local
+ e
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m respects merge.conflictStyle config' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ test_config merge.conflictStyle diff3 &&
+ fill b d >two &&
+ git checkout -m simple &&
+
+ cat <<-EOF >expect &&
+ <<<<<<< simple
+ a
+ c
+ e
+ ||||||| main
+ a
+ b
+ c
+ d
+ e
+ =======
+ b
+ d
+ >>>>>>> local
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m skips stash when no conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m skips stash with non-conflicting dirty index' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m stashes and applies on conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m with mixed staged and unstaged changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on truly conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m produces usable stash on conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "recover your local changes" actual &&
+ git checkout -f main &&
+ git stash pop &&
+ fill 1 2 3 4 5 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on staged conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git add one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m -b skips stash with dirty tree' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m -b newbranch >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ git checkout main &&
+ git branch -D newbranch
+'
+
test_expect_success 'switch to another branch while carrying a deletion' '
git checkout -f main &&
git reset --hard &&
diff --git a/t/t7600-merge.sh b/t/t7600-merge.sh
index 9838094b66..cbef8a534e 100755
--- a/t/t7600-merge.sh
+++ b/t/t7600-merge.sh
@@ -914,7 +914,7 @@ test_expect_success 'merge with conflicted --autostash changes' '
git diff >expect &&
test_when_finished "test_might_fail git stash drop" &&
git merge --autostash c3 2>err &&
- test_grep "Applying autostash resulted in conflicts." err &&
+ test_grep "your local changes resulted in conflicts" err &&
git show HEAD:file >merge-result &&
test_cmp result.1-9 merge-result &&
git stash show -p >actual &&
diff --git a/xdiff-interface.c b/xdiff-interface.c
index f043330f2a..5ee2b96d0a 100644
--- a/xdiff-interface.c
+++ b/xdiff-interface.c
@@ -325,6 +325,18 @@ int parse_conflict_style_name(const char *value)
return -1;
}
+const char *conflict_style_name(int style)
+{
+ switch (style) {
+ case XDL_MERGE_DIFF3:
+ return "diff3";
+ case XDL_MERGE_ZEALOUS_DIFF3:
+ return "zdiff3";
+ default:
+ return "merge";
+ }
+}
+
int git_xmerge_style = -1;
int git_xmerge_config(const char *var, const char *value,
diff --git a/xdiff-interface.h b/xdiff-interface.h
index fbc4ceec40..ce54e1c0e0 100644
--- a/xdiff-interface.h
+++ b/xdiff-interface.h
@@ -55,6 +55,7 @@ void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
void xdiff_clear_find_func(xdemitconf_t *xecfg);
struct config_context;
int parse_conflict_style_name(const char *value);
+const char *conflict_style_name(int style);
int git_xmerge_config(const char *var, const char *value,
const struct config_context *ctx, void *cb);
extern int git_xmerge_style;
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-14 17:17 ` Junio C Hamano
@ 2026-03-16 16:36 ` Phillip Wood
2026-03-16 20:04 ` Junio C Hamano
0 siblings, 1 reply; 29+ messages in thread
From: Phillip Wood @ 2026-03-16 16:36 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Harald Nordgren via GitGitGadget, git, Harald Nordgren
On 14/03/2026 17:17, Junio C Hamano wrote:
> Phillip Wood <phillip.wood123@gmail.com> writes:
>
>> On 12/03/2026 14:40, Junio C Hamano wrote:
>>>
>>> Perhaps doing it this way would make it more worth doing?
>>>
>>> - At the beginning of branch switching, ask a new helper function
>>> that takes the branch we are switching to as an argument this
>>> question:
>>>
>>> Do any paths that are different between the current branch and
>>> the branch we are switching to have local (i.e., either in the
>>> index or in the working tree) change [Yes/No]?
>>>
>>> - When the answer is "yes", save the local changes to a new stash
>>> entry, and clear the local changes from the index and from the
>>> working tree. If not, do not bother with stash at all.
>>
>> Can we avoid the extra check and stash if the user passed "--autostash"
>> and unpack_trees() fails because it would overwrite local changes in
>> merge_working_tree()?
>
> Sorry, but I couldn't quite figure out what you are saying here.
>
> My guess on one part of what it says is that an explicit
> "--autostash", we should stash without second guessing the user
> (i.e., avoid chedk and stash). But then the latter part of the
> sentence "and unpack_trees() fails ..." do not quite parse.
>
> If the user gave "--autostash" and we check with unpack_trees()
> dry-run and find out that a normal branch switch will be interfered
> by the local changes, then we would stash, but that check made by a
> dry-run unpack_trees() is not an "extra" check, so, that does not
> work as a guess of what you are saying, either.
Why is the dry-run of unpack_trees() not an extra check? I was assuming
that it was because we do the dry-run and then do it for real after
possibly stashing any local changes. That's why I was wondering if we
could avoid the dry-run by creating the stash if the non-dry-run
unpack_trees() failed. Looking at the unpack_trees() implementation it
can fail for a variety of reasons, only some (one?) of which can be
addressed by stashing local changes but there does not seem to be a way
for the caller to determine what caused it to fail.
>>> If we can sell it as an improved implementation of "-m", we probably
>>> can lose some code that the current "-m" implementation uses to do
>>> its merge; we'd be instead using the "unstash" code paths.
>>
>> That would be nice but I think "git checkout --recurse-submodules -m
>> <branch>" currently updates submodules whereas "git stash" does not know
>> how to recurse submodules.
>
> Hmph, I do not do submodules outside what we already have, and I
> certainly do not do "checkout --recurse-submodules" with or without
> "-m" with local changes in our submodule.
>
> But does "git stash" even need to know about recursing into
> submodules for this? When checkout recurses into a submodule, that
> checkout that is working in the repository of the submodule can
> handle "-m" itself, which may stash the local changes made in the
> submodule, no?
Oh, because this all happens in a single command then yes, I think we
can. When I wrote that I'd been thinking about a recent question about
rebase not recursing submodules on discord and what it would take to
make "git rebase --recurse-submodules --autostash" work. There we need
to be able to retrive the stash in a different process to the one that
created it so we need some way of tracking the stashed changes in each
submodule.
It turns out I'd misremembered what "git checkout -m
--recurse-submodules" does at the moment - after testing it, it seems to
simply nuke an uncommitted submodule changes rather than merging them.
Thanks
Phillip
>> It would be nice to teach "git stash" to recurse submodules but I don't
>> think it is completly straight forward as we'd need to store the object
>> id of the submodule's stash commit in the parent stash.
>
> No, let's not add more commands that take "--recurse-submodules", if
> we do not have to.
>
> Thanks.
>
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-16 16:36 ` Phillip Wood
@ 2026-03-16 20:04 ` Junio C Hamano
2026-03-17 9:47 ` Harald Nordgren
0 siblings, 1 reply; 29+ messages in thread
From: Junio C Hamano @ 2026-03-16 20:04 UTC (permalink / raw)
To: Phillip Wood; +Cc: Harald Nordgren via GitGitGadget, git, Harald Nordgren
Phillip Wood <phillip.wood123@gmail.com> writes:
> Why is the dry-run of unpack_trees() not an extra check? I was assuming
> that it was because we do the dry-run and then do it for real after
> possibly stashing any local changes. That's why I was wondering if we
> could avoid the dry-run by creating the stash if the non-dry-run
> unpack_trees() failed.
Ah, I didn't even think about that possibility.
Try to unpack anyway, and if unpack_trees() branch switching
succeeds, we are done. Otherwise, we can trust that unpack_trees()
did not do _anything_ to the index or the working tree files, so we
can create the stash at that time.
Makes sense.
> It turns out I'd misremembered what "git checkout -m
> --recurse-submodules" does at the moment - after testing it, it seems to
> simply nuke an uncommitted submodule changes rather than merging them.
Makes sense.
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v6 0/4] checkout: 'autostash' for branch switching
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
` (3 preceding siblings ...)
2026-03-15 11:19 ` [PATCH v5 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
@ 2026-03-17 9:35 ` Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
` (3 more replies)
4 siblings, 4 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-17 9:35 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren
cc: Phillip Wood phillip.wood123@gmail.com
Harald Nordgren (4):
stash: add --ours-label, --theirs-label, --base-label for apply
sequencer: allow create_autostash to run silently
sequencer: teach autostash apply to take optional conflict marker
labels
checkout: -m (--merge) uses autostash when switching branches
Documentation/git-checkout.adoc | 58 ++++++------
Documentation/git-stash.adoc | 11 ++-
Documentation/git-switch.adoc | 27 +++---
builtin/checkout.c | 137 ++++++++++++---------------
builtin/stash.c | 32 +++++--
sequencer.c | 67 +++++++++----
sequencer.h | 4 +
t/t3420-rebase-autostash.sh | 24 +++--
t/t3903-stash.sh | 18 ++++
t/t7201-co.sh | 160 ++++++++++++++++++++++++++++++++
t/t7600-merge.sh | 2 +-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
13 files changed, 403 insertions(+), 150 deletions(-)
base-commit: ca1db8a0f7dc0dbea892e99f5b37c5fe5861be71
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2234%2FHaraldNordgren%2Fcheckout_autostash-v6
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2234/HaraldNordgren/checkout_autostash-v6
Pull-Request: https://github.com/git/git/pull/2234
Range-diff vs v5:
1: 00d8920498 = 1: cd9c64ba60 stash: add --ours-label, --theirs-label, --base-label for apply
2: 5d176f1700 = 2: d572c4bb7d sequencer: allow create_autostash to run silently
3: 3d6829438a = 3: 80a98116fc sequencer: teach autostash apply to take optional conflict marker labels
4: 7f3735d40e ! 4: 7ecb0835b7 checkout: -m (--merge) uses autostash when switching branches
@@ builtin/checkout.c
#include "submodule.h"
#include "symlinks.h"
@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
+ struct tree *new_tree;
+ repo_hold_locked_index(the_repository, &lock_file, LOCK_DIE_ON_ERROR);
+- if (repo_read_index_preload(the_repository, NULL, 0) < 0)
++ if (repo_read_index_preload(the_repository, NULL, 0) < 0) {
++ rollback_lock_file(&lock_file);
+ return error(_("index file corrupt"));
++ }
+
+ resolve_undo_clear_index(the_repository->index);
+ if (opts->new_orphan_branch && opts->orphan_from_empty_tree) {
+@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
+ } else {
+ new_tree = repo_get_commit_tree(the_repository,
+ new_branch_info->commit);
+- if (!new_tree)
++ if (!new_tree) {
++ rollback_lock_file(&lock_file);
+ return error(_("unable to read tree (%s)"),
+ oid_to_hex(&new_branch_info->commit->object.oid));
++ }
+ }
+ if (opts->discard_changes) {
+ ret = reset_tree(new_tree, opts, 1, writeout_error, new_branch_info);
+- if (ret)
++ if (ret) {
++ rollback_lock_file(&lock_file);
+ return ret;
++ }
+ } else {
+ struct tree_desc trees[2];
+ struct tree *tree;
+@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
+ refresh_index(the_repository->index, REFRESH_QUIET, NULL, NULL, NULL);
+
+ if (unmerged_index(the_repository->index)) {
++ rollback_lock_file(&lock_file);
+ error(_("you need to resolve your current index first"));
+ return 1;
+ }
+@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *opts,
ret = unpack_trees(2, trees, &topts);
clear_unpack_trees_porcelain(&topts);
-- if (ret == -1) {
+ if (ret == -1) {
- /*
- * Unpack couldn't do a trivial merge; either
- * give up or do a real merge, depending on
@@ builtin/checkout.c: static int merge_working_tree(const struct checkout_opts *op
- strbuf_release(&old_commit_shortname);
- if (ret)
- return ret;
-- }
-+ if (ret == -1)
++ rollback_lock_file(&lock_file);
+ return 1;
+ }
}
- if (!cache_tree_fully_valid(the_repository->index->cache_tree))
-@@ builtin/checkout.c: static void orphaned_commit_warning(struct commit *old_commit, struct commit *ne
- release_revisions(&revs);
- }
-
-+static int checkout_would_clobber_changes(struct branch_info *old_branch_info,
-+ struct branch_info *new_branch_info)
-+{
-+ struct tree_desc trees[2];
-+ struct tree *old_tree, *new_tree;
-+ struct unpack_trees_options topts;
-+ struct index_state tmp_index = INDEX_STATE_INIT(the_repository);
-+ const struct object_id *old_commit_oid;
-+ int ret;
-+
-+ if (!new_branch_info->commit)
-+ return 0;
-+
-+ old_commit_oid = old_branch_info->commit ?
-+ &old_branch_info->commit->object.oid :
-+ the_hash_algo->empty_tree;
-+ old_tree = repo_parse_tree_indirect(the_repository, old_commit_oid);
-+ if (!old_tree)
-+ return 0;
-+
-+ new_tree = repo_get_commit_tree(the_repository,
-+ new_branch_info->commit);
-+ if (!new_tree)
-+ return 0;
-+ if (repo_parse_tree(the_repository, new_tree) < 0)
-+ return 0;
-+
-+ memset(&topts, 0, sizeof(topts));
-+ topts.head_idx = -1;
-+ topts.src_index = the_repository->index;
-+ topts.dst_index = &tmp_index;
-+ topts.initial_checkout = is_index_unborn(the_repository->index);
-+ topts.merge = 1;
-+ topts.update = 1;
-+ topts.dry_run = 1;
-+ topts.quiet = 1;
-+ topts.fn = twoway_merge;
-+
-+ init_tree_desc(&trees[0], &old_tree->object.oid,
-+ old_tree->buffer, old_tree->size);
-+ init_tree_desc(&trees[1], &new_tree->object.oid,
-+ new_tree->buffer, new_tree->size);
-+
-+ ret = unpack_trees(2, trees, &topts);
-+ discard_index(&tmp_index);
-+
-+ return ret != 0;
-+}
-+
- static int switch_branches(const struct checkout_opts *opts,
- struct branch_info *new_branch_info)
- {
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
struct object_id rev;
int flag, writeout_error = 0;
@@ builtin/checkout.c: static int switch_branches(const struct checkout_opts *opts,
+ stash_label_ancestor = old_commit_shortname.buf;
+ }
+
-+ if (opts->merge) {
-+ if (repo_read_index(the_repository) < 0)
-+ die(_("index file corrupt"));
-+ if (checkout_would_clobber_changes(&old_branch_info,
-+ new_branch_info)) {
+ if (do_merge) {
+ ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
++ if (ret && opts->merge) {
+ create_autostash_ref_silent(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ created_autostash = 1;
++ ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
+ }
-+ }
-+
- if (do_merge) {
- ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
if (ret) {
+ apply_autostash_ref_with_labels(the_repository,
+ "CHECKOUT_AUTOSTASH",
--
gitgitgadget
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v6 1/4] stash: add --ours-label, --theirs-label, --base-label for apply
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
@ 2026-03-17 9:35 ` Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
` (2 subsequent siblings)
3 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-17 9:35 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Allow callers of "git stash apply" to pass custom labels for conflict
markers instead of the default "Updated upstream" and "Stashed changes".
Document the new options and add a test.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
Documentation/git-stash.adoc | 11 ++++++++++-
builtin/stash.c | 2 +-
t/t3903-stash.sh | 18 ++++++++++++++++++
3 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-stash.adoc b/Documentation/git-stash.adoc
index 235d57ddd8..43650a3c1a 100644
--- a/Documentation/git-stash.adoc
+++ b/Documentation/git-stash.adoc
@@ -12,7 +12,7 @@ git stash list [<log-options>]
git stash show [-u | --include-untracked | --only-untracked] [<diff-options>] [<stash>]
git stash drop [-q | --quiet] [<stash>]
git stash pop [--index] [-q | --quiet] [<stash>]
-git stash apply [--index] [-q | --quiet] [<stash>]
+git stash apply [--index] [-q | --quiet] [--ours-label=<label>] [--theirs-label=<label>] [--base-label=<label>] [<stash>]
git stash branch <branchname> [<stash>]
git stash [push [-p | --patch] [-S | --staged] [-k | --[no-]keep-index] [-q | --quiet]
[-u | --include-untracked] [-a | --all] [(-m | --message) <message>]
@@ -197,6 +197,15 @@ the index's ones. However, this can fail, when you have conflicts
(which are stored in the index, where you therefore can no longer
apply the changes as they were originally).
+`--ours-label=<label>`::
+`--theirs-label=<label>`::
+`--base-label=<label>`::
+ These options are only valid for the `apply` command.
++
+Use the given labels in conflict markers instead of the default
+"Updated upstream", "Stashed changes", and "Stash base".
+`--base-label` only has an effect with merge.conflictStyle=diff3.
+
`-k`::
`--keep-index`::
`--no-keep-index`::
diff --git a/builtin/stash.c b/builtin/stash.c
index e79d612e57..252e4df3a9 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -44,7 +44,7 @@
#define BUILTIN_STASH_POP_USAGE \
N_("git stash pop [--index] [-q | --quiet] [<stash>]")
#define BUILTIN_STASH_APPLY_USAGE \
- N_("git stash apply [--index] [-q | --quiet] [<stash>]")
+ N_("git stash apply [--index] [-q | --quiet] [--ours-label=<label>] [--theirs-label=<label>] [--base-label=<label>] [<stash>]")
#define BUILTIN_STASH_BRANCH_USAGE \
N_("git stash branch <branchname> [<stash>]")
#define BUILTIN_STASH_STORE_USAGE \
diff --git a/t/t3903-stash.sh b/t/t3903-stash.sh
index 70879941c2..dd47c1322a 100755
--- a/t/t3903-stash.sh
+++ b/t/t3903-stash.sh
@@ -1666,6 +1666,24 @@ test_expect_success 'restore untracked files even when we hit conflicts' '
)
'
+test_expect_success 'apply with custom conflict labels' '
+ git init conflict_labels &&
+ (
+ cd conflict_labels &&
+ echo base >file &&
+ git add file &&
+ git commit -m base &&
+ echo stashed >file &&
+ git stash push -m "stashed" &&
+ echo upstream >file &&
+ git add file &&
+ git commit -m upstream &&
+ test_must_fail git stash apply --ours-label=UP --theirs-label=STASH &&
+ grep "^<<<<<<< UP" file &&
+ grep "^>>>>>>> STASH" file
+ )
+'
+
test_expect_success 'stash create reports a locked index' '
test_when_finished "rm -rf repo" &&
git init repo &&
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v6 2/4] sequencer: allow create_autostash to run silently
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
@ 2026-03-17 9:35 ` Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
3 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-17 9:35 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Add a silent parameter to create_autostash_internal and introduce
create_autostash_ref_silent so that callers can create an autostash
without printing the "Created autostash" message. Use stderr for
the message when not silent.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
sequencer.c | 15 +++++++++++----
sequencer.h | 1 +
2 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index aafd0bc959..eebefd731b 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4632,7 +4632,8 @@ static enum todo_command peek_command(struct todo_list *todo_list, int offset)
static void create_autostash_internal(struct repository *r,
const char *path,
- const char *refname)
+ const char *refname,
+ int silent)
{
struct strbuf buf = STRBUF_INIT;
struct lock_file lock_file = LOCK_INIT;
@@ -4677,7 +4678,8 @@ static void create_autostash_internal(struct repository *r,
&oid, null_oid(the_hash_algo), 0, UPDATE_REFS_DIE_ON_ERR);
}
- printf(_("Created autostash: %s\n"), buf.buf);
+ if (!silent)
+ fprintf(stderr, _("Created autostash: %s\n"), buf.buf);
if (reset_head(r, &ropts) < 0)
die(_("could not reset --hard"));
discard_index(r->index);
@@ -4689,12 +4691,17 @@ static void create_autostash_internal(struct repository *r,
void create_autostash(struct repository *r, const char *path)
{
- create_autostash_internal(r, path, NULL);
+ create_autostash_internal(r, path, NULL, 0);
}
void create_autostash_ref(struct repository *r, const char *refname)
{
- create_autostash_internal(r, NULL, refname);
+ create_autostash_internal(r, NULL, refname, 0);
+}
+
+void create_autostash_ref_silent(struct repository *r, const char *refname)
+{
+ create_autostash_internal(r, NULL, refname, 1);
}
static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
diff --git a/sequencer.h b/sequencer.h
index 719684c8a9..0b09d6799b 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -227,6 +227,7 @@ void commit_post_rewrite(struct repository *r,
void create_autostash(struct repository *r, const char *path);
void create_autostash_ref(struct repository *r, const char *refname);
+void create_autostash_ref_silent(struct repository *r, const char *refname);
int save_autostash(const char *path);
int save_autostash_ref(struct repository *r, const char *refname);
int apply_autostash(const char *path);
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v6 3/4] sequencer: teach autostash apply to take optional conflict marker labels
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
@ 2026-03-17 9:35 ` Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
3 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-17 9:35 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
Add label1, label2, and label_ancestor parameters to the autostash
apply machinery so callers can pass custom conflict marker labels
through to "git stash apply --ours-label/--theirs-label/--base-label".
Introduce apply_autostash_ref_with_labels() for callers that want
to pass labels.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
sequencer.c | 34 +++++++++++++++++++++++++++-------
sequencer.h | 3 +++
2 files changed, 30 insertions(+), 7 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index eebefd731b..080a25820a 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4704,7 +4704,9 @@ void create_autostash_ref_silent(struct repository *r, const char *refname)
create_autostash_internal(r, NULL, refname, 1);
}
-static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
+static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct child_process child = CHILD_PROCESS_INIT;
int ret = 0;
@@ -4715,6 +4717,12 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply)
child.no_stderr = 1;
strvec_push(&child.args, "stash");
strvec_push(&child.args, "apply");
+ if (label1)
+ strvec_pushf(&child.args, "--ours-label=%s", label1);
+ if (label2)
+ strvec_pushf(&child.args, "--theirs-label=%s", label2);
+ if (label_ancestor)
+ strvec_pushf(&child.args, "--base-label=%s", label_ancestor);
strvec_push(&child.args, stash_oid);
ret = run_command(&child);
}
@@ -4759,7 +4767,8 @@ static int apply_save_autostash(const char *path, int attempt_apply)
}
strbuf_trim(&stash_oid);
- ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid.buf, attempt_apply,
+ NULL, NULL, NULL);
unlink(path);
strbuf_release(&stash_oid);
@@ -4778,11 +4787,13 @@ int apply_autostash(const char *path)
int apply_autostash_oid(const char *stash_oid)
{
- return apply_save_autostash_oid(stash_oid, 1);
+ return apply_save_autostash_oid(stash_oid, 1, NULL, NULL, NULL);
}
static int apply_save_autostash_ref(struct repository *r, const char *refname,
- int attempt_apply)
+ int attempt_apply,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
struct object_id stash_oid;
char stash_oid_hex[GIT_MAX_HEXSZ + 1];
@@ -4798,7 +4809,8 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
return error(_("autostash reference is a symref"));
oid_to_hex_r(stash_oid_hex, &stash_oid);
- ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply);
+ ret = apply_save_autostash_oid(stash_oid_hex, attempt_apply,
+ label1, label2, label_ancestor);
refs_delete_ref(get_main_ref_store(r), "", refname,
&stash_oid, REF_NO_DEREF);
@@ -4808,12 +4820,20 @@ static int apply_save_autostash_ref(struct repository *r, const char *refname,
int save_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 0);
+ return apply_save_autostash_ref(r, refname, 0, NULL, NULL, NULL);
}
int apply_autostash_ref(struct repository *r, const char *refname)
{
- return apply_save_autostash_ref(r, refname, 1);
+ return apply_save_autostash_ref(r, refname, 1, NULL, NULL, NULL);
+}
+
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
+{
+ return apply_save_autostash_ref(r, refname, 1,
+ label1, label2, label_ancestor);
}
static int checkout_onto(struct repository *r, struct replay_opts *opts,
diff --git a/sequencer.h b/sequencer.h
index 0b09d6799b..68b94d86e3 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -233,6 +233,9 @@ int save_autostash_ref(struct repository *r, const char *refname);
int apply_autostash(const char *path);
int apply_autostash_oid(const char *stash_oid);
int apply_autostash_ref(struct repository *r, const char *refname);
+int apply_autostash_ref_with_labels(struct repository *r, const char *refname,
+ const char *label1, const char *label2,
+ const char *label_ancestor);
#define SUMMARY_INITIAL_COMMIT (1 << 0)
#define SUMMARY_SHOW_AUTHOR_DATE (1 << 1)
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v6 4/4] checkout: -m (--merge) uses autostash when switching branches
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
` (2 preceding siblings ...)
2026-03-17 9:35 ` [PATCH v6 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
@ 2026-03-17 9:35 ` Harald Nordgren via GitGitGadget
3 siblings, 0 replies; 29+ messages in thread
From: Harald Nordgren via GitGitGadget @ 2026-03-17 9:35 UTC (permalink / raw)
To: git; +Cc: Harald Nordgren, Harald Nordgren
From: Harald Nordgren <haraldnordgren@gmail.com>
When switching branches with "git checkout -m", local modifications
can block the switch. Teach the -m flow to create a temporary stash
before switching and reapply it after. On success, only "Applied
autostash." is shown. If reapplying causes conflicts, the stash is
kept and the user is told they can resolve and run "git stash drop",
or run "git reset --hard" and later "git stash pop" to recover their
changes.
Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
---
Documentation/git-checkout.adoc | 58 ++++++------
Documentation/git-switch.adoc | 27 +++---
builtin/checkout.c | 137 ++++++++++++---------------
builtin/stash.c | 30 ++++--
sequencer.c | 18 +++-
t/t3420-rebase-autostash.sh | 24 +++--
t/t7201-co.sh | 160 ++++++++++++++++++++++++++++++++
t/t7600-merge.sh | 2 +-
xdiff-interface.c | 12 +++
xdiff-interface.h | 1 +
10 files changed, 332 insertions(+), 137 deletions(-)
diff --git a/Documentation/git-checkout.adoc b/Documentation/git-checkout.adoc
index 43ccf47cf6..18022e610e 100644
--- a/Documentation/git-checkout.adoc
+++ b/Documentation/git-checkout.adoc
@@ -251,20 +251,19 @@ working tree, by copying them from elsewhere, extracting a tarball, etc.
are different between the current branch and the branch to
which you are switching, the command refuses to switch
branches in order to preserve your modifications in context.
- However, with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch
- is done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ With this option, the conflicting local changes are
+ automatically stashed before the switch and reapplied
+ afterwards. If the local changes do not overlap with the
+ differences between branches, the switch proceeds without
+ stashing. If reapplying the stash results in conflicts, the
+ entry is saved to the stash list. Resolve the conflicts
+ and run `git stash drop` when done, or clear the working
+ tree (e.g. with `git reset --hard`) before running `git stash
+ pop` later to re-apply your changes.
+
When checking out paths from the index, this option lets you recreate
the conflicted merge in the specified paths. This option cannot be
used when checking out paths from a tree-ish.
-+
-When switching branches with `--merge`, staged changes may be lost.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -578,39 +577,44 @@ $ git checkout mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a
-three-way merge:
+You can give the `-m` flag to the command, which would carry your local
+changes to the new branch:
------------
$ git checkout -m mytopic
-Auto-merging frotz
+Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
=== 3. Merge conflict
-When a merge conflict happens during switching branches with
-the `-m` option, you would see something like this:
+When the `--merge` (`-m`) option is in effect and the locally
+modified files overlap with files that need to be updated by the
+branch switch, the changes are stashed and reapplied after the
+switch. If the stash application results in conflicts, they are not
+resolved and the stash is saved to the stash list:
------------
$ git checkout -m mytopic
-Auto-merging frotz
-ERROR: Merge conflict in frotz
-fatal: merge program failed
-------------
+Your local changes are stashed, however, applying it to carry
+forward your local changes resulted in conflicts:
-At this point, `git diff` shows the changes cleanly merged as in
-the previous example, as well as the changes in the conflicted
-files. Edit and resolve the conflict and mark it resolved with
-`git add` as usual:
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
------------
-$ edit frotz
-$ git add frotz
-------------
+
+You can try resolving the conflicts now. Edit the conflicting files
+and mark them resolved with `git add` as usual, then run `git stash
+drop` to discard the stash entry. Alternatively, you can clear the
+working tree with `git reset --hard` and recover your local changes
+later with `git stash pop`.
CONFIGURATION
-------------
diff --git a/Documentation/git-switch.adoc b/Documentation/git-switch.adoc
index 87707e9265..3147023dc3 100644
--- a/Documentation/git-switch.adoc
+++ b/Documentation/git-switch.adoc
@@ -126,15 +126,16 @@ variable.
If you have local modifications to one or more files that are
different between the current branch and the branch to which
you are switching, the command refuses to switch branches in
- order to preserve your modifications in context. However,
- with this option, a three-way merge between the current
- branch, your working tree contents, and the new branch is
- done, and you will be on the new branch.
-+
-When a merge conflict happens, the index entries for conflicting
-paths are left unmerged, and you need to resolve the conflicts
-and mark the resolved paths with `git add` (or `git rm` if the merge
-should result in deletion of the path).
+ order to preserve your modifications in context. With this
+ option, the conflicting local changes are automatically
+ stashed before the switch and reapplied afterwards. If the
+ local changes do not overlap with the differences between
+ branches, the switch proceeds without stashing. If
+ reapplying the stash results in conflicts, the entry is
+ saved to the stash list. Resolve the conflicts and run
+ `git stash drop` when done, or clear the working tree
+ (e.g. with `git reset --hard`) before running `git stash pop`
+ later to re-apply your changes.
`--conflict=<style>`::
The same as `--merge` option above, but changes the way the
@@ -217,15 +218,15 @@ $ git switch mytopic
error: You have local changes to 'frotz'; not switching branches.
------------
-You can give the `-m` flag to the command, which would try a three-way
-merge:
+You can give the `-m` flag to the command, which would carry your local
+changes to the new branch:
------------
$ git switch -m mytopic
-Auto-merging frotz
+Switched to branch 'mytopic'
------------
-After this three-way merge, the local modifications are _not_
+After the switch, the local modifications are reapplied and are _not_
registered in your index file, so `git diff` would show you what
changes you made since the tip of the new branch.
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 1d1667fa4c..dc935d6bcb 100644
--- a/builtin/checkout.c
+++ b/builtin/checkout.c
@@ -17,7 +17,6 @@
#include "merge-ll.h"
#include "lockfile.h"
#include "mem-pool.h"
-#include "merge-ort-wrappers.h"
#include "object-file.h"
#include "object-name.h"
#include "odb.h"
@@ -30,6 +29,7 @@
#include "repo-settings.h"
#include "resolve-undo.h"
#include "revision.h"
+#include "sequencer.h"
#include "setup.h"
#include "submodule.h"
#include "symlinks.h"
@@ -783,8 +783,10 @@ static int merge_working_tree(const struct checkout_opts *opts,
struct tree *new_tree;
repo_hold_locked_index(the_repository, &lock_file, LOCK_DIE_ON_ERROR);
- if (repo_read_index_preload(the_repository, NULL, 0) < 0)
+ if (repo_read_index_preload(the_repository, NULL, 0) < 0) {
+ rollback_lock_file(&lock_file);
return error(_("index file corrupt"));
+ }
resolve_undo_clear_index(the_repository->index);
if (opts->new_orphan_branch && opts->orphan_from_empty_tree) {
@@ -797,14 +799,18 @@ static int merge_working_tree(const struct checkout_opts *opts,
} else {
new_tree = repo_get_commit_tree(the_repository,
new_branch_info->commit);
- if (!new_tree)
+ if (!new_tree) {
+ rollback_lock_file(&lock_file);
return error(_("unable to read tree (%s)"),
oid_to_hex(&new_branch_info->commit->object.oid));
+ }
}
if (opts->discard_changes) {
ret = reset_tree(new_tree, opts, 1, writeout_error, new_branch_info);
- if (ret)
+ if (ret) {
+ rollback_lock_file(&lock_file);
return ret;
+ }
} else {
struct tree_desc trees[2];
struct tree *tree;
@@ -814,6 +820,7 @@ static int merge_working_tree(const struct checkout_opts *opts,
refresh_index(the_repository->index, REFRESH_QUIET, NULL, NULL, NULL);
if (unmerged_index(the_repository->index)) {
+ rollback_lock_file(&lock_file);
error(_("you need to resolve your current index first"));
return 1;
}
@@ -846,81 +853,8 @@ static int merge_working_tree(const struct checkout_opts *opts,
ret = unpack_trees(2, trees, &topts);
clear_unpack_trees_porcelain(&topts);
if (ret == -1) {
- /*
- * Unpack couldn't do a trivial merge; either
- * give up or do a real merge, depending on
- * whether the merge flag was used.
- */
- struct tree *work;
- struct tree *old_tree;
- struct merge_options o;
- struct strbuf sb = STRBUF_INIT;
- struct strbuf old_commit_shortname = STRBUF_INIT;
-
- if (!opts->merge)
- return 1;
-
- /*
- * Without old_branch_info->commit, the below is the same as
- * the two-tree unpack we already tried and failed.
- */
- if (!old_branch_info->commit)
- return 1;
- old_tree = repo_get_commit_tree(the_repository,
- old_branch_info->commit);
-
- if (repo_index_has_changes(the_repository, old_tree, &sb))
- die(_("cannot continue with staged changes in "
- "the following files:\n%s"), sb.buf);
- strbuf_release(&sb);
-
- /* Do more real merge */
-
- /*
- * We update the index fully, then write the
- * tree from the index, then merge the new
- * branch with the current tree, with the old
- * branch as the base. Then we reset the index
- * (but not the working tree) to the new
- * branch, leaving the working tree as the
- * merged version, but skipping unmerged
- * entries in the index.
- */
-
- add_files_to_cache(the_repository, NULL, NULL, NULL, 0,
- 0, 0);
- init_ui_merge_options(&o, the_repository);
- o.verbosity = 0;
- work = write_in_core_index_as_tree(the_repository);
-
- ret = reset_tree(new_tree,
- opts, 1,
- writeout_error, new_branch_info);
- if (ret)
- return ret;
- o.ancestor = old_branch_info->name;
- if (!old_branch_info->name) {
- strbuf_add_unique_abbrev(&old_commit_shortname,
- &old_branch_info->commit->object.oid,
- DEFAULT_ABBREV);
- o.ancestor = old_commit_shortname.buf;
- }
- o.branch1 = new_branch_info->name;
- o.branch2 = "local";
- o.conflict_style = opts->conflict_style;
- ret = merge_ort_nonrecursive(&o,
- new_tree,
- work,
- old_tree);
- if (ret < 0)
- die(NULL);
- ret = reset_tree(new_tree,
- opts, 0,
- writeout_error, new_branch_info);
- strbuf_release(&o.obuf);
- strbuf_release(&old_commit_shortname);
- if (ret)
- return ret;
+ rollback_lock_file(&lock_file);
+ return 1;
}
}
@@ -1165,6 +1099,9 @@ static int switch_branches(const struct checkout_opts *opts,
struct object_id rev;
int flag, writeout_error = 0;
int do_merge = 1;
+ int created_autostash = 0;
+ struct strbuf old_commit_shortname = STRBUF_INIT;
+ const char *stash_label_ancestor = NULL;
trace2_cmd_mode("branch");
@@ -1202,10 +1139,31 @@ static int switch_branches(const struct checkout_opts *opts,
do_merge = 0;
}
+ if (old_branch_info.name)
+ stash_label_ancestor = old_branch_info.name;
+ else if (old_branch_info.commit) {
+ strbuf_add_unique_abbrev(&old_commit_shortname,
+ &old_branch_info.commit->object.oid,
+ DEFAULT_ABBREV);
+ stash_label_ancestor = old_commit_shortname.buf;
+ }
+
if (do_merge) {
ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
+ if (ret && opts->merge) {
+ create_autostash_ref_silent(the_repository,
+ "CHECKOUT_AUTOSTASH");
+ created_autostash = 1;
+ ret = merge_working_tree(opts, &old_branch_info, new_branch_info, &writeout_error);
+ }
if (ret) {
+ apply_autostash_ref_with_labels(the_repository,
+ "CHECKOUT_AUTOSTASH",
+ new_branch_info->name,
+ "local",
+ stash_label_ancestor);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret;
}
}
@@ -1215,8 +1173,29 @@ static int switch_branches(const struct checkout_opts *opts,
update_refs_for_switch(opts, &old_branch_info, new_branch_info);
+ if (opts->conflict_style >= 0) {
+ struct strbuf cfg = STRBUF_INIT;
+ strbuf_addf(&cfg, "merge.conflictStyle=%s",
+ conflict_style_name(opts->conflict_style));
+ git_config_push_parameter(cfg.buf);
+ strbuf_release(&cfg);
+ }
+ apply_autostash_ref_with_labels(the_repository, "CHECKOUT_AUTOSTASH",
+ new_branch_info->name, "local",
+ stash_label_ancestor);
+
+ discard_index(the_repository->index);
+ if (repo_read_index(the_repository) < 0)
+ die(_("index file corrupt"));
+
+ if (created_autostash && !opts->discard_changes && !opts->quiet &&
+ new_branch_info->commit)
+ show_local_changes(&new_branch_info->commit->object,
+ &opts->diff_options);
+
ret = post_checkout_hook(old_branch_info.commit, new_branch_info->commit, 1);
branch_info_release(&old_branch_info);
+ strbuf_release(&old_commit_shortname);
return ret || writeout_error;
}
diff --git a/builtin/stash.c b/builtin/stash.c
index 252e4df3a9..ab9c1359b1 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -590,8 +590,11 @@ static void unstage_changes_unless_new(struct object_id *orig_tree)
die(_("could not write index"));
}
-static int do_apply_stash(const char *prefix, struct stash_info *info,
- int index, int quiet)
+static int do_apply_stash_with_labels(const char *prefix,
+ struct stash_info *info,
+ int index, int quiet,
+ const char *label1, const char *label2,
+ const char *label_ancestor)
{
int clean, ret;
int has_index = index;
@@ -643,9 +646,9 @@ static int do_apply_stash(const char *prefix, struct stash_info *info,
init_ui_merge_options(&o, the_repository);
- o.branch1 = "Updated upstream";
- o.branch2 = "Stashed changes";
- o.ancestor = "Stash base";
+ o.branch1 = label1 ? label1 : "Updated upstream";
+ o.branch2 = label2 ? label2 : "Stashed changes";
+ o.ancestor = label_ancestor ? label_ancestor : "Stash base";
if (oideq(&info->b_tree, &c_tree))
o.branch1 = "Version stash was based on";
@@ -717,17 +720,31 @@ restore_untracked:
return ret;
}
+static int do_apply_stash(const char *prefix, struct stash_info *info,
+ int index, int quiet)
+{
+ return do_apply_stash_with_labels(prefix, info, index, quiet,
+ NULL, NULL, NULL);
+}
+
static int apply_stash(int argc, const char **argv, const char *prefix,
struct repository *repo UNUSED)
{
int ret = -1;
int quiet = 0;
int index = use_index;
+ const char *label1 = NULL, *label2 = NULL, *label_ancestor = NULL;
struct stash_info info = STASH_INFO_INIT;
struct option options[] = {
OPT__QUIET(&quiet, N_("be quiet, only report errors")),
OPT_BOOL(0, "index", &index,
N_("attempt to recreate the index")),
+ OPT_STRING(0, "ours-label", &label1, N_("label"),
+ N_("label for the upstream side in conflict markers")),
+ OPT_STRING(0, "theirs-label", &label2, N_("label"),
+ N_("label for the stashed side in conflict markers")),
+ OPT_STRING(0, "base-label", &label_ancestor, N_("label"),
+ N_("label for the base in diff3 conflict markers")),
OPT_END()
};
@@ -737,7 +754,8 @@ static int apply_stash(int argc, const char **argv, const char *prefix,
if (get_stash_info(&info, argc, argv))
goto cleanup;
- ret = do_apply_stash(prefix, &info, index, quiet);
+ ret = do_apply_stash_with_labels(prefix, &info, index, quiet,
+ label1, label2, label_ancestor);
cleanup:
free_stash_info(&info);
return ret;
diff --git a/sequencer.c b/sequencer.c
index 080a25820a..53e04d8a94 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -4741,15 +4741,23 @@ static int apply_save_autostash_oid(const char *stash_oid, int attempt_apply,
strvec_push(&store.args, stash_oid);
if (run_command(&store))
ret = error(_("cannot store %s"), stash_oid);
+ else if (attempt_apply)
+ fprintf(stderr,
+ _("Your local changes are stashed, however, applying it to carry\n"
+ "forward your local changes resulted in conflicts:\n"
+ "\n"
+ " - You can try resolving them now. If you resolved them\n"
+ " successfully, discard the stash entry with \"git stash drop\".\n"
+ "\n"
+ " - Alternatively you can \"git reset --hard\" if you do not want\n"
+ " to deal with them right now, and later \"git stash pop\" to\n"
+ " recover your local changes.\n"));
else
fprintf(stderr,
- _("%s\n"
+ _("Autostash exists; creating a new stash entry.\n"
"Your changes are safe in the stash.\n"
"You can run \"git stash pop\" or"
- " \"git stash drop\" at any time.\n"),
- attempt_apply ?
- _("Applying autostash resulted in conflicts.") :
- _("Autostash exists; creating a new stash entry."));
+ " \"git stash drop\" at any time.\n"));
}
return ret;
diff --git a/t/t3420-rebase-autostash.sh b/t/t3420-rebase-autostash.sh
index ad3ba6a984..e4e2cb19ce 100755
--- a/t/t3420-rebase-autostash.sh
+++ b/t/t3420-rebase-autostash.sh
@@ -61,18 +61,30 @@ create_expected_failure_apply () {
First, rewinding head to replay your work on top of it...
Applying: second commit
Applying: third commit
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
EOF
}
create_expected_failure_merge () {
cat >expected <<-EOF
$(grep "^Created autostash: [0-9a-f][0-9a-f]*\$" actual)
- Applying autostash resulted in conflicts.
- Your changes are safe in the stash.
- You can run "git stash pop" or "git stash drop" at any time.
+ Your local changes are stashed, however, applying it to carry
+ forward your local changes resulted in conflicts:
+
+ - You can try resolving them now. If you resolved them
+ successfully, discard the stash entry with "git stash drop".
+
+ - Alternatively you can "git reset --hard" if you do not want
+ to deal with them right now, and later "git stash pop" to
+ recover your local changes.
Successfully rebased and updated refs/heads/rebased-feature-branch.
EOF
}
diff --git a/t/t7201-co.sh b/t/t7201-co.sh
index 9bcf7c0b40..f4fea7bb7a 100755
--- a/t/t7201-co.sh
+++ b/t/t7201-co.sh
@@ -210,6 +210,166 @@ test_expect_success 'checkout --merge --conflict=diff3 <branch>' '
test_cmp expect two
'
+test_expect_success 'checkout --merge --conflict=zdiff3 <branch>' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ fill a b X d e >two &&
+ git checkout --merge --conflict=zdiff3 simple &&
+
+ cat <<-EOF >expect &&
+ a
+ <<<<<<< simple
+ c
+ ||||||| main
+ b
+ c
+ d
+ =======
+ b
+ X
+ d
+ >>>>>>> local
+ e
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m respects merge.conflictStyle config' '
+ git checkout -f main &&
+ git reset --hard &&
+ git clean -f &&
+
+ test_config merge.conflictStyle diff3 &&
+ fill b d >two &&
+ git checkout -m simple &&
+
+ cat <<-EOF >expect &&
+ <<<<<<< simple
+ a
+ c
+ e
+ ||||||| main
+ a
+ b
+ c
+ d
+ e
+ =======
+ b
+ d
+ >>>>>>> local
+ EOF
+ test_cmp expect two
+'
+
+test_expect_success 'checkout -m skips stash when no conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m skips stash with non-conflicting dirty index' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same
+'
+
+test_expect_success 'checkout -m stashes and applies on conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m with mixed staged and unstaged changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git add same &&
+ fill 1 2 3 4 5 6 7 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "Applied autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ fill 1 2 3 4 5 6 7 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on truly conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ test_must_fail git checkout side 2>stderr &&
+ test_grep "Your local changes" stderr &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m produces usable stash on conflict' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep "recover your local changes" actual &&
+ git checkout -f main &&
+ git stash pop &&
+ fill 1 2 3 4 5 >expect &&
+ test_cmp expect one
+'
+
+test_expect_success 'checkout -m stashes on staged conflicting changes' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 1 2 3 4 5 >one &&
+ git add one &&
+ git checkout -m side >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ test_grep "resulted in conflicts" actual &&
+ test_grep "git stash drop" actual &&
+ git stash drop &&
+ git reset --hard
+'
+
+test_expect_success 'checkout -m -b skips stash with dirty tree' '
+ git checkout -f main &&
+ git clean -f &&
+
+ fill 0 x y z >same &&
+ git checkout -m -b newbranch >actual 2>&1 &&
+ test_grep ! "Created autostash" actual &&
+ fill 0 x y z >expect &&
+ test_cmp expect same &&
+ git checkout main &&
+ git branch -D newbranch
+'
+
test_expect_success 'switch to another branch while carrying a deletion' '
git checkout -f main &&
git reset --hard &&
diff --git a/t/t7600-merge.sh b/t/t7600-merge.sh
index 9838094b66..cbef8a534e 100755
--- a/t/t7600-merge.sh
+++ b/t/t7600-merge.sh
@@ -914,7 +914,7 @@ test_expect_success 'merge with conflicted --autostash changes' '
git diff >expect &&
test_when_finished "test_might_fail git stash drop" &&
git merge --autostash c3 2>err &&
- test_grep "Applying autostash resulted in conflicts." err &&
+ test_grep "your local changes resulted in conflicts" err &&
git show HEAD:file >merge-result &&
test_cmp result.1-9 merge-result &&
git stash show -p >actual &&
diff --git a/xdiff-interface.c b/xdiff-interface.c
index f043330f2a..5ee2b96d0a 100644
--- a/xdiff-interface.c
+++ b/xdiff-interface.c
@@ -325,6 +325,18 @@ int parse_conflict_style_name(const char *value)
return -1;
}
+const char *conflict_style_name(int style)
+{
+ switch (style) {
+ case XDL_MERGE_DIFF3:
+ return "diff3";
+ case XDL_MERGE_ZEALOUS_DIFF3:
+ return "zdiff3";
+ default:
+ return "merge";
+ }
+}
+
int git_xmerge_style = -1;
int git_xmerge_config(const char *var, const char *value,
diff --git a/xdiff-interface.h b/xdiff-interface.h
index fbc4ceec40..ce54e1c0e0 100644
--- a/xdiff-interface.h
+++ b/xdiff-interface.h
@@ -55,6 +55,7 @@ void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
void xdiff_clear_find_func(xdemitconf_t *xecfg);
struct config_context;
int parse_conflict_style_name(const char *value);
+const char *conflict_style_name(int style);
int git_xmerge_config(const char *var, const char *value,
const struct config_context *ctx, void *cb);
extern int git_xmerge_style;
--
gitgitgadget
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-16 20:04 ` Junio C Hamano
@ 2026-03-17 9:47 ` Harald Nordgren
2026-03-19 8:25 ` Harald Nordgren
0 siblings, 1 reply; 29+ messages in thread
From: Harald Nordgren @ 2026-03-17 9:47 UTC (permalink / raw)
To: gitster; +Cc: git, gitgitgadget, haraldnordgren, phillip.wood123
>> Why is the dry-run of unpack_trees() not an extra check? I was assuming
>> that it was because we do the dry-run and then do it for real after
>> possibly stashing any local changes. That's why I was wondering if we
>> could avoid the dry-run by creating the stash if the non-dry-run
>> unpack_trees() failed.
>
> Ah, I didn't even think about that possibility.
>
> Try to unpack anyway, and if unpack_trees() branch switching
> succeeds, we are done. Otherwise, we can trust that unpack_trees()
> did not do _anything_ to the index or the working tree files, so we
> can create the stash at that time.
>
> Makes sense.
Interesting idea, and thanks for your help with this! I gave it a shot with
this simplification.
It passes the tests, which either means it works, or just that the test
coverage is not good enough to detect new issues introduced by me here.
Harald
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-17 9:47 ` Harald Nordgren
@ 2026-03-19 8:25 ` Harald Nordgren
2026-03-19 16:48 ` Junio C Hamano
0 siblings, 1 reply; 29+ messages in thread
From: Harald Nordgren @ 2026-03-19 8:25 UTC (permalink / raw)
To: haraldnordgren; +Cc: git, gitgitgadget, gitster, phillip.wood123
Hi Junio and Jeff!
Did you get a chance to look at the latest changes?
The scope of this grew a lot from my original idea of auto-stashing, so I'm
not 100% convinced that changing '-m' is necessary here. My fear is to
break something, especially since 'checkout -m' is a feature I never used
before touching it here, so I don't have a good sense of how it should
work.
Harald
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH] checkout: add --autostash option for branch switching
2026-03-19 8:25 ` Harald Nordgren
@ 2026-03-19 16:48 ` Junio C Hamano
0 siblings, 0 replies; 29+ messages in thread
From: Junio C Hamano @ 2026-03-19 16:48 UTC (permalink / raw)
To: Harald Nordgren; +Cc: git, gitgitgadget, phillip.wood123
Harald Nordgren <haraldnordgren@gmail.com> writes:
> Hi Junio and Jeff!
>
> Did you get a chance to look at the latest changes?
>
> The scope of this grew a lot from my original idea of auto-stashing, so I'm
> not 100% convinced that changing '-m' is necessary here. My fear is to
> break something, especially since 'checkout -m' is a feature I never used
> before touching it here, so I don't have a good sense of how it should
> work.
FWIW, I very much like what I see in
$ git checkout hn/git-checkout-m-with-stash && git diff @{1}
output. It is great that we do not have to do any dry-run, because
the "real" run safely aborts, we can do the "stash && merge && unstash"
dance as a fallback instead. All the credit goes to Phillip and you
for the idea and the execution of this.
I do use "checkout -m" a few times a week, but I do not do anything
complex with submodules or run the command with unrelated local
modifications, so there may be changes in behaviour I haven't seen
in corner cases that I do not exercise.
^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2026-03-19 16:48 UTC | newest]
Thread overview: 29+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-12 13:26 [PATCH] checkout: add --autostash option for branch switching Harald Nordgren via GitGitGadget
2026-03-12 14:40 ` Junio C Hamano
2026-03-12 19:33 ` [PATCH v31 0/2] status: add status.compareBranches config for multiple branch comparisons Harald Nordgren
2026-03-13 14:29 ` [PATCH] checkout: add --autostash option for branch switching Phillip Wood
2026-03-14 17:17 ` Junio C Hamano
2026-03-16 16:36 ` Phillip Wood
2026-03-16 20:04 ` Junio C Hamano
2026-03-17 9:47 ` Harald Nordgren
2026-03-19 8:25 ` Harald Nordgren
2026-03-19 16:48 ` Junio C Hamano
2026-03-12 19:33 ` [PATCH v2] " Harald Nordgren via GitGitGadget
2026-03-12 19:50 ` Junio C Hamano
2026-03-13 9:22 ` [PATCH] " Harald Nordgren
2026-03-13 9:23 ` [PATCH v3] " Harald Nordgren via GitGitGadget
2026-03-13 17:16 ` Junio C Hamano
2026-03-13 19:33 ` [PATCH] " Harald Nordgren
2026-03-13 20:30 ` Junio C Hamano
2026-03-14 9:59 ` [PATCH v4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
2026-03-15 2:25 ` Junio C Hamano
2026-03-15 11:19 ` [PATCH v5 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
2026-03-15 11:19 ` [PATCH v5 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 0/4] checkout: 'autostash' for branch switching Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 1/4] stash: add --ours-label, --theirs-label, --base-label for apply Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 2/4] sequencer: allow create_autostash to run silently Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 3/4] sequencer: teach autostash apply to take optional conflict marker labels Harald Nordgren via GitGitGadget
2026-03-17 9:35 ` [PATCH v6 4/4] checkout: -m (--merge) uses autostash when switching branches Harald Nordgren via GitGitGadget
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox