Re: [RFC PATCH] builtin-log: Add options to --coverletter
- From
James Cloos <cloos@jhcloos.com>
- Date
- May 16, 2009, 17:35 UTC
- Message-ID
- <m3my9dhrw5.fsf@lugabout.jhcloos.org>
- In-Reply-To
- <1242349041.646.8.camel@Joe-Laptop.home>
>>>>> "Joe" == Joe Perches <joe@perches.com> writes:
Joe> Currently the coverletter options for wrapping long lines Joe> in the shortlog are: on, wrap as position72, with fixed indents.
Joe> I think these defaults can produce poor looking output.
Joe> This patch allows these to be optionally specified on the Joe> command line with --cover-letter[=wrap[,pos[,in1[,in2]]]]
Joe> I'm not sure this is the right approach though.
If one wants to be really cool, one could borrow the relevant code from GNU fmt(1). (Be sure to grab from a release before the switch to GPL3.)
GNU fmt(1) uses an algorithm based on TeX's and produces *much* better results than anything else I've seen.
It is currently distributed as part of coreutils.
-JimC
-- James Cloos <cloos@jhcloos.com> OpenPGP: 1024D/ED7DAEA6