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

[Jeff King <peff@peff.net>]

On Sun, Jan 24, 2016 at 02:19:52PM -0800, Junio C Hamano wrote:

> >> Instead, let's extend the "short" modifier in the formatting
> >> language to handle a specific prefix. This fixes "git tag",
> >> and lets users invoke the same behavior from their own
> >> custom formats (for "tag" or "for-each-ref") while leaving
> >> ":short" with its same consistent meaning in all places.
> 
> Yeah, I do agree with the analysis.
> 
> I however wonder if short=$prefix is a good end-user interface,
> though, as strip=$prefix may be more intuitive to them, I suspect.

Yeah, I picked "short" just because of the existing feature. But we are
not bound to use the same name at all, and "strip" is probably more
descriptive (I thought "prefix" might also be, but that only
communicates what it _is_, not that we are removing it).

> Also there is "what happens if the expected prefix is not there?"
> question.

I think "do not strip anything" (as I have here) is an OK behavior. It
would not come up for sane requests (i.e., you would generally be
filtering to match your prefix anyway).

But...

> Perhaps strip=2 can be defined to "strip 2 levels of
> hierarchy prefix no matter what that is", and strip refs/tags/foo,
> refs/heads/foo and refs/remotes/origin/foo to foo, foo, origin/foo,
> respectively?  The filtering natively done by the listing mode of
> "branch" and "tags" would ensure the prefixes are always what we
> implicitly expect, so the case we need to worry about how we should
> signal errors becomes "what if there are not enough levels".  That
> may be simpler to handle.

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, but
maybe there are special cases, like:

  git for-each-ref --format='%(refname:strip=3)' refs/heads/jk/

to get my list of topics, sans initials.

I had originally hoped to avoid exposing any of this to the user, and
just make things internal, so that we would not be locked into a
particular formatting behavior. But since we now have "tag --format" and
advertise "%(refname:short)" as its default, I think it has become
user-visible.

Let me see what the "strip=X" patch looks like.

-Peff