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=500254 Andrew Overholt <overholt@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |overholt@xxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |overholt@xxxxxxxxxx Flag| |fedora-review? --- Comment #1 from Andrew Overholt <overholt@xxxxxxxxxx> 2009-05-14 15:54:17 EDT --- It would be nice to have a URL for source0. I found it upstream regardless, matched the md5sum upstream to the one I downloaded and to the one in the SRPM. I looked through a random sampling of 5 or so source files and they all say ASL 2.0 as does LICENSE.txt. Therefore, the License field should reflect this. There's a one line difference between the pom file included in the SRPM and the one from upstream (line 41): - <organization/> + <organization></organization> Otherwise, pre-review checklist seems okay to me. -- 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