{"thread":{"id":"57726","subject":"[PATCH] Documentation/RelNotes: fix a typo in 2.36's relnotes","startedAt":"2022-04-13T21:03:34Z","lastAt":"2022-04-13T21:46:09Z","messageCount":3,"participants":["Taylor Blau","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"453599","messageId":"d23e51b8dc78651dbd751473c0f8b3dcd6bc3ac6.1649883760.git.me@ttaylorr.com","threadId":"57726","inReplyTo":null,"subject":"[PATCH] Documentation/RelNotes: fix a typo in 2.36's relnotes","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-04-13T21:03:28Z","receivedAt":"2022-04-13T21:03:34Z","isPatch":true,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"Signed-off-by: Taylor Blau \n---\nNoticed while reading the RelNotes for GitHub's release highlights blog\npost.\n\nI'm pretty sure that this is a typo for \"not-so-uncommon\", but I could\nbe wrong (in which case, feel free to drop this patch on the floor).\n\n Documentation/RelNotes/2.36.0.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/RelNotes/2.36.0.txt b/Documentation/RelNotes/2.36.0.txt\nindex d61ade21e2..98e1c97617 100644\n--- a/Documentation/RelNotes/2.36.0.txt\n+++ b/Documentation/RelNotes/2.36.0.txt\n@@ -46,7 +46,7 @@ UI, Workflows & Features\n\n * \"git branch\" learned the \"--recurse-submodules\" option.\n\n- * A not-so-common mistake is to write a script to feed \"git bisect\n+ * A not-so-uncommon mistake is to write a script to feed \"git bisect\n run\" without making it executable, in which case all tests will\n exit with 126 or 127 error codes, even on revisions that are marked\n as good. Try to recognize this situation and stop iteration early.\n--\n2.35.1.73.gccc5557600\n"},{"id":"453601","messageId":"xmqqczhk7klg.fsf@gitster.g","threadId":"57726","inReplyTo":"d23e51b8dc78651dbd751473c0f8b3dcd6bc3ac6.1649883760.git.me@ttaylorr.com","subject":"Re: [PATCH] Documentation/RelNotes: fix a typo in 2.36's relnotes","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-04-13T21:13:31Z","receivedAt":"2022-04-13T21:16:10Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Taylor Blau writes:\n\n> Signed-off-by: Taylor Blau \n> ---\n> Noticed while reading the RelNotes for GitHub's release highlights blog\n> post.\n\nThanks as always for highlighting more interesting bits from the\nflat-and-bland-list-of-changes which is what I prepare as the\nrelease notes.\n\n> - * A not-so-common mistake is to write a script to feed \"git bisect\n> + * A not-so-uncommon mistake is to write a script to feed \"git bisect\n> run\" without making it executable, in which case all tests will\n\nI actually meant that this is \"not a common mistake that deserves our\nbandwidth but because a change was already made to help such a case,\nwhy not take it\".\n\nWe can probably rewrite _without_ mentioning our expectation of how\ncommon it would be.\n\n A user can forget to make a script file executable before giving\n it to \"git bisect run\". In such a case, all tests will exit ...\n"},{"id":"453606","messageId":"YldEmwQfLo2LbDGZ@nand.local","threadId":"57726","inReplyTo":"xmqqczhk7klg.fsf@gitster.g","subject":"Re: [PATCH] Documentation/RelNotes: fix a typo in 2.36's relnotes","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-04-13T21:46:03Z","receivedAt":"2022-04-13T21:46:09Z","isPatch":true,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Wed, Apr 13, 2022 at 02:13:31PM -0700, Junio C Hamano wrote:\n> Taylor Blau writes:\n>\n> > Signed-off-by: Taylor Blau \n> > ---\n> > Noticed while reading the RelNotes for GitHub's release highlights blog\n> > post.\n>\n> Thanks as always for highlighting more interesting bits from the\n> flat-and-bland-list-of-changes which is what I prepare as the\n> release notes.\n\nIt's a pleasure to do. Thanks for maintaining the release notes in the\nmeantime which make my job much easier to do.\n\n> > - * A not-so-common mistake is to write a script to feed \"git bisect\n> > + * A not-so-uncommon mistake is to write a script to feed \"git bisect\n> > run\" without making it executable, in which case all tests will\n>\n> I actually meant that this is \"not a common mistake that deserves our\n> bandwidth but because a change was already made to help such a case,\n> why not take it\".\n\nHa! I wasn't sure if that's what you meant or not. I think your\nsuggestion below is an appropriate way to remove the ambiguity. Thanks.\n\n> We can probably rewrite _without_ mentioning our expectation of how\n> common it would be.\n>\n> A user can forget to make a script file executable before giving\n> it to \"git bisect run\". In such a case, all tests will exit ...\n\nLooks good to me.\n\nThanks,\nTaylor\n"}]}