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=782225 --- Comment #3 from Steven Dake <sdake@xxxxxxxxxx> 2012-01-16 19:40:11 EST --- From:http://fedoraproject.org/wiki/Java_review_template Maven === [PASS] Use %{_mavenpomdir} macro for placing pom files instead of %{_datadir}/maven2/poms [N/A] If package uses "-Dmaven.test.skip=true" explain why it was needed in a comment [N/A] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why it's needed in a comment [FAIL] Package DOES NOT use %update_maven_depmap in %post/%postun [FAIL] Packages DOES NOT have Requires(post) and Requires(postun) on jpackage-utils for %update_maven_depmap macro The last two issues are blockers, although I don't understand maven enough to know why. -- 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