Re: [PATCH] tag: do not show ambiguous tag names as "tags/foo"

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 25, 2016, 02:26 UTC
Message-ID: <xmqqsi1m9yxh.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20160124222736.GA29115@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> Yeah, "strip=2" would also get the job done, and extends more naturally
> to the branch case.
>
> To be honest, I cannot imagine anybody using anything _but_ strip=2...

I 100% agree, and I do consider this to be internal implementation detail for the listing modes of "tag" (and "branch"), which may be exposed to the user (by documenting that %(refname:X) is used by default), so perhaps even the flexibility of strip=2 is unwanted.

I know what "remove-standard-prefix" is way too long for the value of X above, but then we can say "the command will error out if you allow your for-each-ref invocation to step outside of the area that has standard prefix to be removed." without having to worry about "what is the sensible thing to do when the prefixes are not what we expect (too short for strip=2 or no match for short=refs/tags/)".