On Mon, Apr 4, 2016 at 9:46 PM, Santiago Torres <santiago@xxxxxxx> wrote: > Eric Sunshine wrote: >> > +test_expect_success GPG 'verify multiple tags' ' >> > + tags="fourth-signed sixth-signed seventh-signed" && >> > + for i in $tags; do >> > + git verify-tag -v --raw $i || return 1 >> > + done >expect.stdout 2>expect.stderr.1 && >> > + grep GOODSIG <expect.stderr.1 >expect.stderr && >> > + git verify-tag -v --raw $tags >actual.stdout 2>actual.stderr.1 && >> > + grep GOODSIG <actual.stderr.1 >actual.stderr && >> >> Hmm, I had expected you to adopt Peff's suggestion[1] for the greps: >> >> grep '^.GNUPG:.' ... >> >> [1]: http://article.gmane.org/gmane.comp.version-control.git/290691 > > I thought this was an stylistic thing. I can of course adopt this > suggestion. It's probably not a big deal, but Peff's suggestion (at least feels like it) makes the test a bit more comprehensive. By the way, as the test is heavily inspired by Peff's example, it might be worth giving him a nod via a Helped-by: just above your Signed-off-by:. -- 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