{"thread":{"id":"44026","subject":"[PATCH] rebase -i: improve advice on bad instruction lines","startedAt":"2016-09-06T18:08:48Z","lastAt":"2016-09-09T12:21:58Z","messageCount":7,"participants":["Ralf Thielow","Dennis Kaarsemaker","Junio C Hamano","Johannes Schindelin"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"301217","messageId":"20160906180838.865-1-ralf.thielow@gmail.com","threadId":"44026","inReplyTo":null,"subject":"[PATCH] rebase -i: improve advice on bad instruction lines","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2016-09-06T18:08:38Z","receivedAt":"2016-09-06T18:08:48Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"If we found bad instruction lines in the instruction sheet\nof interactive rebase, we give the user advice on how to\nfix it.  However, we don't tell the user what to do afterwards.\nGive the user advice to run 'git rebase --continue' after\nthe fix.\n\nSigned-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n---\n git-rebase--interactive.sh | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh\nindex b1ba21c..029594e 100644\n--- a/git-rebase--interactive.sh\n+++ b/git-rebase--interactive.sh\n@@ -1041,7 +1041,7 @@ The possible behaviours are: ignore, warn, error.\")\"\n \t\t# placed before the commit of the next action\n \t\tcheckout_onto\n \n-\t\twarn \"$(gettext \"You can fix this with 'git rebase --edit-todo'.\")\"\n+\t\twarn \"$(gettext \"You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\")\"\n \t\tdie \"$(gettext \"Or you can abort the rebase with 'git rebase --abort'.\")\"\n \tfi\n }\n-- \n2.10.0.304.gf2ff484\n\n"},{"id":"301223","messageId":"CAN0XMOJofQwsPJvoHLZJ-fT6SudtquatYV=GwpGb6FkF4aXcxg@mail.gmail.com","threadId":"44026","inReplyTo":"20160906180838.865-1-ralf.thielow@gmail.com","subject":"Re: [PATCH] rebase -i: improve advice on bad instruction lines","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2016-09-06T18:50:10Z","receivedAt":"2016-09-06T18:50:17Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"2016-09-06 20:08 GMT+02:00 Ralf Thielow <ralf.thielow@gmail.com>:\n> -               warn \"$(gettext \"You can fix this with 'git rebase --edit-todo'.\")\"\n> +               warn \"$(gettext \"You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\")\"\n>                 die \"$(gettext \"Or you can abort the rebase with 'git rebase --abort'.\")\"\n\nPlease don't apply as is. There are some test failures due to the text change.\nI'll send an updated version.\n"},{"id":"301225","messageId":"20160906185918.21286-1-ralf.thielow@gmail.com","threadId":"44026","inReplyTo":"20160906180838.865-1-ralf.thielow@gmail.com","subject":"[PATCH v2] rebase -i: improve advice on bad instruction lines","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2016-09-06T18:59:18Z","receivedAt":"2016-09-06T19:00:40Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"If we found bad instruction lines in the instruction sheet\nof interactive rebase, we give the user advice on how to\nfix it.  However, we don't tell the user what to do afterwards.\nGive the user advice to run 'git rebase --continue' after\nthe fix.\n\nSigned-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n---\nChanges in v2:\n- adjust tests\n\n git-rebase--interactive.sh    | 2 +-\n t/t3404-rebase-interactive.sh | 6 +++---\n 2 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh\nindex b1ba21c..029594e 100644\n--- a/git-rebase--interactive.sh\n+++ b/git-rebase--interactive.sh\n@@ -1041,7 +1041,7 @@ The possible behaviours are: ignore, warn, error.\")\"\n \t\t# placed before the commit of the next action\n \t\tcheckout_onto\n \n-\t\twarn \"$(gettext \"You can fix this with 'git rebase --edit-todo'.\")\"\n+\t\twarn \"$(gettext \"You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\")\"\n \t\tdie \"$(gettext \"Or you can abort the rebase with 'git rebase --abort'.\")\"\n \tfi\n }\ndiff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh\nindex 597e94e..e38e296 100755\n--- a/t/t3404-rebase-interactive.sh\n+++ b/t/t3404-rebase-interactive.sh\n@@ -1195,7 +1195,7 @@ To avoid this message, use \"drop\" to explicitly remove a commit.\n Use 'git config rebase.missingCommitsCheck' to change the level of warnings.\n The possible behaviours are: ignore, warn, error.\n \n-You can fix this with 'git rebase --edit-todo'.\n+You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\n Or you can abort the rebase with 'git rebase --abort'.\n EOF\n \n@@ -1219,7 +1219,7 @@ cat >expect <<EOF\n Warning: the command isn't recognized in the following line:\n  - badcmd $(git rev-list --oneline -1 master~1)\n \n-You can fix this with 'git rebase --edit-todo'.\n+You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\n Or you can abort the rebase with 'git rebase --abort'.\n EOF\n \n@@ -1254,7 +1254,7 @@ cat >expect <<EOF\n Warning: the SHA-1 is missing or isn't a commit in the following line:\n  - edit XXXXXXX False commit\n \n-You can fix this with 'git rebase --edit-todo'.\n+You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.\n Or you can abort the rebase with 'git rebase --abort'.\n EOF\n \n-- \n2.10.0.304.gf2ff484.dirty\n\n"},{"id":"301249","messageId":"1473240491.20551.1.camel@kaarsemaker.net","threadId":"44026","inReplyTo":"20160906185918.21286-1-ralf.thielow@gmail.com","subject":"Re: [PATCH v2] rebase -i: improve advice on bad instruction lines","fromName":"Dennis Kaarsemaker","fromEmail":"dennis@kaarsemaker.net","sentAt":"2016-09-07T09:28:11Z","receivedAt":"2016-09-07T09:28:20Z","isPatch":true,"sender":{"key":"dennis@kaarsemaker.net","avatar":"https://avatars.githubusercontent.com/u/200649?v=4"},"body":"Hi Ralf,\n\nThere are quite a few patch series in flight these days around\ninteractive rebase. Have you checked for conflicts with those?\n\nOn di, 2016-09-06 at 20:59 +0200, Ralf Thielow wrote:\n> If we found bad instruction lines in the instruction sheet\n> of interactive rebase, we give the user advice on how to\n> fix it.  However, we don't tell the user what to do afterwards.\n> Give the user advice to run 'git rebase --continue' after\n> the fix.\n> \n> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n> ---\n> Changes in v2:\n> - adjust tests\n> \n>  git-rebase--interactive.sh    | 2 +-\n>  t/t3404-rebase-interactive.sh | 6 +++---\n>  2 files changed, 4 insertions(+), 4 deletions(-)\n> \n> diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh\n> index b1ba21c..029594e 100644\n> --- a/git-rebase--interactive.sh\n> +++ b/git-rebase--interactive.sh\n> @@ -1041,7 +1041,7 @@ The possible behaviours are: ignore, warn,\n> error.\")\"\n>  \t\t# placed before the commit of the next action\n>  \t\tcheckout_onto\n>  \n> -\t\twarn \"$(gettext \"You can fix this with 'git rebase\n> --edit-todo'.\")\"\n> +\t\twarn \"$(gettext \"You can fix this with 'git rebase\n> --edit-todo' and then run 'git rebase --continue'.\")\"\n>  \t\tdie \"$(gettext \"Or you can abort the rebase with\n> 'git rebase --abort'.\")\"\n>  \tfi\n>  }\n> diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-\n> interactive.sh\n> index 597e94e..e38e296 100755\n> --- a/t/t3404-rebase-interactive.sh\n> +++ b/t/t3404-rebase-interactive.sh\n> @@ -1195,7 +1195,7 @@ To avoid this message, use \"drop\" to explicitly\n> remove a commit.\n>  Use 'git config rebase.missingCommitsCheck' to change the level of\n> warnings.\n>  The possible behaviours are: ignore, warn, error.\n>  \n> -You can fix this with 'git rebase --edit-todo'.\n> +You can fix this with 'git rebase --edit-todo' and then run 'git\n> rebase --continue'.\n>  Or you can abort the rebase with 'git rebase --abort'.\n>  EOF\n>  \n> @@ -1219,7 +1219,7 @@ cat >expect <<EOF\n>  Warning: the command isn't recognized in the following line:\n>   - badcmd $(git rev-list --oneline -1 master~1)\n>  \n> -You can fix this with 'git rebase --edit-todo'.\n> +You can fix this with 'git rebase --edit-todo' and then run 'git\n> rebase --continue'.\n>  Or you can abort the rebase with 'git rebase --abort'.\n>  EOF\n>  \n> @@ -1254,7 +1254,7 @@ cat >expect <<EOF\n>  Warning: the SHA-1 is missing or isn't a commit in the following\n> line:\n>   - edit XXXXXXX False commit\n>  \n> -You can fix this with 'git rebase --edit-todo'.\n> +You can fix this with 'git rebase --edit-todo' and then run 'git\n> rebase --continue'.\n>  Or you can abort the rebase with 'git rebase --abort'.\n>  EOF\n>  \n"},{"id":"301295","messageId":"CAN0XMO+diDLXUrxjNGnUi6oUEwkcGavEX8vyGWy2+Wju-N4WJQ@mail.gmail.com","threadId":"44026","inReplyTo":"1473240491.20551.1.camel@kaarsemaker.net","subject":"Re: [PATCH v2] rebase -i: improve advice on bad instruction lines","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2016-09-07T17:07:47Z","receivedAt":"2016-09-07T17:07:53Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"2016-09-07 11:28 GMT+02:00 Dennis Kaarsemaker <dennis@kaarsemaker.net>:\n> Hi Ralf,\n>\n> There are quite a few patch series in flight these days around\n> interactive rebase. Have you checked for conflicts with those?\n>\n\nThanks. I did not check against 'pu' when I created this patch but I'm able\nto apply without any conflicts.\n"},{"id":"301315","messageId":"xmqqr38vleps.fsf@gitster.mtv.corp.google.com","threadId":"44026","inReplyTo":"1473240491.20551.1.camel@kaarsemaker.net","subject":"Re: [PATCH v2] rebase -i: improve advice on bad instruction lines","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-09-07T18:58:07Z","receivedAt":"2016-09-07T18:58:14Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Dennis Kaarsemaker <dennis@kaarsemaker.net> writes:\n\n> There are quite a few patch series in flight these days around\n> interactive rebase. Have you checked for conflicts with those?\n\nIIRC, Ralf's fix would need to be ported to C version, and when it\nhappens, it won't just be a mere conflict but will disappear from\nthe scripted version ;-)\n\nBut the C version is not anywhere near 'pu' yet and being rerolled,\nso I do not mind queuing it on 'pu' now merely as a reminder.\n"},{"id":"301486","messageId":"alpine.DEB.2.20.1609091421010.129229@virtualbox","threadId":"44026","inReplyTo":"CAN0XMO+diDLXUrxjNGnUi6oUEwkcGavEX8vyGWy2+Wju-N4WJQ@mail.gmail.com","subject":"Re: [PATCH v2] rebase -i: improve advice on bad instruction lines","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2016-09-09T12:21:45Z","receivedAt":"2016-09-09T12:21:58Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Ralf,\n\nOn Wed, 7 Sep 2016, Ralf Thielow wrote:\n\n> 2016-09-07 11:28 GMT+02:00 Dennis Kaarsemaker <dennis@kaarsemaker.net>:\n> > Hi Ralf,\n> >\n> > There are quite a few patch series in flight these days around\n> > interactive rebase. Have you checked for conflicts with those?\n> >\n> \n> Thanks. I did not check against 'pu' when I created this patch but I'm able\n> to apply without any conflicts.\n\nYeah, rewrites in C won't conflict with your patches. If you rebase\ninteractively, the worst that will happen is that you update code that was\nmoved into contrib/...\n\nCiao,\nJohannes\n"}]}