[Bug 519236] Review Request: plexus-resources - Plexus Resource Manager

[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=519236


Jeff Johnston <jjohnstn@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jjohnstn@xxxxxxxxxx
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |jjohnstn@xxxxxxxxxx
               Flag|                            |fedora-review?,
                   |                            |needinfo?(overholt@xxxxxxxx
                   |                            |om)




--- Comment #2 from Jeff Johnston <jjohnstn@xxxxxxxxxx>  2009-08-25 16:31:34 EDT ---
rpmlint successful

Comments:

1. Packaging naming convention has the number incrementing ahead of the
   alphatag.  So release should be 0.2.a4 and then 0.3.a4 etc..

2. Javadoc description should not reference itself (use API documentation for
   xxx)

3. No %doc files

4. post and postun sections are maven-only.  Is this a problem since with_maven
   is 0?

5. defattr non-standard.

-- 
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]