From: Jeff King <peff@peff.net>
Date: Thu, 21 Oct 2010 20:58:59 GMT
Subject: Re: [PATCH v2] blame: Add option to show author email instead of name
Message-ID: <20101021205859.GA29550@sigill.intra.peff.net>
In-Reply-To: <1D51CC29-0248-4C12-BD1B-4013434AEA03@sb.org>

On Thu, Oct 21, 2010 at 01:54:22PM -0700, Kevin Ballard wrote:

> > Isn't that what it is already?
> > 
> >  $ grep blame builtin/annotate.c
> >  return cmd_blame(argc + 1, nargv, prefix);
> 
> IIRC, git-annotate is equivalent to `git blame -c`, which is why the
> tests for git-blame are so useless (they're all tests for `git blame
> -c`, which makes them identical to the tests for git-annotate).

Yeah, I believe that t8002 is totally redundant with t8001 these days. I
wonder if we should just drop t8001 entirely.

The other blame tests use the "native" format. It would be nice to have
some --porcelain and --incremental tests, too, but I expect it is too
boring for anyone to volunteer to work on. :)

-Peff