[PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Eric Sunshine
Introduce command check-mailmap, similar to check-attr and check-ignore, which allows direct testing of .mailmap configuration. As plumbing accessible to scripts and other porcelain, check-mailmap publishes the stable, well-tested .mailmap functionality employed by built-in Git commands. Conseque

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Junio C Hamano
Eric Sunshine writes: > +DESCRIPTION > +--- > + > +For each ``Name '' or ``'' provided on > +the command-line or standard input (when using `--stdin`), prints a line with > +the canonical contact information for that person according to the 'mailmap' > +(see "Mapping Authors" belo

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Eric Sunshine
On Thu, Jul 11, 2013 at 3:04 PM, Junio C Hamano wrote: > Eric Sunshine writes: > >> +DESCRIPTION >> +--- >> + >> +For each ``Name '' or ``'' provided on >> +the command-line or standard input (when using `--stdin`), prints a line >> with >> +the canonical contact information for

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Junio C Hamano
Eric Sunshine writes: >> For each contact information (either in the form of ``Name >> '' or ...) >> >> in order to clarify that the two forms of input is what you call >> "contact information". > > Is this easier to read? > > For each ``Name '' or ``'' from the >

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Eric Sunshine
On Fri, Jul 12, 2013 at 1:47 AM, Junio C Hamano wrote: > Eric Sunshine writes: > >>> For each contact information (either in the form of ``Name >>> '' or ...) >>> >>> in order to clarify that the two forms of input is what you call >>> "contact information". >> >> Is this easier t

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Junio C Hamano
Eric Sunshine writes: >> I find it easier than your original, but I do not know if you would >> want to repeat the "Name... or " at the end. It does not >> seem to add much useful information and is distracting. > > Next attempt: > > For each ``Name '' or ``'' from the > command-

Re: [PATCH v2 1/4] builtin: add git-check-mailmap command

2013-07-11 Thread Eric Sunshine
On Fri, Jul 12, 2013 at 2:34 AM, Junio C Hamano wrote: > Eric Sunshine writes: > >> ... Is it desirable to do so >> or should the user have more fine-grained control? ("xargs -0" comes >> to mind when thinking of a null-termination input switch.) > > For the purposes of check-attr and check-ignor