{"thread":{"id":"34281","subject":"[PATCH] am: replace uses of --resolved with --continue","startedAt":"2013-06-26T20:06:41Z","lastAt":"2013-06-27T16:36:18Z","messageCount":3,"participants":["Kevin Bracey","John Keeping","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"222054","messageId":"1372277201-29324-1-git-send-email-kevin@bracey.fi","threadId":"34281","inReplyTo":null,"subject":"[PATCH] am: replace uses of --resolved with --continue","fromName":"Kevin Bracey","fromEmail":"kevin@bracey.fi","sentAt":"2013-06-26T20:06:41Z","receivedAt":"2013-06-26T20:06:41Z","isPatch":true,"sender":{"key":"kevin@bracey.fi","avatar":"https://avatars.githubusercontent.com/u/96079793?v=4"},"body":"git am was previously modified to provide --continue for consistency\nwith rebase, merge etc, and the documentation changed to showing\n--continue as the primary form.\n\nComplete the work by replacing remaining uses of --resolved by\n--continue, most notably in suggested command reminders.\n\nSigned-off-by: Kevin Bracey <kevin@bracey.fi>\n---\n Documentation/git-am.txt      | 4 ++--\n Documentation/user-manual.txt | 2 +-\n git-am.sh                     | 8 ++++----\n t/t7512-status-help.sh        | 4 ++--\n wt-status.c                   | 2 +-\n 5 files changed, 10 insertions(+), 10 deletions(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex 5bbe7b6..54d8461 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -132,7 +132,7 @@ default.   You can use `--no-utf8` to override this.\n --resolvemsg=<msg>::\n \tWhen a patch failure occurs, <msg> will be printed\n \tto the screen before exiting.  This overrides the\n-\tstandard message informing you to use `--resolved`\n+\tstandard message informing you to use `--continue`\n \tor `--skip` to handle the failure.  This is solely\n \tfor internal use between 'git rebase' and 'git am'.\n \n@@ -176,7 +176,7 @@ aborts in the middle.  You can recover from this in one of two ways:\n \n . hand resolve the conflict in the working directory, and update\n   the index file to bring it into a state that the patch should\n-  have produced.  Then run the command with the '--resolved' option.\n+  have produced.  Then run the command with the '--continue' option.\n \n The command refuses to process new mailboxes until the current\n operation is finished, so if you decide to start over from scratch,\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex e831cc2..8218cf9 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1835,7 +1835,7 @@ Once the index is updated with the results of the conflict\n resolution, instead of creating a new commit, just run\n \n -------------------------------------------------\n-$ git am --resolved\n+$ git am --continue\n -------------------------------------------------\n \n and Git will create the commit for you and continue applying the\ndiff --git a/git-am.sh b/git-am.sh\nindex 9f44509..7ea40fe 100755\n--- a/git-am.sh\n+++ b/git-am.sh\n@@ -6,7 +6,7 @@ SUBDIRECTORY_OK=Yes\n OPTIONS_KEEPDASHDASH=\n OPTIONS_SPEC=\"\\\n git am [options] [(<mbox>|<Maildir>)...]\n-git am [options] (--resolved | --skip | --abort)\n+git am [options] (--continue | --skip | --abort)\n --\n i,interactive   run interactively\n b,binary*       (historical option -- no-op)\n@@ -102,7 +102,7 @@ stop_here_user_resolve () {\n \t    printf '%s\\n' \"$resolvemsg\"\n \t    stop_here $1\n     fi\n-    eval_gettextln \"When you have resolved this problem, run \\\"\\$cmdline --resolved\\\".\n+    eval_gettextln \"When you have resolved this problem, run \\\"\\$cmdline --continue\\\".\n If you prefer to skip this patch, run \\\"\\$cmdline --skip\\\" instead.\n To restore the original branch and stop patching, run \\\"\\$cmdline --abort\\\".\"\n \n@@ -523,7 +523,7 @@ Use \\\"git am --abort\\\" to remove it.\")\"\n \t\tesac\n \tfi\n \n-\t# Make sure we are not given --skip, --resolved, nor --abort\n+\t# Make sure we are not given --skip, --continue, nor --abort\n \ttest \"$skip$resolved$abort\" = \"\" ||\n \t\tdie \"$(gettext \"Resolve operation not in progress, we are not resuming.\")\"\n \n@@ -670,7 +670,7 @@ do\n \t#  - patch is the patch body.\n \t#\n \t# When we are resuming, these files are either already prepared\n-\t# by the user, or the user can tell us to do so by --resolved flag.\n+\t# by the user, or the user can tell us to do so by --continue flag.\n \tcase \"$resume\" in\n \t'')\n \t\tif test -f \"$dotest/rebasing\"\ndiff --git a/t/t7512-status-help.sh b/t/t7512-status-help.sh\nindex 4f09bec..bd8aab0 100755\n--- a/t/t7512-status-help.sh\n+++ b/t/t7512-status-help.sh\n@@ -510,7 +510,7 @@ test_expect_success 'status in an am session: file already exists' '\n \tcat >expected <<-\\EOF &&\n \t# On branch am_already_exists\n \t# You are in the middle of an am session.\n-\t#   (fix conflicts and then run \"git am --resolved\")\n+\t#   (fix conflicts and then run \"git am --continue\")\n \t#   (use \"git am --skip\" to skip this patch)\n \t#   (use \"git am --abort\" to restore the original branch)\n \t#\n@@ -532,7 +532,7 @@ test_expect_success 'status in an am session: file does not exist' '\n \tcat >expected <<-\\EOF &&\n \t# On branch am_not_exists\n \t# You are in the middle of an am session.\n-\t#   (fix conflicts and then run \"git am --resolved\")\n+\t#   (fix conflicts and then run \"git am --continue\")\n \t#   (use \"git am --skip\" to skip this patch)\n \t#   (use \"git am --abort\" to restore the original branch)\n \t#\ndiff --git a/wt-status.c b/wt-status.c\nindex 438a40d..b191c65 100644\n--- a/wt-status.c\n+++ b/wt-status.c\n@@ -826,7 +826,7 @@ static void show_am_in_progress(struct wt_status *s,\n \tif (advice_status_hints) {\n \t\tif (!state->am_empty_patch)\n \t\t\tstatus_printf_ln(s, color,\n-\t\t\t\t_(\"  (fix conflicts and then run \\\"git am --resolved\\\")\"));\n+\t\t\t\t_(\"  (fix conflicts and then run \\\"git am --continue\\\")\"));\n \t\tstatus_printf_ln(s, color,\n \t\t\t_(\"  (use \\\"git am --skip\\\" to skip this patch)\"));\n \t\tstatus_printf_ln(s, color,\n-- \n1.8.3.rc0.28.g4b02ef5\n"},{"id":"222068","messageId":"20130627083346.GB2232@serenity.lan","threadId":"34281","inReplyTo":"1372277201-29324-1-git-send-email-kevin@bracey.fi","subject":"Re: [PATCH] am: replace uses of --resolved with --continue","fromName":"John Keeping","fromEmail":"john@keeping.me.uk","sentAt":"2013-06-27T08:33:46Z","receivedAt":"2013-06-27T08:33:46Z","isPatch":true,"sender":{"key":"john@keeping.me.uk","avatar":"https://avatars.githubusercontent.com/u/1702081?v=4"},"body":"On Wed, Jun 26, 2013 at 11:06:41PM +0300, Kevin Bracey wrote:\n> git am was previously modified to provide --continue for consistency\n> with rebase, merge etc, and the documentation changed to showing\n> --continue as the primary form.\n> \n> Complete the work by replacing remaining uses of --resolved by\n> --continue, most notably in suggested command reminders.\n> \n> Signed-off-by: Kevin Bracey <kevin@bracey.fi>\n> ---\n>  Documentation/git-am.txt      | 4 ++--\n>  Documentation/user-manual.txt | 2 +-\n>  git-am.sh                     | 8 ++++----\n>  t/t7512-status-help.sh        | 4 ++--\n>  wt-status.c                   | 2 +-\n>  5 files changed, 10 insertions(+), 10 deletions(-)\n> \n> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\n> index 5bbe7b6..54d8461 100644\n> --- a/Documentation/git-am.txt\n> +++ b/Documentation/git-am.txt\n> @@ -132,7 +132,7 @@ default.   You can use `--no-utf8` to override this.\n>  --resolvemsg=<msg>::\n>  \tWhen a patch failure occurs, <msg> will be printed\n>  \tto the screen before exiting.  This overrides the\n> -\tstandard message informing you to use `--resolved`\n> +\tstandard message informing you to use `--continue`\n>  \tor `--skip` to handle the failure.  This is solely\n>  \tfor internal use between 'git rebase' and 'git am'.\n>  \n> @@ -176,7 +176,7 @@ aborts in the middle.  You can recover from this in one of two ways:\n>  \n>  . hand resolve the conflict in the working directory, and update\n>    the index file to bring it into a state that the patch should\n> -  have produced.  Then run the command with the '--resolved' option.\n> +  have produced.  Then run the command with the '--continue' option.\n\nIt isn't new in this patch, but there is an inconsistency in the quoting\nof the options here.  In the previous hunk we use backticks but here it\nuses SQs.\n\nThe documentation isn't at all consistent on this, but\nbackticks seem to be the preferred style (there are some false positives\nin both counts but this gives a good indication):\n\n $ git grep \"'--\" -- Documentation/ | wc -l\n 186\n $ git grep '`--' -- Documentation/ | wc -l\n 487\n\n>  The command refuses to process new mailboxes until the current\n>  operation is finished, so if you decide to start over from scratch,\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index e831cc2..8218cf9 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -1835,7 +1835,7 @@ Once the index is updated with the results of the conflict\n>  resolution, instead of creating a new commit, just run\n>  \n>  -------------------------------------------------\n> -$ git am --resolved\n> +$ git am --continue\n>  -------------------------------------------------\n>  \n>  and Git will create the commit for you and continue applying the\n> diff --git a/git-am.sh b/git-am.sh\n> index 9f44509..7ea40fe 100755\n> --- a/git-am.sh\n> +++ b/git-am.sh\n> @@ -6,7 +6,7 @@ SUBDIRECTORY_OK=Yes\n>  OPTIONS_KEEPDASHDASH=\n>  OPTIONS_SPEC=\"\\\n>  git am [options] [(<mbox>|<Maildir>)...]\n> -git am [options] (--resolved | --skip | --abort)\n> +git am [options] (--continue | --skip | --abort)\n>  --\n>  i,interactive   run interactively\n>  b,binary*       (historical option -- no-op)\n> @@ -102,7 +102,7 @@ stop_here_user_resolve () {\n>  \t    printf '%s\\n' \"$resolvemsg\"\n>  \t    stop_here $1\n>      fi\n> -    eval_gettextln \"When you have resolved this problem, run \\\"\\$cmdline --resolved\\\".\n> +    eval_gettextln \"When you have resolved this problem, run \\\"\\$cmdline --continue\\\".\n>  If you prefer to skip this patch, run \\\"\\$cmdline --skip\\\" instead.\n>  To restore the original branch and stop patching, run \\\"\\$cmdline --abort\\\".\"\n>  \n> @@ -523,7 +523,7 @@ Use \\\"git am --abort\\\" to remove it.\")\"\n>  \t\tesac\n>  \tfi\n>  \n> -\t# Make sure we are not given --skip, --resolved, nor --abort\n> +\t# Make sure we are not given --skip, --continue, nor --abort\n>  \ttest \"$skip$resolved$abort\" = \"\" ||\n>  \t\tdie \"$(gettext \"Resolve operation not in progress, we are not resuming.\")\"\n>  \n> @@ -670,7 +670,7 @@ do\n>  \t#  - patch is the patch body.\n>  \t#\n>  \t# When we are resuming, these files are either already prepared\n> -\t# by the user, or the user can tell us to do so by --resolved flag.\n> +\t# by the user, or the user can tell us to do so by --continue flag.\n>  \tcase \"$resume\" in\n>  \t'')\n>  \t\tif test -f \"$dotest/rebasing\"\n> diff --git a/t/t7512-status-help.sh b/t/t7512-status-help.sh\n> index 4f09bec..bd8aab0 100755\n> --- a/t/t7512-status-help.sh\n> +++ b/t/t7512-status-help.sh\n> @@ -510,7 +510,7 @@ test_expect_success 'status in an am session: file already exists' '\n>  \tcat >expected <<-\\EOF &&\n>  \t# On branch am_already_exists\n>  \t# You are in the middle of an am session.\n> -\t#   (fix conflicts and then run \"git am --resolved\")\n> +\t#   (fix conflicts and then run \"git am --continue\")\n>  \t#   (use \"git am --skip\" to skip this patch)\n>  \t#   (use \"git am --abort\" to restore the original branch)\n>  \t#\n> @@ -532,7 +532,7 @@ test_expect_success 'status in an am session: file does not exist' '\n>  \tcat >expected <<-\\EOF &&\n>  \t# On branch am_not_exists\n>  \t# You are in the middle of an am session.\n> -\t#   (fix conflicts and then run \"git am --resolved\")\n> +\t#   (fix conflicts and then run \"git am --continue\")\n>  \t#   (use \"git am --skip\" to skip this patch)\n>  \t#   (use \"git am --abort\" to restore the original branch)\n>  \t#\n> diff --git a/wt-status.c b/wt-status.c\n> index 438a40d..b191c65 100644\n> --- a/wt-status.c\n> +++ b/wt-status.c\n> @@ -826,7 +826,7 @@ static void show_am_in_progress(struct wt_status *s,\n>  \tif (advice_status_hints) {\n>  \t\tif (!state->am_empty_patch)\n>  \t\t\tstatus_printf_ln(s, color,\n> -\t\t\t\t_(\"  (fix conflicts and then run \\\"git am --resolved\\\")\"));\n> +\t\t\t\t_(\"  (fix conflicts and then run \\\"git am --continue\\\")\"));\n>  \t\tstatus_printf_ln(s, color,\n>  \t\t\t_(\"  (use \\\"git am --skip\\\" to skip this patch)\"));\n>  \t\tstatus_printf_ln(s, color,\n> -- \n> 1.8.3.rc0.28.g4b02ef5\n"},{"id":"222095","messageId":"7vvc4ztssd.fsf@alter.siamese.dyndns.org","threadId":"34281","inReplyTo":"20130627083346.GB2232@serenity.lan","subject":"Re: [PATCH] am: replace uses of --resolved with --continue","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-06-27T16:36:18Z","receivedAt":"2013-06-27T16:36:18Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Keeping <john@keeping.me.uk> writes:\n\n>> @@ -176,7 +176,7 @@ aborts in the middle.  You can recover from this in one of two ways:\n>>  \n>>  . hand resolve the conflict in the working directory, and update\n>>    the index file to bring it into a state that the patch should\n>> -  have produced.  Then run the command with the '--resolved' option.\n>> +  have produced.  Then run the command with the '--continue' option.\n>\n> It isn't new in this patch, but there is an inconsistency in the quoting\n> of the options here.  In the previous hunk we use backticks but here it\n> uses SQs.\n\nWhatever the user is expected to literally type, like the dashed options,\nshould be typeset in tt so `--continue` is the right form.\n\nI am not inviting anybody to throw a patch that was generated by\nblindly running:\n\n\tsed -e \"s/'\\(--[a-z_]*\\)'/`\\1`/g\"\n\nto touch 50+ files without thinking, by the way.  It is so easy that\nit can even be done by a trained monkey, but it also is very easy to\nfail to spot misconversion.  Done in small batches with a throttled\nrate of patchflow, we may have sufficient review bandwidth to audit\nthem.\n"}]}