[PATCH v2 2/2] stash: allow "git stash [<options>] --patch <pathspec>" to assume push
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- May 20, 2025, 09:27 UTC
- Message-ID
- <98ad3de977090a793408b25ca880b65f058ea44e.1747733203.git.phillip.wood@dunelm.org.uk>
- In-Reply-To
- <cover.1747733203.git.phillip.wood@dunelm.org.uk>
From: Phillip Wood <phillip.wood@dunelm.org.uk>
The support for assuming "push" when "-p" is given introduced in 9e140909f61 (stash: allow pathspecs in the no verb form, 2017-02-28) is very narrow, neither "git stash -m <message> -p <pathspec>" nor "git stash --patch <pathspec>" imply "push" and die instead. Relax this by passing PARSE_OPT_STOP_AT_NON_OPTION when push is being assumed and then setting "force_assume" if "--patch" was present. This means "git stash <pathspec> -p" still dies so that it does not assume the user meant "push" if they mistype a subcommand name but "git stash -m <message> -p <pathspec>" will now succeed. The test added in the last commit is adjusted to check that push is still assumed when "--patch" comes after other options on the command-line.
Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk> --- builtin/stash.c | 10 +++++++--- t/t3903-stash.sh | 4 ++-- 2 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/builtin/stash.c b/builtin/stash.c index bc2c34fa048..b12fd6c40f1 100644 --- a/builtin/stash.c +++ b/builtin/stash.c @@ -1789,11 +1789,15 @@ static int push_stash(int argc, const char **argv, const char *prefix, int ret; if (argc) { - force_assume = argc > 1 && !strcmp(argv[1], "-p"); + int flags = PARSE_OPT_KEEP_DASHDASH; + + if (push_assumed) + flags |= PARSE_OPT_STOP_AT_NON_OPTION; + argc = parse_options(argc, argv, prefix, options, push_assumed ? git_stash_usage : - git_stash_push_usage, - PARSE_OPT_KEEP_DASHDASH); + git_stash_push_usage, flags); + force_assume |= patch_mode; } if (argc) { diff --git a/t/t3903-stash.sh b/t/t3903-stash.sh index d24559a328d..295cb508a35 100755 --- a/t/t3903-stash.sh +++ b/t/t3903-stash.sh @@ -1177,11 +1177,11 @@ test_expect_success 'stash -- <pathspec> stashes and restores the file' ' test_path_is_file bar ' -test_expect_success 'stash -p <pathspec> stash and restores the file' ' +test_expect_success 'stash --patch <pathspec> stash and restores the file' ' cat file >expect-file && echo changed-file >file && echo changed-other-file >other-file && - echo a | git stash -p file && + echo a | git stash -m "stash bar" --patch file && test_cmp expect-file file && echo changed-other-file >expect && test_cmp expect other-file &&
-- 2.49.0.897.gfad3eb7d210