Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=550067 --- Comment #5 from Philip Prindeville <philipp@xxxxxxxxxxxxxxxxxxxxx> 2010-01-30 14:06:18 EST --- (In reply to comment #4) > Haven't finished review yet but I've stumbled on a couple of problems. > > [...] > The spec file (and META.yaml) say this module requires Test::More but I'm not > saying this in the code. Everything else looks fine but I'ld rather hold off > review until the above two items are fixed. > > FAIL. Look in Makefile.PL and in t/*.t ... that's the test suite. You'll see that it's used there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review