{"thread":{"id":"19172","subject":"[PATCH 0/3] git-am documentation","startedAt":"2009-05-04T06:46:55Z","lastAt":"2009-05-04T11:43:25Z","messageCount":8,"participants":["Stephen Boyd","Jeff King","Junio C Hamano","Michael J Gruber"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"112953","messageId":"1241419618-20304-1-git-send-email-bebarino@gmail.com","threadId":"19172","inReplyTo":null,"subject":"[PATCH 0/3] git-am documentation","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-05-04T06:46:55Z","receivedAt":"2009-05-04T06:46:55Z","isPatch":true,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"The first two are minor grammatical fixes. I'm interested in what\npeople think about patch 3.\n\nStephen Boyd (3):\n git-am.txt: add an 'a', say what 'it' is, simplify a sentence\n git-am.txt: Use date instead of time or timestamp\n git-am.txt: reword extra headers in message body\n\n Documentation/git-am.txt | 24 ++++++++++++------------\n 1 files changed, 12 insertions(+), 12 deletions(-)\n"},{"id":"112956","messageId":"1241419618-20304-2-git-send-email-bebarino@gmail.com","threadId":"19172","inReplyTo":"1241419618-20304-1-git-send-email-bebarino@gmail.com","subject":"[PATCH 1/3] git-am.txt: add an 'a', say what 'it' is, simplify a sentence","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-05-04T06:46:56Z","receivedAt":"2009-05-04T06:46:56Z","isPatch":true,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"It's nice to know that 'it' is git-am or the subject line. Whitespace\nimplies characters so just remove characters.\n\nSigned-off-by: Stephen Boyd \n---\n Documentation/git-am.txt | 12 ++++++------\n 1 files changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex 1e71dd5..715531b 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -32,7 +32,7 @@ OPTIONS\n \n -s::\n --signoff::\n-\tAdd `Signed-off-by:` line to the commit message, using\n+\tAdd a `Signed-off-by:` line to the commit message, using\n \tthe committer identity of yourself.\n \n -k::\n@@ -118,8 +118,8 @@ The commit author name is taken from the \"From: \" line of the\n message, and commit author time is taken from the \"Date: \" line\n of the message. The \"Subject: \" line is used as the title of\n the commit, after stripping common prefix \"[PATCH ]\".\n-It is supposed to describe what the commit is about concisely as\n-a one line text.\n+The \"Subject: \" line is supposed to concisely describe what the\n+commit is about in one line of text.\n \n The body of the message (the rest of the message after the blank line\n that terminates the RFC2822 headers) can begin with \"Subject: \" and\n@@ -128,8 +128,8 @@ to override the values of these fields.\n \n The commit message is formed by the title taken from the\n \"Subject: \", a blank line and the body of the message up to\n-where the patch begins. Excess whitespace characters at the end of the\n-lines are automatically stripped.\n+where the patch begins. Excess whitespace at the end of each\n+line is automatically stripped.\n \n The patch is expected to be inline, directly following the\n message. Any line that is of the form:\n@@ -141,7 +141,7 @@ message. Any line that is of the form:\n is taken as the beginning of a patch, and the commit log message\n is terminated before the first occurrence of such a line.\n \n-When initially invoking it, you give it the names of the mailboxes\n+When initially invoking 'git-am', you give it the names of the mailboxes\n to process. Upon seeing the first patch that does not apply, it\n aborts in the middle. You can recover from this in one of two ways:\n \n-- \n1.6.2.3\n"},{"id":"112955","messageId":"1241419618-20304-3-git-send-email-bebarino@gmail.com","threadId":"19172","inReplyTo":"1241419618-20304-2-git-send-email-bebarino@gmail.com","subject":"[PATCH 2/3] git-am.txt: Use date instead of time or timestamp","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-05-04T06:46:57Z","receivedAt":"2009-05-04T06:46:57Z","isPatch":true,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Signed-off-by: Stephen Boyd \n---\n Documentation/git-am.txt | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex 715531b..c141261 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -85,7 +85,7 @@ default. You can use `--no-utf8` to override this.\n \tBy default the command records the date from the e-mail\n \tmessage as the commit author date, and uses the time of\n \tcommit creation as the committer date. This allows the\n-\tuser to lie about author timestamp by using the same\n+\tuser to lie about the author date by using the same\n \ttimestamp as the committer date.\n \n --skip::\n@@ -115,7 +115,7 @@ DISCUSSION\n ----------\n \n The commit author name is taken from the \"From: \" line of the\n-message, and commit author time is taken from the \"Date: \" line\n+message, and commit author date is taken from the \"Date: \" line\n of the message. The \"Subject: \" line is used as the title of\n the commit, after stripping common prefix \"[PATCH ]\".\n The \"Subject: \" line is supposed to concisely describe what the\n-- \n1.6.2.3\n"},{"id":"112954","messageId":"1241419618-20304-4-git-send-email-bebarino@gmail.com","threadId":"19172","inReplyTo":"1241419618-20304-3-git-send-email-bebarino@gmail.com","subject":"[PATCH 3/3] git-am.txt: reword extra headers in message body","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-05-04T06:46:58Z","receivedAt":"2009-05-04T06:46:58Z","isPatch":true,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Signed-off-by: Stephen Boyd \n---\n Documentation/git-am.txt | 8 ++++----\n 1 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex c141261..95e6089 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -121,10 +121,10 @@ the commit, after stripping common prefix \"[PATCH ]\".\n The \"Subject: \" line is supposed to concisely describe what the\n commit is about in one line of text.\n \n-The body of the message (the rest of the message after the blank line\n-that terminates the RFC2822 headers) can begin with \"Subject: \" and\n-\"From: \" lines that are different from those of the mail header,\n-to override the values of these fields.\n+\"From: \" and \"Subject: \" lines starting the body (the rest of the\n+message after the blank line terminating the RFC2822 headers)\n+override the respective commit author name and title values taken\n+from the headers.\n \n The commit message is formed by the title taken from the\n \"Subject: \", a blank line and the body of the message up to\n-- \n1.6.2.3\n"},{"id":"112958","messageId":"20090504071534.GB12654@coredump.intra.peff.net","threadId":"19172","inReplyTo":"1241419618-20304-3-git-send-email-bebarino@gmail.com","subject":"Re: [PATCH 2/3] git-am.txt: Use date instead of time or timestamp","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-05-04T07:15:34Z","receivedAt":"2009-05-04T07:15:34Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, May 03, 2009 at 11:46:57PM -0700, Stephen Boyd wrote:\n\n> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\n> index 715531b..c141261 100644\n> --- a/Documentation/git-am.txt\n> +++ b/Documentation/git-am.txt\n> @@ -85,7 +85,7 @@ default. You can use `--no-utf8` to override this.\n> \tBy default the command records the date from the e-mail\n> \tmessage as the commit author date, and uses the time of\n> \tcommit creation as the committer date. This allows the\n> -\tuser to lie about author timestamp by using the same\n> +\tuser to lie about the author date by using the same\n> \ttimestamp as the committer date.\n\nBut you leave the \"timestamp\" in the next context line?\n\nThe other two patches in the series look fine to me, though.\n\n-Peff\n"},{"id":"112959","messageId":"7vprep49h3.fsf@alter.siamese.dyndns.org","threadId":"19172","inReplyTo":"20090504071534.GB12654@coredump.intra.peff.net","subject":"Re: [PATCH 2/3] git-am.txt: Use date instead of time or timestamp","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-05-04T07:23:52Z","receivedAt":"2009-05-04T07:23:52Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> On Sun, May 03, 2009 at 11:46:57PM -0700, Stephen Boyd wrote:\n>\n>> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\n>> index 715531b..c141261 100644\n>> --- a/Documentation/git-am.txt\n>> +++ b/Documentation/git-am.txt\n>> @@ -85,7 +85,7 @@ default. You can use `--no-utf8` to override this.\n>> \tBy default the command records the date from the e-mail\n>> \tmessage as the commit author date, and uses the time of\n>> \tcommit creation as the committer date. This allows the\n>> -\tuser to lie about author timestamp by using the same\n>> +\tuser to lie about the author date by using the same\n>> \ttimestamp as the committer date.\n>\n> But you leave the \"timestamp\" in the next context line?\n>\n> The other two patches in the series look fine to me, though.\n\nLikewise. Perhaps \"using the same value as the committer date\" sounds\nbetter?\n"},{"id":"112960","messageId":"20090504072523.GA12759@coredump.intra.peff.net","threadId":"19172","inReplyTo":"7vprep49h3.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 2/3] git-am.txt: Use date instead of time or timestamp","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-05-04T07:25:23Z","receivedAt":"2009-05-04T07:25:23Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, May 04, 2009 at 12:23:52AM -0700, Junio C Hamano wrote:\n\n> >> \tcommit creation as the committer date. This allows the\n> >> -\tuser to lie about author timestamp by using the same\n> >> +\tuser to lie about the author date by using the same\n> >> \ttimestamp as the committer date.\n> >\n> > But you leave the \"timestamp\" in the next context line?\n> >\n> > The other two patches in the series look fine to me, though.\n> \n> Likewise. Perhaps \"using the same value as the committer date\" sounds\n> better?\n\nMuch better IMHO.\n\n-Peff\n"},{"id":"112969","messageId":"49FED4DD.3090103@drmicha.warpmail.net","threadId":"19172","inReplyTo":"1241419618-20304-2-git-send-email-bebarino@gmail.com","subject":"Re: [PATCH 1/3] git-am.txt: add an 'a', say what 'it' is, simplify a sentence","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2009-05-04T11:43:25Z","receivedAt":"2009-05-04T11:43:25Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Stephen Boyd venit, vidit, dixit 04.05.2009 08:46:\n> It's nice to know that 'it' is git-am or the subject line. Whitespace\n> implies characters so just remove characters.\n> \n> Signed-off-by: Stephen Boyd \n> ---\n> Documentation/git-am.txt | 12 ++++++------\n> 1 files changed, 6 insertions(+), 6 deletions(-)\n> \n> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\n> index 1e71dd5..715531b 100644\n> --- a/Documentation/git-am.txt\n> +++ b/Documentation/git-am.txt\n> @@ -32,7 +32,7 @@ OPTIONS\n> \n> -s::\n> --signoff::\n> -\tAdd `Signed-off-by:` line to the commit message, using\n> +\tAdd a `Signed-off-by:` line to the commit message, using\n> \tthe committer identity of yourself.\n> \n> -k::\n> @@ -118,8 +118,8 @@ The commit author name is taken from the \"From: \" line of the\n> message, and commit author time is taken from the \"Date: \" line\n> of the message. The \"Subject: \" line is used as the title of\n> the commit, after stripping common prefix \"[PATCH ]\".\n> -It is supposed to describe what the commit is about concisely as\n> -a one line text.\n> +The \"Subject: \" line is supposed to concisely describe what the\n> +commit is about in one line of text.\n> \n> The body of the message (the rest of the message after the blank line\n> that terminates the RFC2822 headers) can begin with \"Subject: \" and\n> @@ -128,8 +128,8 @@ to override the values of these fields.\n> \n> The commit message is formed by the title taken from the\n> \"Subject: \", a blank line and the body of the message up to\n> -where the patch begins. Excess whitespace characters at the end of the\n> -lines are automatically stripped.\n> +where the patch begins. Excess whitespace at the end of each\n> +line is automatically stripped.\n> \n> The patch is expected to be inline, directly following the\n> message. Any line that is of the form:\n> @@ -141,7 +141,7 @@ message. Any line that is of the form:\n> is taken as the beginning of a patch, and the commit log message\n> is terminated before the first occurrence of such a line.\n> \n> -When initially invoking it, you give it the names of the mailboxes\n> +When initially invoking 'git-am', you give it the names of the mailboxes\n\nWe try to spell git commands in the form 'git am' these days. Also, `git\nam` should be the quoting for commands, although we don't have a style\nguide and things are not consistent anyways.\n\n> to process. Upon seeing the first patch that does not apply, it\n> aborts in the middle. You can recover from this in one of two ways:\n> \n"}]}