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: seom - Desktop video capture and playback utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227309 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-02-05 10:36 EST ------- Well, * License - No license text exists, and no source code contains license term. tomc, do you agree that this is licensed under GPL? If so, please include GPL license document to your tarball from next release. * Requires for -devel package - Is "freeglut-devel" for Requires on -devel package is needed? This seems to be of no relation for this package. And according to the header files included, libX11-devel is needed for Requires on -devel package. Note that libX11-devel automatically pulls mesa-libGL-devel. * Timestamps - Generally keeping timestamps on text files, such as header files is preferrred. For this package, this can be done by replacing all "install" in Makefile by "install -p". And .. libX11-devel requires mesa-libGL-devel finally. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review