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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jeff King <peff@xxxxxxxx> 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/)".

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]