On Tue, Jun 07, 2016 at 03:05:50PM -0700, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > On Tue, Jun 07, 2016 at 02:50:23PM -0700, Junio C Hamano wrote: > > > >> >> Or it could even do this: > >> >> > >> >> tag="$1" > >> >> if ! git tag -v "$tag" > >> >> if ! git tag -v "$tag" > >> >> then > >> >> echo >&2 "Bad tag." > >> >> exit 1 > >> >> fi > >> >> + tag=$(git tag --show-tagname $tag) > >> >> make dest=/usr/local/$package/$tag install > >> > > >> > It is racy. That probably doesn't matter for most callers, but it would > >> > be nice to be able to get a custom format out of the "-v" invocation. > >> > >> Heh, you can do > >> > >> - tag="$1" > >> + tag=$(git rev-parse --verify "$1") > >> > >> upfront and it no longer is racy, no? > > > > Yes, though that doesn't quite work today. The formatted output comes > > from "tag -l", which wants a refname. > > Puzzled. I didn't even use --format=%(tagname) in the above. No, but you used --show-tagname, which does not exist today (and which IMHO should be implemented as --format). Would --show-tagname take either a tagname _or_ a sha1? I assume it would not be calling get_sha1(), as having it find refs/heads/$tag would be silly. -Peff -- 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