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 #7 from Steven Dake <sdake@xxxxxxxxxx> 2012-01-17 14:54:39 EST --- This is first java package I have reviewed, so forgive my ignorance here: line 23-24: Requires(post): jpackage-utils Requires(postun): jpackage-utils Requirement says "Package _DOES NOT_ have Requires(post/postun) on jpackage-utils There is clearly a requires on jpackage-utils in the post/postun section line 43-47: %post %update_maven_depmap %postun %update_maven_depmap Requirement says "Package _DOES NOT_ use %update_maven_depmap in %post/%postun" They are clearly used in these sections. Since java is out of my area of expertise (I normally review python/c packages), if you can find a confident java packager to signoff on this portion, obtain an exception from the java sig, or have someone from the java sig educate me on the rationale for these requirements, I'd be happy to approve the package. Regards -steve -- 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