{"thread":{"id":"46488","subject":"[PATCH] Fix minor typo in git-diff docs.","startedAt":"2017-07-31T14:00:40Z","lastAt":"2017-07-31T17:02:21Z","messageCount":4,"participants":["Anthony Sottile","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"325297","messageId":"CA+dzEB=cDvp7ZS8x+p+U-5NbK3SNd0FPyj_wP=gvi8mJi6D2ag@mail.gmail.com","threadId":"46488","inReplyTo":null,"subject":"[PATCH] Fix minor typo in git-diff docs.","fromName":"Anthony Sottile","fromEmail":"asottile@umich.edu","sentAt":"2017-07-31T14:00:31Z","receivedAt":"2017-07-31T14:00:40Z","isPatch":true,"sender":{"key":"asottile@umich.edu","avatar":"https://avatars.githubusercontent.com/u/1810591?v=4"},"body":"To be honest, I'm a bit overwhelmed by the documentation for submitting a patch!\n\nI tried to follow as best I could, here's my attempt (please advise).\n\nFrom e88ad689a7587c11f270a10f191a3b6bc52a90d4 Mon Sep 17 00:00:00 2001\nFrom: Anthony Sottile \nDate: Mon, 31 Jul 2017 06:54:14 -0700\nSubject: [PATCH] Fix minor typo in git-diff docs.\n\nSigned-off-by: Anthony Sottile \n---\n Documentation/diff-options.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\nindex 89cc0f4..43d18a4 100644\n--- a/Documentation/diff-options.txt\n+++ b/Documentation/diff-options.txt\n@@ -392,7 +392,7 @@ endif::git-log[]\n the diff between the preimage and `/dev/null`. The resulting patch\n is not meant to be applied with `patch` or `git apply`; this is\n solely for people who want to just concentrate on reviewing the\n- text after the change. In addition, the output obviously lack\n+ text after the change. In addition, the output obviously lacks\n enough information to apply such a patch in reverse, even manually,\n hence the name of the option.\n +\n-- \n2.7.4\n"},{"id":"325300","messageId":"xmqqy3r4im5y.fsf@gitster.mtv.corp.google.com","threadId":"46488","inReplyTo":"CA+dzEB=cDvp7ZS8x+p+U-5NbK3SNd0FPyj_wP=gvi8mJi6D2ag@mail.gmail.com","subject":"Re: [PATCH] Fix minor typo in git-diff docs.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-07-31T16:52:25Z","receivedAt":"2017-07-31T16:52:39Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Anthony Sottile writes:\n\n> To be honest, I'm a bit overwhelmed by the documentation for submitting a patch!\n>\n> I tried to follow as best I could, here's my attempt (please advise).\n\nOK ;-) Thanks for a patch. Let's nitpick.\n\nOur titles (your \"Subject:\" line in the e-mail header) state the\narea, colon and then a one line summary and ends without the final\nfull stop, e.g.\n\n Subject: [PATCH] diff-options: grammar fix\n\n>\n> From e88ad689a7587c11f270a10f191a3b6bc52a90d4 Mon Sep 17 00:00:00 2001\n> From: Anthony Sottile \n> Date: Mon, 31 Jul 2017 06:54:14 -0700\n> Subject: [PATCH] Fix minor typo in git-diff docs.\n\nGenerally these do not belong to the body of the message. \n\nWhat object name the commit has locally in your repository is\nimmaterial. Whose change it is, when it was shown to the general\npublic for the first time, and what the title of the change is, are\nall in the e-mail header of your message.\n\nOne exception is when you cannot convince your e-mail client to put\nthe name under which you sign-off the patch on the \"From: \" line in\nyour e-mail header, or you are relaying somebody else's patch. You\ncan then have \"From: \" line to record the right name, a blank line\nand then write body of the message.\n\n>\n> Signed-off-by: Anthony Sottile \n\nThe space above \"Signed-off-by: \" is to describe what the change is\nabout, when the title is not clear enough, but I think there is no\nneed for such extra explanation for this particular patch.\n\nThanks for correcting my grammar. Will queue with a tweaked title.\n\n> ---\n> Documentation/diff-options.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\n> index 89cc0f4..43d18a4 100644\n> --- a/Documentation/diff-options.txt\n> +++ b/Documentation/diff-options.txt\n> @@ -392,7 +392,7 @@ endif::git-log[]\n> the diff between the preimage and `/dev/null`. The resulting patch\n> is not meant to be applied with `patch` or `git apply`; this is\n> solely for people who want to just concentrate on reviewing the\n> - text after the change. In addition, the output obviously lack\n> + text after the change. In addition, the output obviously lacks\n> enough information to apply such a patch in reverse, even manually,\n> hence the name of the option.\n> +\n"},{"id":"325301","messageId":"xmqqtw1silu3.fsf@gitster.mtv.corp.google.com","threadId":"46488","inReplyTo":"CA+dzEB=cDvp7ZS8x+p+U-5NbK3SNd0FPyj_wP=gvi8mJi6D2ag@mail.gmail.com","subject":"Re: [PATCH] Fix minor typo in git-diff docs.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-07-31T16:59:32Z","receivedAt":"2017-07-31T16:59:45Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Anthony Sottile writes:\n\n> diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\n> index 89cc0f4..43d18a4 100644\n> --- a/Documentation/diff-options.txt\n> +++ b/Documentation/diff-options.txt\n> @@ -392,7 +392,7 @@ endif::git-log[]\n> the diff between the preimage and `/dev/null`. The resulting patch\n> is not meant to be applied with `patch` or `git apply`; this is\n> solely for people who want to just concentrate on reviewing the\n> - text after the change. In addition, the output obviously lack\n> + text after the change. In addition, the output obviously lacks\n> enough information to apply such a patch in reverse, even manually,\n> hence the name of the option.\n> +\n\nAnother thing that is more severe. You seem to have replaced all\nleading tabs with whitespaces, which makes the patch unusable. For\nthis single character patch, I can pretend as if I applied your\npatch while making the fix myself in my editor, so there is no need\nto resend, but please make sure your e-mail client does not do that\nthe next time.\n\nThanks. Queued.\n"},{"id":"325303","messageId":"CA+dzEB=EV+wR-ETvm9wrR6cYGFZQbtdxsPHMTkfz6EOgah_AnQ@mail.gmail.com","threadId":"46488","inReplyTo":"xmqqtw1silu3.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH] Fix minor typo in git-diff docs.","fromName":"Anthony Sottile","fromEmail":"asottile@umich.edu","sentAt":"2017-07-31T17:02:16Z","receivedAt":"2017-07-31T17:02:21Z","isPatch":true,"sender":{"key":"asottile@umich.edu","avatar":"https://avatars.githubusercontent.com/u/1810591?v=4"},"body":"Thanks!\n\nI'll keep this in mind next time I send a patch.\n\nAnthony\n\nOn Mon, Jul 31, 2017 at 9:59 AM, Junio C Hamano wrote:\n> Anthony Sottile writes:\n>\n>> diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\n>> index 89cc0f4..43d18a4 100644\n>> --- a/Documentation/diff-options.txt\n>> +++ b/Documentation/diff-options.txt\n>> @@ -392,7 +392,7 @@ endif::git-log[]\n>> the diff between the preimage and `/dev/null`. The resulting patch\n>> is not meant to be applied with `patch` or `git apply`; this is\n>> solely for people who want to just concentrate on reviewing the\n>> - text after the change. In addition, the output obviously lack\n>> + text after the change. In addition, the output obviously lacks\n>> enough information to apply such a patch in reverse, even manually,\n>> hence the name of the option.\n>> +\n>\n> Another thing that is more severe. You seem to have replaced all\n> leading tabs with whitespaces, which makes the patch unusable. For\n> this single character patch, I can pretend as if I applied your\n> patch while making the fix myself in my editor, so there is no need\n> to resend, but please make sure your e-mail client does not do that\n> the next time.\n>\n> Thanks. Queued.\n"}]}