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=549884 --- Comment #2 from Alexander Kurtakov <akurtako@xxxxxxxxxx> 2009-12-23 09:18:56 EDT --- (In reply to comment #1) > I made a full review. Here are my comments, questions etc: > > * Package does not build in mock. It pulls gcj. We want openjdk. I added that > BR fol the rest of the review. > As far as I know, all java packages need these BR's and R's: > http://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires Fixed. > > * Additionally, the guidelines tell us to include these R's for maven: > http://fedoraproject.org/wiki/Packaging:Java#maven Fixed. > > * rpmlint says > plexus-io.noarch: W: non-conffile-in-etc /etc/maven/fragments/plexus-io > Is this ignorable? Why? This is the place where mvn looks for fragments. And they are clearly non conffile because users should not edit them. > > ! javadoc package does not require the main package. (just a warning. I don't > know if this is intentional) Perfectly ok. There are many times when you want to install just the javadoc for local reference and studing the API without any need to use it. > > ? Shouldn't the release tag just be 0.1.a5%{?dist} Fixed. > > ! For macro consistency plexus-io can be replaced by %{name} at two places Fixed Spec URL: http://akurtakov.fedorapeople.org/plexus-io.spec SRPM URL: http://akurtakov.fedorapeople.org/plexus-io-1.0-0.2.a5.fc12.src.rpm -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review