Re: [PATCH 0/3] pretty: format aliases

From: Will Palmer <wmpalmer@gmail.com>
Date: Apr 26, 2010, 21:14 UTC
Message-ID: <1272316461.5368.4.camel@walleee>
In-Reply-To: <20100426183755.GC11378@progeny.tock>

On Mon, 2010-04-26 at 13:37 -0500, Jonathan Nieder wrote:

Show 9 quoted lines

> Alternatively, we could use
> 
>  [logformat]
> 	changelog = ...
> 
> which is arguably just as clear and may be closer to existing
> practice.  But really, I am happy as long as the configuration exists.
> 
> Jonathan

I like the one-level-of-hierarchy thing ([format "pretty"] is ugly for a
number of reasons), but two problems with that specific name:
 - it says "log", while this effects more than the "log" command
 - it says "format", while this doesn't effect --format.

[pretty]
    changelog = ...

is not taken, and my only hesitation there is that some other config option may want to be there. Still, it's what I'll call it in the next version of the patch, unless there are specific objections. (which I certainly may have already missed, as I've just switched mail clients twice today)

-- Will