From: Junio C Hamano <junkio@cox.net>
To: Xavier Maillard <zedek@gnu.org>
Cc: git@vger.kernel.org
Subject: Re: format-patch and send-email
Date: Sun, 18 Feb 2007 01:22:18 -0800 [thread overview]
Message-ID: <7vps87hlbp.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <5680.1171788594@localhost> (Xavier Maillard's message of "Sun, 18 Feb 2007 09:49:54 +0100")
Xavier Maillard <zedek@gnu.org> writes:
> From 50c7032fa011b1e5c7bd63e4bc474c802017677d Mon Sep 17 00:00:00 2001
> From: Xavier Maillard <zedek@gnu.org>
> Date: Fri, 16 Feb 2007 20:20:18 +0100
> Subject: * Added 2007 to the copyright notice
> To: <maillaxa@gmail.com>
> * Added a note on relation between org-publish and blorg
> * Changed/corrected things here and there
> * Added a warning on needed skills before manipulating your own templates
> ---
> doc/blorg.texi | 31 +++++++++++++++++++------------
> 1 files changed, 19 insertions(+), 12 deletions(-)
> The "To" header has been set up in my .git/config file. But why
> the 'Subject' is like this and also why are there 2 'From'
> headers ?
The "format-patch" output was designed to (minimally) mimic what
you would see in a UNIX mbox as if you (or whoever would apply
the e-mailed patches to repository) received it, so that anybody
can later run "am" on it. The first "From " is what sometimes
is called "UNIX-From " line, which separates each piece of
e-mail in mbox formatted mailbox file. The second one is used
to record the author of the patch.
The commit log message is expected to be:
A single line description of a patch
More detailed explanation of the patch with
possibly multi-line or multi-paragraph text.
and the single line description becomes the subject.
Your commit seems to squash four completely unrelated changes
into one, and enumerates only single-liner description for each
independent item, like this:
* foo
* bar
* baz
* boa
which makes format-patch think you wanted to title it "* foo"
with more detailed description "* bar *baz *boa". I think you
would be much happier if you acquired a habit to make smaller
commits, that touch only one thing at a time without mixing up
unrelated things together, and give more descriptive messages to
each of them.
> Actually, why can't I just do like this ? :
>
> git format-patch <origin> | git send-email ?
While I know format-patch can be told to write to standard
output, I do not know if send-email can be told to read from the
standard input (I do not use send-email myself). Some people
seem to use send-imap to stuff format-patch output to imap
drafts. I just read format-patch output into message
composition buffer of my MUA (message mode of Emacs) and send it
out just as an ordinary e-mail (iow, as if I typed it). In
other words, you do not have to use send-email as-is, if what it
does does not suit your needs.
I suspect the intended usage original author of "send-email"
envisioned was to have lots of format-patch output files and
send them in bulk, so probably reading from standard input was
not high on its design goals. I dunno.
While I do believe some people use it, I haven't seen much
active development/enhancement on send-email from the users
lately.
next prev parent reply other threads:[~2007-02-18 9:22 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-18 8:49 format-patch and send-email Xavier Maillard
2007-02-18 9:22 ` Junio C Hamano [this message]
2007-02-18 18:17 ` Xavier Maillard
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=7vps87hlbp.fsf@assigned-by-dhcp.cox.net \
--to=junkio@cox.net \
--cc=git@vger.kernel.org \
--cc=zedek@gnu.org \
/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;
as well as URLs for NNTP newsgroup(s).