Re: [PATCH v2] blame: add blame.showEmail configuration

2015-05-31 Thread Quentin Neill
On Sun, May 31, 2015 at 1:13 PM, Junio C Hamano wrote: > Quentin Neill writes: > >> From: Quentin Neill >> >> Complement existing --show-email option with fallback >> configuration variable, with tests. >> --- > > The patch itself looks very reasonable. Thanks for getting back to > us ;-) > > A

Re: [PATCH v2] blame: add blame.showEmail configuration

2015-05-31 Thread Junio C Hamano
Quentin Neill writes: > From: Quentin Neill > > Complement existing --show-email option with fallback > configuration variable, with tests. > --- The patch itself looks very reasonable. Thanks for getting back to us ;-) A few minor nits: - Your in-body "From:" is redundant and unnecessar

[PATCH v2] blame: add blame.showEmail configuration

2015-05-30 Thread Quentin Neill
From: Quentin Neill Complement existing --show-email option with fallback configuration variable, with tests. --- Documentation/git-blame.txt | 2 ++ builtin/blame.c | 10 +++- t/t8002-blame.sh| 62 + 3 files changed, 73 in