Re: [PATCH v2] blame: Add option to show author email instead of name

From: Jeff King <peff@peff.net>
Date: Oct 21, 2010, 20:58 UTC
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:

Show 8 quoted lines

> > 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