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

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Apr 26, 2010, 17:57 UTC
Message-ID: <20100426175707.GB11022@progeny.tock>
In-Reply-To: <20100426172235.GA626@coredump.intra.peff.net>

Jeff King wrote:

> To be clear, I don't have any problem with the _functionality_. I just
> think it probably should have been log.pretty, or pretty.default or
> something. Too late now, though.

Ah, sorry to be dense.  Maybe something like the following would work:

 [log "format"]
	changelog = "* [%h] %s"

This is rev-list’s code, and log is its most noticeable manifestation.

By the way: I just noticed today that rev-list is not very friendly when given a --pretty=format: specification:

  $ git rev-list -1 --oneline HEAD
  651b91e log --format=email: fix confusing variable name
  $ git rev-list -1 --format='%h %s' HEAD
  commit 651b91ef7a15a46145d65193c8709670b969161f
  651b91e log --format=email: fix confusing variable name

Is this deliberate? I mention it because I seem to remember from the discussion of --decorate configuration that scripts should be using rev-list in preference to log even when a --format option is involved.

Jonathan