Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: monitor-edid - Tool for probing and parsing EDID https://bugzilla.redhat.com/show_bug.cgi?id=437691 ville.skytta@xxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@xxxxxxxxxxxxxxxxx |ville.skytta@xxxxxx Status|NEW |ASSIGNED Flag| |fedora-review? ------- Additional Comments From ville.skytta@xxxxxx 2008-03-16 15:24 EST ------- License: GPLv2 is incorrect. The tarball includes COPYING which is the LGPLv2 text, but none of the source files have any GPL or LGPL references, instead they have various BSD/MIT like notices. Providing a checkout script would be more convenient than including comments in the specfile. Could also use svn export instead of checkout (does not checkout .svn directories), and bzip2 or lzma the tarball instead of gzipping to save space. Looks also like there's a private copy of lrmi in the tarball, is there a reason why the Fedora packaged one is not used instead? Including the word "monitor" in the Summary would be good. Will continue the review after the license issues have been sorted out. -- 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, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review