Re: [PATCH] Teach the --cover-letter option to format-patch
- From
Peter Oberndorfer <kumbayo84@arcor.de>
- Date
- Jul 24, 2007, 20:12 UTC
- Message-ID
- <200707242212.27837.kumbayo84@arcor.de>
- In-Reply-To
- <Pine.LNX.4.64.0707101819350.4047@racer.site>
On Tuesday 10 July 2007 19:20, Johannes Schindelin wrote:
Show 24 quoted lines
> Hi, > > On Tue, 10 Jul 2007, Daniel Barkalow wrote: > > > I think this is missing the ultra-important "extra_headers" stuff, which > > is what makes my messages actually reach the right people. That's why > > I'd like the code shared for generating headers (except for Subject) for > > a rev_info between the code that does it for patch messages and the code > > for the cover letter. I think it's also missing making [PATCH 1/N] a > > reply to it if the series is set up as replies. > > Ah yes. Both issues should be relatively easy to integrate into my patch. > > > I like the design, in any case. I want the blurb actually stored in the > > objects directory somehow, so that I don't have to trawl through my sent > > email for it when I send the series again for some reason, but that's a > > relatively straightforward extension to your code. (Read an object with > > a hash and stick the text in instead of the ***...*** parts.) > > I do not understand. But then, my patch should be a good starting point. > Go wild. > > Ciao, > Dscho
Anybody working on this? Somebody, *cough* convinced me into working on this. I already split up the existing patch into logic parts, and will now try to read message / header from file or a commit.
Greetings Peter