From: Mark Brown <broonie@sirena.org.uk>
To: openembedded-devel@lists.openembedded.org
Subject: Re: Using `git send-email` (was: Number 1 tip for getting your patch committed)
Date: Mon, 25 Jan 2010 13:46:48 +0000 [thread overview]
Message-ID: <20100125134647.GA30102@sirena.org.uk> (raw)
In-Reply-To: <1264284382.3738.175.camel@mattotaupa.wohnung.familie-menzel.net>
On Sat, Jan 23, 2010 at 11:06:22PM +0100, Paul Menzel wrote:
> Secondly for a lot of people using `git send-email` is no option because
> setup is too difficult, their MUA does not integrate well with `git
> send-email` so the address book is not available, they do want to have
> sent messages archived in their Sent folder, they want to ask some
> questions regarding the patch and therefore need to write a message and
> attach it.
There's also other variants like git imap-send. The send-email
suggestion could be rewritten as "do something which git am understands
out of the box", it's approximately equivalent.
> In my opinion we should therefore find a way to also work easily with
> attached patches having been created with `git format-patch`. Can some
> of the developers mention what the disadvantage of attached patches is.
> You can easily quote them to and saving and applying should be as easy
> as with messages sent using `git send-email`.
> The only disadvantage I can think of seems to be that it is not
> displayed in the mailing list archive.
Attaching patches also upsets most tools, they tend to assume that the
body of the mail is supposed to be part of the commit message (which it
would normally be) which tends not to look too pretty. Replying can be
an issue with some MUAs too, especially if the attachment gets reencoded
due to not being seven bit clean.
prev parent reply other threads:[~2010-01-25 13:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-21 12:58 Number 1 tip for getting your patch committed Philip Balister
2010-01-23 22:06 ` Using `git send-email` (was: Number 1 tip for getting your patch committed) Paul Menzel
2010-01-24 2:54 ` Holger Hans Peter Freyther
2010-01-25 7:32 ` Khem Raj
2010-01-25 13:46 ` Mark Brown [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=20100125134647.GA30102@sirena.org.uk \
--to=broonie@sirena.org.uk \
--cc=openembedded-devel@lists.openembedded.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