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=501854 Orcan 'oget' Ogetbil <oget.fedora@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |oget.fedora@xxxxxxxxx Flag|fedora-review? |fedora-review+ --- Comment #2 from Orcan 'oget' Ogetbil <oget.fedora@xxxxxxxxx> 2009-06-10 14:52:23 EDT --- I'm done reviewing this. There are two minor issues to be resolved: - rpmlint is silent - There are certain files in the source, such as include/lcdf/hashmap.cc include/lcdf/vector.cc include/lcdf/clp.h liblcdf/string.cc liblcdf/error.cc liblcdf/vectorv.cc liblcdf/clp.c liblcdf/straccum.cc that are under MIT license. But since during the compilation, they are linked to GPL parts of the code, and since GPL is stricter than MIT, the overall license as GPLv2+ is acceptable. - koji rawhide build is fine: http://koji.fedoraproject.org/koji/taskinfo?taskID=1403904 * The directory %{_datadir}/lcdf-typetools/ is unowned! This needs attention. * Please include the COPYING file in %doc, as required by the guidelines. Please do these changes before you commit. ------------------------------------------------- This package (lcdf-typetools) is APPROVED by oget ------------------------------------------------- -- 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