From: Jeff King <peff@peff.net>
Date: Wed, 23 Mar 2011 18:17:56 GMT
Subject: Re: [PATCH 3/3] diffcore-rename: fall back to -C when -C -C busts the rename limit
Message-ID: <20110323181756.GA17415@sigill.intra.peff.net>
In-Reply-To: <20110323155854.GB30337@sigill.intra.peff.net>

On Wed, Mar 23, 2011 at 11:58:54AM -0400, Jeff King wrote:

> Also, on a somewhat related note: which commands should have rename
> progress reporting turned on? It makes sense for "git diff" to do it to
> me. And probably even "git show". Probably not for "git log", though.
> I think we'd also have to look at how that interacts with the
> stderr-to-stdout pager thing. We obviously don't want progress going to
> the pager.

This quick-and-dirty series makes rename progress work properly for "git
show". Patch 1/3 would also be the building block for having "git log"
show per-commit warnings when stderr and stdout are mixed, but a single
warning otherwise. I built it on top of your patches, but I think there
is no reason it could not be done in parallel.

  [1/3]: pager: save the original stderr when redirecting to pager
  [2/3]: progress: use pager's original_stderr if available
  [3/3]: show: turn on rename progress

-Peff