[Bug 549884] Review Request: plexus-io - Plexus IO Components

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]