From: Jeff King <peff@peff.net>
Date: Fri, 19 Aug 2011 03:37:34 GMT
Subject: Re: [PATCH 11/10] support pager.* for aliases
Message-ID: <20110819033733.GB2993@sigill.intra.peff.net>
In-Reply-To: <7v8vqq72kp.fsf@alter.siamese.dyndns.org>

On Thu, Aug 18, 2011 at 03:54:46PM -0700, Junio C Hamano wrote:

> Jeff King <peff@peff.net> writes:
> 
> > Until this patch, doing something like:
> >
> >   git config alias.foo log
> >   git config pager.foo /some/specific/pager
> >
> > would not respect pager.foo at all.
> 
> Is it a good thing? Looks too confusing and I am having a hard time to
> decide if this is "just because we could" or "because we need to be able
> to do this for such and such reasons".

I don't have a particular use for it myself. However, I don't see what's
confusing about it. Would would you expect the above commands to do with
respect to paging? I think the behavior after my patch does what users
will expect, whether they have configured pager.foo, pager.log, or
nothing.

-Peff