From: Junio C Hamano <gitster@pobox•com>
To: Adam Monsen <haircut@gmail•com>
Cc: git@vger•kernel.org, Ivan Heffner <iheffner@gmail•com>
Subject: Re: [PATCH 0/3] "commit --template" fixes
Date: Sun, 01 Apr 2012 15:28:39 -0700 [thread overview]
Message-ID: <7vaa2vhyso.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: <4F775ACF.50007@gmail.com> (Adam Monsen's message of "Sat, 31 Mar 2012 12:28:15 -0700")
Adam Monsen <haircut@gmail•com> writes:
> ...
> * I don't know the kosher procedure to add this commit to your patch
> series for further review, so hopefully this works.
> * I'm not sure if the "Helped-by:" lines are kosher, I'm happy to
> remove them if not.
One established way to do this is to have a discussion like the above
(mostly elided), followed by a "scissors" line "-- >8 --", and then the
output from format-patch with most headers except for "Subject: " removed
(as From: and Date: will be taken from your e-mailed message anyway, and
the first "From <commit object name> <magic constant date>" is a signal to
allow automated tools to tell if it is a format-patch output or a random
mbox file, and is not appropriate if you are sending it over e-mail).
>
> From 91a62baa1fe89032e7a3598e5d39241f3eb8f84b Mon Sep 17 00:00:00 2001
> From: Adam Monsen <haircut@gmail•com>
> Date: Sat, 31 Mar 2012 12:09:29 -0700
> Subject: [PATCH] git-commit.txt: clarify -t requires editing message
>
> Make it clear that, when using commit --template, the message *must* be
> changed or the commit will be aborted.
>
>
> Helped-by: Junio C Hamano <gitster@pobox•com>
> Helped-by: Ivan Heffner <iheffner@gmail•com>
> Signed-off-by: Adam Monsen <haircut@gmail•com>
> ---
> I wrapped the text at 77 characters because that was the longest
> line in the file (according to wc -L).
>
> I used ":set noet nosta ts=8 sw=8 tw=77" in Vim.
When rewording or clarifying only a handful of words in the documentation,
it is often better to avoid reflowing lines in the same patch. It makes
it harder to see what you really changed, and what is merely reflowed.
I'll queue it as-is, though. Thanks.
This is a tangent, but we might want to rephrase the first sentence
without using the word "version"; every time I read this paragraph, the
"initial version" makes me go "Huh?" because the word sounds as if it is
talking about commits in the context of SCM, which is not the case here.
I know that the description wanted to avoid use of the word "template" to
explain what the template is, but still...
> Documentation/git-commit.txt | 10 +++++-----
> 1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt
> index 5cc84a1..f584a62 100644
> --- a/Documentation/git-commit.txt
> +++ b/Documentation/git-commit.txt
> @@ -132,11 +132,11 @@ OPTIONS
>
> -t <file>::
> --template=<file>::
> - Use the contents of the given file as the initial version
> - of the commit message. The editor is invoked and you can
> - make subsequent changes. If a message is specified using
> - the `-m` or `-F` options, this option has no effect. This
> - overrides the `commit.template` configuration variable.
> + Use the contents of the given file as the initial version of the
> + commit message. The editor is invoked so you can make subsequent
> + changes. If you make no changes, the commit is aborted. If a message
> + is specified using the `-m` or `-F` options, this option has no
> + effect. This overrides the `commit.template` configuration variable.
>
> -s::
> --signoff::
next prev parent reply other threads:[~2012-04-01 22:29 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-29 17:57 [PATCH] git-commit doc: say -t requires editing commit message Adam Monsen
2012-03-29 18:09 ` Ivan Heffner
2012-03-29 23:04 ` [PATCH v2] git-commit.txt: clarify -t requires editing message Adam Monsen
2012-03-30 2:05 ` Junio C Hamano
2012-03-30 3:07 ` Adam Monsen
2012-03-30 3:52 ` Junio C Hamano
2012-03-30 4:53 ` Adam Monsen
2012-03-30 5:08 ` Junio C Hamano
2012-03-30 5:43 ` Adam Monsen
2012-03-30 18:17 ` Junio C Hamano
2012-03-30 19:45 ` [PATCH 0/3] "commit --template" fixes Junio C Hamano
2012-03-30 19:45 ` [PATCH 1/3] t7501: test the right kind of breakage Junio C Hamano
2012-03-30 19:45 ` [PATCH 2/3] commit: do not trigger bogus "has templated message edited" check Junio C Hamano
2012-03-30 19:45 ` [PATCH 3/3] commit: rephrase the error when user did not touch templated log message Junio C Hamano
2012-03-31 19:28 ` [PATCH 0/3] "commit --template" fixes Adam Monsen
2012-04-01 22:28 ` Junio C Hamano [this message]
2012-04-03 17:11 ` Adam Monsen
2012-04-03 21:55 ` Junio C Hamano
2012-04-05 14:29 ` Adam Monsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7vaa2vhyso.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=haircut@gmail$(echo .)com \
--cc=iheffner@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox