Re: [PATCH 6/6] builtin/tag: add --format argument for tag -v

2016-09-23 Thread Santiago Torres
> OK, you said something about for_each_ref() in an earlier commit, > but what you meant was this one, which takes each_tag_name_fn. Oh yeah, sorry for the confusion. > > The function for_each_tag_name(), the type each_tag_name_fn, and the > function of that type verify_tag(), are ALL

Re: [PATCH 6/6] builtin/tag: add --format argument for tag -v

2016-09-22 Thread Junio C Hamano
santi...@nyu.edu writes: > @@ -425,8 +431,11 @@ int cmd_tag(int argc, const char **argv, const char > *prefix) > die(_("--merged and --no-merged option are only allowed with > -l")); > if (cmdmode == 'd') > return for_each_tag_name(argv, delete_tag); > - if

[PATCH 6/6] builtin/tag: add --format argument for tag -v

2016-09-22 Thread santiago
From: Lukas P Adding --format to git tag -v mutes the default output of the GPG verification and instead prints the formatted tag object. This allows callers to cross-check the tagname from refs/tags with the tagname from the tag object header upon GPG verification.