Re: git cherry-pick --continue?
From: Jeff King <hidden>
Date: 2016-06-15 22:48:13
Subsystem:
documentation, the rest · Maintainers:
Jonathan Corbet, Linus Torvalds
Possibly related (same subject, not in this thread)
- 2016-06-15 · git cherry-pick --continue? · Sverre Rabbelier <hidden>
- 2016-06-15 · Re: git cherry-pick --continue? · Jeff King <hidden>
- 2016-06-15 · Re: git cherry-pick --continue? · Jeff King <hidden>
- 2016-06-15 · Re: git cherry-pick --continue? · Sverre Rabbelier <hidden>
- 2016-06-15 · Re: git cherry-pick --continue? · Junio C Hamano <hidden>
On Thu, Feb 11, 2010 at 12:36:52PM -0800, Junio C Hamano wrote:
quoted
Do you just want to pick up my patch from earlier in the thread, or do you have further comments? The only thing I could think to change would be that we may not want to even bother advertising --continue in the usage message (conversely, we could go a step further and actually advertise it in the manpage).I would say our eventual goal should be to make "--continue" the primary word the end users would see. It would bring us closer to that goal to start advertising --continue early.
OK. Then I think my patch is fine. But we could also do this if we wanted to push it further now: -- >8 -- Subject: [PATCH] am: switch --resolved to --continue Rebase calls this same function "--continue", which means users may be trained to type it. There is no reason to deprecate --resolved (or -r), so we will keep it as a synonym. Signed-off-by: Jeff King <redacted> --- Between this and the previous patch, I don't have a strong preference. You can decide. Documentation/git-am.txt | 3 ++- git-am.sh | 5 +++-- 2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt
index c3e4f12..c66c565 100644
--- a/Documentation/git-am.txt
+++ b/Documentation/git-am.txt@@ -15,7 +15,7 @@ SYNOPSIS [--whitespace=<option>] [-C<n>] [-p<n>] [--directory=<dir>] [--reject] [-q | --quiet] [--scissors | --no-scissors] [<mbox> | <Maildir>...] -'git am' (--skip | --resolved | --abort) +'git am' (--continue | --skip | --abort) DESCRIPTION -----------
@@ -107,6 +107,7 @@ default. You can use `--no-utf8` to override this. Skip the current patch. This is only meaningful when restarting an aborted patch. +--continue:: -r:: --resolved:: After a patch failure (e.g. attempting to apply
diff --git a/git-am.sh b/git-am.sh
index c8b9cbb..3c08d53 100755
--- a/git-am.sh
+++ b/git-am.sh@@ -25,7 +25,8 @@ p= pass it through git-apply patch-format= format the patch(es) are in reject pass it through git-apply resolvemsg= override error message when patch failure occurs -r,resolved to be used after a patch failure +continue continue applying patches after resolving a conflict +r,resolved synonyms for --continue skip skip the current patch abort restore the original branch and abort the patching operation. committer-date-is-author-date lie about committer date
@@ -318,7 +319,7 @@ do scissors=t ;; --no-scissors) scissors=f ;; - -r|--resolved) + -r|--resolved|--continue) resolved=t ;; --skip) skip=t ;;
--
1.7.0.rc2.37.g157e8.dirty