> > diff --git a/builtin/verify-tag.c b/builtin/verify-tag.c
> > index f776778..8abc357 100644
> > --- a/builtin/verify-tag.c
> > +++ b/builtin/verify-tag.c
> > @@ -30,6 +30,8 @@ int cmd_verify_tag(int argc, const char **argv, const 
> > char *prefix)
> >  {
> >     int i = 1, verbose = 0, had_error = 0;
> >     unsigned flags = 0;
> > +   unsigned char sha1[20];
> > +   const char *name;
> >     const struct option verify_tag_options[] = {
> >             OPT__VERBOSE(&verbose, N_("print tag contents")),
> >             OPT_BIT(0, "raw", &flags, N_("print raw gpg status output"), 
> > GPG_VERIFY_RAW),
> > @@ -46,8 +48,16 @@ int cmd_verify_tag(int argc, const char **argv, const 
> > char *prefix)
> >     if (verbose)
> >             flags |= GPG_VERIFY_VERBOSE;
> >  
> > -   while (i < argc)
> > -           if (pgp_verify_tag(argv[i++], flags))
> > +   while (i < argc) {
> > +           name = argv[i++];
> > +           if (get_sha1(name, sha1)) {
> > +                   error("tag '%s' not found.", name);
> >                     had_error = 1;
> > +           }
> > +
> > +           if (pgp_verify_tag(name, NULL, sha1, flags))
> > +                   had_error = 1;
> > +
> > +   }
> 
> So this is a good example of the rippling I mentioned earlier.
> 
> As a side note, it might actually be an improvement for pgp_verify_tag
> to take a sha1 (so that git-tag is sure that it is verifying the same
> object that it is printing), but that refactoring should probably come
> separately, I think.
> 
> -Peff

Just to be sure, this refactoring is something we should still include
in this set of patches, right? I think that otherwise we'd lose the
desambigutaion that git tag -v does in this patch.

I also think that most of the rippling is gone if we use and adaptor as
you suggested. Should I add a patch on top of this to support a sha1 as
part for gpg_verify_tag()?

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

Reply via email to