Bugzilla (bugzilla.redhat.com) will be under maintenance for infrastructure upgrades and will not be available on July 31st between 12:30 AM - 05:30 AM UTC. We appreciate your understanding and patience. You can follow status.redhat.com for details.
Bug 230892 - Review Request: gtk-recordmydesktop - GUI Desktop session recorder with audio and video
Summary: Review Request: gtk-recordmydesktop - GUI Desktop session recorder with audio...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: manuel wolfshant
QA Contact: Fedora Package Reviews List
Depends On: 230891
TreeView+ depends on / blocked
Reported: 2007-03-04 10:46 UTC by Sindre Pedersen Bjørdal
Modified: 2012-02-08 13:04 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-03-14 16:25:41 UTC
Type: ---
wolfy: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Sindre Pedersen Bjørdal 2007-03-04 10:46:56 UTC
Spec URL: http://folk.ntnu.no/sindrb/packages/gtk-recordmydesktop.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/gtk-recordmydesktop-


Graphical frontend for the recordmydesktop desktop session recorder.

recordMyDesktop is a desktop session recorder for linux that attempts to be 
easy to use, yet also effective at it's primary task.

Comment 1 manuel wolfshant 2007-03-05 12:10:06 UTC
MUSTFIXes: missing BR gettext,desktop-file-utils


- package meets naming guidelines
- package meets packaging guidelines
- license (GPL ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream, is latest available version, sha1sum
d1ba549ebff68157912b41e332941d84836d1fbb gtk-recordmydesktop-
- package compiles on devel (x86_64) [*]
- no unnecessary BR
- locales properly handled [*]
- not relocatable
- owns all files/directories that it creates, does not take ownership of foreign
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for separate -docs
- nothing in %doc affects runtime
- there are no scriptlets, no static code, no libtool/pkgconfig files

- the package compiles and builds into binary rpms on FC6 and rawhide (tested
only on x86_64); generated binary is a noarch so no issues should exist for x86 [*]
- the package runs as advertised [*]

[*] after adding the missing BRs

Sindre, please fix the spec adding the missing BRs and I'll reverify/approve the
package. As a sidenote, in the future I think it would be a good ideaif you
would test your packages using rpmlint and by bulding in mock before submitting

Comment 3 manuel wolfshant 2007-03-06 00:24:20 UTC
Good, we are almost done. All problems mentioned have been solved. There is one
more left: please preserve the timestamps when installing. This can be done by
adding INSTALL="{__install} -c -p" either to the %configure line or to the "make
install" line.

Additional review bits not touched before
- python libs are properly installed (package follows the guidelines from
- program is a GUI; a desktop file is properly installed for it
- the desktop file respects packaging guidelines (no output from
desktop-file-validate fedora-gtk-recordmydesktop.desktop)

As I see no further blockers, the package is APPROVED. I'll trust you to fix 
preserving of timestamps before importing into CVS.

Comment 4 Sindre Pedersen Bjørdal 2007-03-06 07:42:27 UTC
New Package CVS Request
Package Name: gtk-recordmydesktop
Short Description: GUI Desktop session recorder with audio and video
Owners: foolish@guezz.net
Branches: FC-5 FC-6

Comment 5 Andy Grover 2012-02-08 01:21:26 UTC
Package Change Request
Package Name: gtk-recordmydesktop
New Branches: el5 el6
Owners: mbooth narasim grover

Comment 6 Gwyn Ciesla 2012-02-08 13:04:16 UTC
Git done (by process-git-requests).

Note You need to log in before you can comment on or make changes to this bug.