From: Jeff King <peff@peff.net>
To: Luben Tuikov <ltuikov@yahoo.com>
Cc: git@vger.kernel.org
Subject: Re: git-format-patch little gripe
Date: Fri, 3 Nov 2006 13:50:26 -0500 [thread overview]
Message-ID: <20061103185026.GA28566@coredump.intra.peff.net> (raw)
In-Reply-To: <376237.14965.qm@web31805.mail.mud.yahoo.com>
On Fri, Nov 03, 2006 at 01:07:20AM -0800, Luben Tuikov wrote:
> So I still make the mistake of:
> # git-rev-list --no-merges --pretty=one-line HEAD -- <somepath>
> I select a commit
> # git-format-patch -o /tmp/ <commit>
> ...
> Oh, f@#$! <CTRL-C>
> # git-format-patch -o /tmp/ <commit>^..<commit>
For my own workflow, I don't want to have to pick the commit out of
rev-list (or log) output. I want to find it and hit a button to say "OK,
now mail this patch." So I put _all_ of my patches into an mbox, and
then browse them with mutt. Sort of a poor man's patch browser, but then
I'm ready to jump into mailing them immediately.
I use the following script:
#!/bin/sh
root=${1:-origin}
git-format-patch -s --stdout $root >.mbox
mutt -f .mbox
rm -f .mbox
And then in mutt, I use this macro to bind 'b' to editing the full
message w/ headers:
macro index b ":set edit_headers=yes<enter><resend-message>:set edit_headers=no<enter>"
I know that may be useless if you're not using mutt, but I just wanted
to stimulate some discussion among patch submitters about how they
actually do it. I'm not sure how configurable tig is, but it shouldn't
be too hard to add something like this to it.
next prev parent reply other threads:[~2006-11-03 18:50 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-03 9:07 git-format-patch little gripe Luben Tuikov
2006-11-03 9:22 ` Martin Langhoff
2006-11-03 9:59 ` Luben Tuikov
2006-11-03 11:17 ` Junio C Hamano
2006-11-03 17:32 ` Carl Worth
2006-11-03 17:51 ` Jakub Narebski
2006-11-03 19:52 ` Luben Tuikov
2006-11-03 20:53 ` Jakub Narebski
2006-11-03 22:26 ` Junio C Hamano
2006-11-03 22:35 ` Carl Worth
2006-11-03 23:16 ` Jeff King
2006-11-03 23:37 ` Jakub Narebski
2006-11-04 0:44 ` Linus Torvalds
2006-11-04 1:05 ` Jakub Narebski
2006-11-03 18:50 ` Jeff King [this message]
2006-11-03 20:37 ` Luben Tuikov
2006-11-25 15:30 ` Sean
[not found] ` <20061125103033.2ea742d3.seanlkml@sympatico.ca>
2006-11-25 15:39 ` Jeff King
2006-11-25 15:42 ` Sean
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=20061103185026.GA28566@coredump.intra.peff.net \
--to=peff@peff.net \
--cc=git@vger.kernel.org \
--cc=ltuikov@yahoo.com \
/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).