Re: [PATCH] Teach the --cover-letter option to format-patch
- From
Daniel Barkalow <barkalow@iabervon.org>
- Date
- Jul 10, 2007, 17:20 UTC
- Message-ID
- <Pine.LNX.4.64.0707101300450.6977@iabervon.org>
- In-Reply-To
- <Pine.LNX.4.64.0707101424140.4047@racer.site>
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.
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.)
-Daniel *This .sig left intentionally blank*