Re: [PATCH] Add format.date config variable

On Wed, May 21, 2008 at 11:24:42AM -0700, Junio C Hamano wrote:

quoted

Also, add missing "short" alternative to --date in rev-list-options.txt.

Good change but is a separate issue.  Please make it a [PATCH 1/2].

Will do that a separate patch.

Lacking Sign-off.

OK. Thanks.

The patch itself is good, especially that you made sure that format-patch
output is not affected.

As to the configuration variable name, I'd suggest "log.date" instead.

OK.

Will submit a patch later today.

-- 
Heikki Orsila
heikki.orsila@iki.fi
http://www.iki.fi/shd