{"thread":{"id":"46737","subject":"[PATCH] commit-template: change a message to be more intuitive","startedAt":"2017-09-12T10:55:47Z","lastAt":"2017-09-15T12:40:04Z","messageCount":15,"participants":["Kaartic Sivaraam","Kevin Daudt","Jeff King","Michael J Gruber","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"327882","messageId":"20170912105536.6094-1-kaarticsivaraam91196@gmail.com","threadId":"46737","inReplyTo":null,"subject":"[PATCH] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-12T10:55:36Z","receivedAt":"2017-09-12T10:55:47Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"It's not possible to 'touch' the cut-line that is shown when the\nuser requests a patch in his commit template.\n\nSo, make the sentence more intuitive.\n\nSigned-off-by: Kaartic Sivaraam \n---\n Just a small tweak. May or may not be worth the patch.\n\n wt-status.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/wt-status.c b/wt-status.c\nindex 77c27c511..1f54b1df2 100644\n--- a/wt-status.c\n+++ b/wt-status.c\n@@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n \n void wt_status_add_cut_line(FILE *fp)\n {\n-\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n+\tconst char *explanation = _(\"Do not edit the line above.\\nEverything below will be removed.\");\n \tstruct strbuf buf = STRBUF_INIT;\n \n \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n-- \n2.14.1.1006.g90ad9a07c\n\n"},{"id":"327919","messageId":"20170913102915.GA21499@alpha.vpn.ikke.info","threadId":"46737","inReplyTo":"20170912105536.6094-1-kaarticsivaraam91196@gmail.com","subject":"Re: [PATCH] commit-template: change a message to be more intuitive","fromName":"Kevin Daudt","fromEmail":"me@ikke.info","sentAt":"2017-09-13T10:29:15Z","receivedAt":"2017-09-13T10:44:51Z","isPatch":true,"sender":{"key":"me@ikke.info","avatar":"https://avatars.githubusercontent.com/u/135698?v=4"},"body":"On Tue, Sep 12, 2017 at 04:25:36PM +0530, Kaartic Sivaraam wrote:\n> It's not possible to 'touch' the cut-line that is shown when the\n> user requests a patch in his commit template.\n> \n\nTouching something can also mean to disturb or change something, which\nis the meaning being used here, so it is not an incorrect use of the\nword.\n\n> So, make the sentence more intuitive.\n\nI can understand however that it might be not so clear for people with\nless fluency in English.\n\n> \n> Signed-off-by: Kaartic Sivaraam \n> ---\n> Just a small tweak. May or may not be worth the patch.\n> \n> wt-status.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/wt-status.c b/wt-status.c\n> index 77c27c511..1f54b1df2 100644\n> --- a/wt-status.c\n> +++ b/wt-status.c\n> @@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n> \n> void wt_status_add_cut_line(FILE *fp)\n> {\n> -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n> +\tconst char *explanation = _(\"Do not edit the line above.\\nEverything below will be removed.\");\n> \tstruct strbuf buf = STRBUF_INIT;\n> \n> \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n> -- \n> 2.14.1.1006.g90ad9a07c\n> \n"},{"id":"327922","messageId":"20170913112013.sk7bqarq2ktrva67@sigill.intra.peff.net","threadId":"46737","inReplyTo":"20170913102915.GA21499@alpha.vpn.ikke.info","subject":"Re: [PATCH] commit-template: change a message to be more intuitive","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-09-13T11:20:13Z","receivedAt":"2017-09-13T11:20:23Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Sep 13, 2017 at 12:29:15PM +0200, Kevin Daudt wrote:\n\n> On Tue, Sep 12, 2017 at 04:25:36PM +0530, Kaartic Sivaraam wrote:\n> > It's not possible to 'touch' the cut-line that is shown when the\n> > user requests a patch in his commit template.\n> > \n> \n> Touching something can also mean to disturb or change something, which\n> is the meaning being used here, so it is not an incorrect use of the\n> word.\n> \n> > So, make the sentence more intuitive.\n> \n> I can understand however that it might be not so clear for people with\n> less fluency in English.\n\nI agree with both of your points. It is very clear to me as a native\nspeaker, but I can see how it might not be for everyone.\n\nInterestingly, the change here:\n\n> > -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n> > +\tconst char *explanation = _(\"Do not edit the line above.\\nEverything below will be removed.\");\n\nactually seems less clear to me. I think of \"edit\" as \"modify\". But\nobviously it also should not be removed. Perhaps\n\n Do not modify or remove the line above.\n\nwould be the most clear. Or perhaps it is overly verbose.\n\n-Peff\n"},{"id":"327926","messageId":"1d400615-be2c-7ed7-9687-d78ab163eaea@gmail.com","threadId":"46737","inReplyTo":"20170913102915.GA21499@alpha.vpn.ikke.info","subject":"Re: [PATCH] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-13T12:29:03Z","receivedAt":"2017-09-13T12:29:22Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Wednesday 13 September 2017 03:59 PM, Kevin Daudt wrote:\n> Touching something can also mean to disturb or change something, which\n> is the meaning being used here, so it is not an incorrect use of the\n> word.\n\nI do get your point but I should have been clearer in my commit message \nabout the\nfact that it's a word that could be misinterpreted.\n\n>> So, make the sentence more intuitive.\n> I can understand however that it might be not so clear for people with\n> less fluency in English.\n\nNot just that, it could also make the translations a little unclear. \ni.e., the when the words that convey\nclear meaning in a particular context in english speaker get translated \n\"as such\" to another language,\nthey might turn out to become unclear or might even convey a totally \ndifferent meaning. I'm not sure\nif is currently happening as I am not familiar with any languages for \nwhich translations are currently\navailable but I'm pretty sure that the sentence \"Do not touch the line \nabove.\" would not give a clear\nmeaning when it gets translated \"as such\" to my native language (Tamil). \nOf course, it's only a problem\nif the sentences get translated \"as such\" but I'm not sure if \ntranslators (human/non-human) try do\nmeaningful translations or try to preserve a one-to-one relationship \nwith the words in the source\nlanguage, either.\n\n---\nKaartic\n"},{"id":"327927","messageId":"624d873b-74af-b3e8-714c-8a439f4ee81f@gmail.com","threadId":"46737","inReplyTo":"20170913112013.sk7bqarq2ktrva67@sigill.intra.peff.net","subject":"Re: [PATCH] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-13T12:31:48Z","receivedAt":"2017-09-13T12:31:49Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Wednesday 13 September 2017 04:50 PM, Jeff King wrote:\n> I agree with both of your points. It is very clear to me as a native\n> speaker, but I can see how it might not be for everyone.\n>\n> Interestingly, the change here:\n>\n>>> -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n>>> +\tconst char *explanation = _(\"Do not edit the line above.\\nEverything below will be removed.\");\n> actually seems less clear to me. I think of \"edit\" as \"modify\". But\n> obviously it also should not be removed. Perhaps\n>\n> Do not modify or remove the line above.\n>\n> would be the most clear. Or perhaps it is overly verbose.\n>\nSeems to be a valid point. Further, it doesn't seem to be too verbose at \nleast to me. So, I'll change the\nsentence and revert back to the one I use if someone finds it to be verbose.\n\n---\nKaartic\n"},{"id":"327932","messageId":"e656d1b4-c95e-e51d-5f48-bb3450580fa4@gmail.com","threadId":"46737","inReplyTo":"624d873b-74af-b3e8-714c-8a439f4ee81f@gmail.com","subject":"[PATCH v2] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-13T13:05:00Z","receivedAt":"2017-09-13T13:05:03Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"Changes in v2:\n\n - Changed the message as suggested by Jeff\n\n - Made the commit message to be even more clear\n\n\n---\n\nKaartic\n\n"},{"id":"327933","messageId":"0102015e7b585037-5f88e775-2696-49ec-8dd9-5e1635a34607-000000@eu-west-1.amazonses.com","threadId":"46737","inReplyTo":"e656d1b4-c95e-e51d-5f48-bb3450580fa4@gmail.com","subject":"[PATCH v2] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-13T13:05:38Z","receivedAt":"2017-09-13T13:05:44Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"It's not good to use the phrase 'do not touch' to convey the information\nthat the cut-line should not be modified or removed as it could possibly\nbe mis-interpreted by a person who doesn't know that the word 'touch' has\nthe meaning of 'tamper with'. Further, it could make translations a little\ndifficult as it might not have the intended meaning in a few languages when\ntranslated as such.\n\nSo, use more intuitive terms in the sentence.\n\nSigned-off-by: Kaartic Sivaraam \n---\n wt-status.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/wt-status.c b/wt-status.c\nindex 77c27c51134d2..be53579760ee7 100644\n--- a/wt-status.c\n+++ b/wt-status.c\n@@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n \n void wt_status_add_cut_line(FILE *fp)\n {\n-\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n+\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below will be removed.\");\n \tstruct strbuf buf = STRBUF_INIT;\n \n \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n\n--\nhttps://github.com/git/git/pull/401\n"},{"id":"328028","messageId":"862e71a1-2dee-4a20-d828-69a0547fa5ec@grubix.eu","threadId":"46737","inReplyTo":"0102015e7b585037-5f88e775-2696-49ec-8dd9-5e1635a34607-000000@eu-west-1.amazonses.com","subject":"Re: [PATCH v2] commit-template: change a message to be more intuitive","fromName":"Michael J Gruber","fromEmail":"git@grubix.eu","sentAt":"2017-09-14T07:36:36Z","receivedAt":"2017-09-14T07:36:48Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Kaartic Sivaraam venit, vidit, dixit 13.09.2017 15:05:\n> It's not good to use the phrase 'do not touch' to convey the information\n> that the cut-line should not be modified or removed as it could possibly\n> be mis-interpreted by a person who doesn't know that the word 'touch' has\n> the meaning of 'tamper with'. Further, it could make translations a little\n> difficult as it might not have the intended meaning in a few languages when\n> translated as such.\n> \n> So, use more intuitive terms in the sentence.\n> \n> Signed-off-by: Kaartic Sivaraam \n> ---\n> wt-status.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/wt-status.c b/wt-status.c\n> index 77c27c51134d2..be53579760ee7 100644\n> --- a/wt-status.c\n> +++ b/wt-status.c\n> @@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n> \n> void wt_status_add_cut_line(FILE *fp)\n> {\n> -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n> +\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below will be removed.\");\n\nI don't want to complicate things. But now - due to the repeated usage\nof \"remove\" - these two sentences seem to be connected by an invisible\n\"or else\" (\"will\" vs. \"would\" not withstanding). i.e. \"or else\neverything below will be removed, too\", which is wrong. Before, they\nwere separated more clearly.\n\nAlso, given all the translations that we have, it seems somewhat strange\nto try and foresee and workaround possible misunderstandings of commonly\nused English phrases.\n\n> \tstruct strbuf buf = STRBUF_INIT;\n> \n> \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n> \n> --\n> https://github.com/git/git/pull/401\n> \n\n"},{"id":"328053","messageId":"2480c33b-7e44-5c7c-cd31-feae9168b9dc@gmail.com","threadId":"46737","inReplyTo":"862e71a1-2dee-4a20-d828-69a0547fa5ec@grubix.eu","subject":"Re: [PATCH v2] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-14T14:17:51Z","receivedAt":"2017-09-14T14:17:50Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Thursday 14 September 2017 01:06 PM, Michael J Gruber wrote:\n> Kaartic Sivaraam venit, vidit, dixit 13.09.2017 15:05:\n>> void wt_status_add_cut_line(FILE *fp)\n>> {\n>> -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n>> +\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below will be removed.\");\n> I don't want to complicate things. But now - due to the repeated usage\n> of \"remove\" - these two sentences seem to be connected by an invisible\n> \"or else\" (\"will\" vs. \"would\" not withstanding). i.e. \"or else\n> everything below will be removed, too\", which is wrong. Before, they\n> were separated more clearly.\n\nAt least I don't think they are connected by an invisible \"or else\" because\nthey appear on different lines,\n\n #\n\n # ------------------------ >8 ------------------------\n\n # Do not modify or remove the line above.\n\n # Everything below will be removed.\n\n diff --git a/hook-checks/hooks/applypatch-msg b/hook-checks/hooks/applypatch-msg\n\n new file mode 100755\n\n index 0000000..a5d7b84\n\n --- /dev/null\n\n +++ b/hook-checks/hooks/applypatch-msg\n\n> Also, given all the translations that we have, it seems somewhat strange\n> to try and foresee and workaround possible misunderstandings of commonly\n> used English phrases.\nI thought the translators wouldn't mind this! If they do, then it's \nbetter to drop\nthis patch.\n\n---\nKaartic\n"},{"id":"328058","messageId":"20170914144120.oqx2trhccweiu7qo@sigill.intra.peff.net","threadId":"46737","inReplyTo":"2480c33b-7e44-5c7c-cd31-feae9168b9dc@gmail.com","subject":"Re: [PATCH v2] commit-template: change a message to be more intuitive","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-09-14T14:41:20Z","receivedAt":"2017-09-14T14:41:26Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Sep 14, 2017 at 07:47:51PM +0530, Kaartic Sivaraam wrote:\n\n> > > -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n> > > +\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below will be removed.\");\n> > I don't want to complicate things. But now - due to the repeated usage\n> > of \"remove\" - these two sentences seem to be connected by an invisible\n> > \"or else\" (\"will\" vs. \"would\" not withstanding). i.e. \"or else\n> > everything below will be removed, too\", which is wrong. Before, they\n> > were separated more clearly.\n> \n> At least I don't think they are connected by an invisible \"or else\" because\n> they appear on different lines,\n\nYou could switch out the second \"removed\" for \"ignored\", which I think\nhas the same meaning but avoids repeating the same word. So:\n\n # ------------------------ >8 ------------------------\n # Do not modify or remove the line above.\n # Everything below will be ignored.\n\n(I might say \"everything below it\" to be more precise).\n\nI dunno. We are deep in bikeshed territory here, and I admit I don't\noverly care that much in the first place.\n\n-Peff\n"},{"id":"328091","messageId":"1505440687.1862.15.camel@gmail.com","threadId":"46737","inReplyTo":"862e71a1-2dee-4a20-d828-69a0547fa5ec@grubix.eu","subject":"Re: [PATCH v2] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-15T01:58:07Z","receivedAt":"2017-09-15T01:58:16Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Thu, 2017-09-14 at 09:36 +0200, Michael J Gruber wrote:\n> \n> Also, given all the translations that we have, it seems somewhat strange\n> to try and foresee and workaround possible misunderstandings of commonly\n> used English phrases.\n\nIn case you're trying to say that \"Translators have successfully\ntranslated the phrase with misunderstanding for many languages\" then I\nhave to say that I foresaw the difficulty to translate it in other\nlanguages for which a translation does not exist yet (specifically my\nmother tongue). If that's not useful as it might only be an rare case,\nthen this patch isn't either.\n\n-- \nKaartic\n"},{"id":"328092","messageId":"xmqqefr8zouy.fsf@gitster.mtv.corp.google.com","threadId":"46737","inReplyTo":"20170914144120.oqx2trhccweiu7qo@sigill.intra.peff.net","subject":"Re: [PATCH v2] commit-template: change a message to be more intuitive","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-09-15T02:18:13Z","receivedAt":"2017-09-15T02:21:09Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> # ------------------------ >8 ------------------------\n> # Do not modify or remove the line above.\n> # Everything below will be ignored.\n>\n> (I might say \"everything below it\" to be more precise).\n>\n> I dunno. We are deep in bikeshed territory here, and I admit I don't\n> overly care that much in the first place.\n\nSo should I drop this topic altogether? My inclination is to\ndeclare that \"everything below it will be ignored\" is the best\nversion and stop reading further updates to this thread ;-)\n\n"},{"id":"328100","messageId":"20170915045025.4272-1-kaarticsivaraam91196@gmail.com","threadId":"46737","inReplyTo":"xmqqefr8zouy.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v3] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-15T04:50:25Z","receivedAt":"2017-09-15T04:50:49Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"It's not good to use the phrase 'do not touch' to convey the information\nthat the cut-line should not be modified or removed as it could possibly\nbe mis-interpreted by a person who doesn't know that the word 'touch' has\nthe meaning of 'tamper with'. Further, it could make translations a little\ndifficult as it might not have the intended meaning in a few languages (for\nwhich translations don't exist yet) when translated as such.\n\nSo, use intuitive terms in the sentence. Replacing the word 'touch' with\nother terms has introduced the possibility of the following sentence to be\nmis-interpreted, so change the terms in that too.\n\nSigned-off-by: Kaartic Sivaraam \n---\n Changes in v3:\n \n - changed the wordings of the second sentence as there seemed to be a\n magical 'or else' connecting the two lines.\n \n I didn't expect the least that this would go upto v3. In case anyboy finds\n something wrong with this change too, it's a lot better to drop this altogether\n than going for a v4.\n\n wt-status.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/wt-status.c b/wt-status.c\nindex 77c27c511..23e87e74d 100644\n--- a/wt-status.c\n+++ b/wt-status.c\n@@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n \n void wt_status_add_cut_line(FILE *fp)\n {\n-\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n+\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below it will be ignored.\");\n \tstruct strbuf buf = STRBUF_INIT;\n \n \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n-- \n2.14.1.1006.g90ad9a07c\n\n"},{"id":"328111","messageId":"a8bfb8dc-583f-3126-bbe7-55cb845f1037@grubix.eu","threadId":"46737","inReplyTo":"20170915045025.4272-1-kaarticsivaraam91196@gmail.com","subject":"Re: [PATCH v3] commit-template: change a message to be more intuitive","fromName":"Michael J Gruber","fromEmail":"git@grubix.eu","sentAt":"2017-09-15T10:00:20Z","receivedAt":"2017-09-15T10:00:35Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Kaartic Sivaraam venit, vidit, dixit 15.09.2017 06:50:\n> It's not good to use the phrase 'do not touch' to convey the information\n> that the cut-line should not be modified or removed as it could possibly\n> be mis-interpreted by a person who doesn't know that the word 'touch' has\n> the meaning of 'tamper with'. Further, it could make translations a little\n> difficult as it might not have the intended meaning in a few languages (for\n> which translations don't exist yet) when translated as such.\n> \n> So, use intuitive terms in the sentence. Replacing the word 'touch' with\n> other terms has introduced the possibility of the following sentence to be\n> mis-interpreted, so change the terms in that too.\n> \n> Signed-off-by: Kaartic Sivaraam \n> ---\n> Changes in v3:\n> \n> - changed the wordings of the second sentence as there seemed to be a\n> magical 'or else' connecting the two lines.\n> \n> I didn't expect the least that this would go upto v3. In case anyboy finds\n> something wrong with this change too, it's a lot better to drop this altogether\n> than going for a v4.\n\nThat happens more often than not :)\n\nYour original intent was to avoid any misunderstandings, and all the\ncomments and iterations made sure that we don't trade one possible\nsource of misunderstanding for another but avoid them all.\n\nI consider v4 to be a strict improvement over the status quo and (as fas\nas I can see) to serve your original intent as good as possible.\n\n> wt-status.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/wt-status.c b/wt-status.c\n> index 77c27c511..23e87e74d 100644\n> --- a/wt-status.c\n> +++ b/wt-status.c\n> @@ -934,7 +934,7 @@ size_t wt_status_locate_end(const char *s, size_t len)\n> \n> void wt_status_add_cut_line(FILE *fp)\n> {\n> -\tconst char *explanation = _(\"Do not touch the line above.\\nEverything below will be removed.\");\n> +\tconst char *explanation = _(\"Do not modify or remove the line above.\\nEverything below it will be ignored.\");\n> \tstruct strbuf buf = STRBUF_INIT;\n> \n> \tfprintf(fp, \"%c %s\", comment_line_char, cut_line);\n> \n"},{"id":"328121","messageId":"1505479183.2638.4.camel@gmail.com","threadId":"46737","inReplyTo":"a8bfb8dc-583f-3126-bbe7-55cb845f1037@grubix.eu","subject":"Re: [PATCH v3] commit-template: change a message to be more intuitive","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-15T12:39:43Z","receivedAt":"2017-09-15T12:40:04Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Fri, 2017-09-15 at 12:00 +0200, Michael J Gruber wrote:\n> Kaartic Sivaraam venit, vidit, dixit 15.09.2017 06:50:\n> > \n> > I didn't expect the least that this would go upto v3. In case anyboy finds\n\nThat's should have been 'anybody'.\n\n> > something wrong with this change too, it's a lot better to drop this altogether\n> > than going for a v4.\n> \n> That happens more often than not :)\n> \n\n:)\n\n> Your original intent was to avoid any misunderstandings, and all the\n> comments and iterations made sure that we don't trade one possible\n> source of misunderstanding for another but avoid them all.\n> \n\nOf course they did. \n\n> I consider v4 to be a strict improvement over the status quo and (as fas\n> as I can see) to serve your original intent as good as possible.\n\nI thought I shouldn't go for a v4 as I feared it might make things\nworse than better because the original sentence wasn't that confusing\nin the first place ;-)\n\n-- \nKaartic\n"}]}