Re: [PATCH] sha1_name: don't resolve refs when core.warnambiguousrefs is false
- From
Jeff King <peff@peff.net>
- Date
- Jan 7, 2014, 17:52 UTC
- 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:
Show 6 quoted lines
> 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