From: Jeff King <peff@peff.net>
Date: Tue, 07 Jan 2014 17:52:42 GMT
Subject: Re: [PATCH] sha1_name: don't resolve refs when core.warnambiguousrefs is false
Message-ID: <20140107175241.GA20415@sigill.intra.peff.net>
In-Reply-To: <xmqqzjn7el4k.fsf@gitster.dls.corp.google.com>

On Tue, Jan 07, 2014 at 09:51:07AM -0800, Junio C Hamano wrote:

> Jeff King <peff@peff.net> writes:
> 
> > Alternatively, I guess "cat-file
> > --batch" could just turn off warn_ambiguous_refs itself.
> 
> Sounds like a sensible way to go, perhaps on top of this change?

The downside is that we would not warn about ambiguous refs anymore,
even if the user was expecting it to. I don't know if that matters much.
I kind of feel in the --batch situation that it is somewhat useless (I
wonder if "rev-list --stdin" should turn it off, too).

-Peff