From: Jeff King <peff@peff.net>
To: Markus Heidelberg <markus.heidelberg@web.de>
Cc: git@vger.kernel.org
Subject: Re: meaning of --8<--
Date: Sun, 18 Jan 2009 12:44:44 -0500 [thread overview]
Message-ID: <20090118174444.GB17055@coredump.intra.peff.net> (raw)
In-Reply-To: <200901181656.37813.markus.heidelberg@web.de>
On Sun, Jan 18, 2009 at 04:56:37PM +0100, Markus Heidelberg wrote:
> I've seen lines like "--8<--" several times on this list, but have no
> clue what it is about. OK, seems like it's used to insert diffs in the
> middle of a mail message.
> But is this a common convention or git specific and handled by git-am?
> Is it documented anywhere?
As others have explained, it is supposed to be scissors, and it is not
handled by git-am. But the real point of it is to help the person
applying (e.g., Junio) manually separate cover letter from the commit
message. So one way of sending cover material is putting it after the
"---" and before the diff:
Subject: [PATCH] first line of commit message
more commit message
---
Here is cover letter material that doesn't go in the commit message.
It is stripped automatically by git-am.
diff --git ...
but that is often unnatural to read, because the cover letter material
often introduces the commit, especially when you are replying in the
middle of a thread. So then you end up with:
Subject: Re: whatever thread you're in
Somebody else said:
> blah blah blah
I disagree. You should do it like this instead:
-- >8 --
first line of commit message
more commit message
---
diff --git ...
git-am will put everything down to the "---" into the commit message,
but it is simple enough to amend away everything else.
So in that sense, it doesn't really matter _what_ the separator is, but
it should be visibly obvious to a human so that they can fix up the
commit message. The only exception is that it should _not_ be "---",
because that is treated specially by git-am.
-Peff
prev parent reply other threads:[~2009-01-18 17:46 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-18 15:56 meaning of --8<-- Markus Heidelberg
2009-01-18 16:04 ` Peter Harris
2009-01-19 11:36 ` Samuel Tardieu
2009-01-18 16:13 ` Jay Soffian
2009-01-18 17:44 ` Jeff King [this message]
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=20090118174444.GB17055@coredump.intra.peff.net \
--to=peff@peff.net \
--cc=git@vger.kernel.org \
--cc=markus.heidelberg@web.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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