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=620826 Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomspur@xxxxxxxxxxxxxxxxx Blocks| |177841(FE-NEEDSPONSOR) --- Comment #1 from Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> 2010-08-04 05:24:32 EDT --- Just some comments (I'm no sponsor anyway): - How do you get GPLv3+? Website says LGPL, There is a LGPLv2+ license and no headers on the first sign. - You could add a %check section for running the testsuite. No issues beyond that, your %files section looks nice, not much are so explicit like you (how it should be done). -- 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