[Bug 227309] Review Request: seom - Desktop video capture and playback utility

[bugzilla at redhat.com]

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 at ioa.s.u-tokyo.ac.jp  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.