This service will be undergoing maintenance at 00:00 UTC, 2016-09-28. It is expected to last about 1 hours
Bug 230891 - Review Request: recordmydesktop - Desktop session recorder with audio and video
Review Request: recordmydesktop - Desktop session recorder with audio and video
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: manuel wolfshant
Fedora Package Reviews List
:
Depends On:
Blocks: 230892
  Show dependency treegraph
 
Reported: 2007-03-04 05:44 EST by Sindre Pedersen Bjørdal
Modified: 2012-02-08 08:02 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-14 12:24:54 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
wolfy: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Sindre Pedersen Bjørdal 2007-03-04 05:44:52 EST
Spec URL: http://folk.ntnu.no/sindrb/packages/recordmydesktop.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/recordmydesktop-0.3.3.1-1.src.rpm

Description: 

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-04 20:24:58 EST
Just a quick note: there are several duplicate BuildRequires: libXfixes-devel
(by libXdamage-devel), libICE-devel (by libSM-devel), libogg-devel (by
libtheora-devel) (all those are minor problems) and at least one missing BR
(zlib-devel probably - that is a MUSTFIX):

In file included from ../include/recordmydesktop.h:35,
                 from recordmydesktop.c:28:
../include/rmdtypes.h:56:18: error: zlib.h: No such file or directory
In file included from ../include/recordmydesktop.h:35,
                 from recordmydesktop.c:28:
../include/rmdtypes.h:229: error: expected specifier-qualifier-list before 'gzFile'
In file included from ../include/recordmydesktop.h:37,
                 from recordmydesktop.c:28:
../include/rmdfunc.h:465: error: expected declaration specifiers or '...' before
'gzFile'
../include/rmdfunc.h:481: error: expected declaration specifiers or '...' before
'gzFile'
make[2]: *** [recordmydesktop-recordmydesktop.o] Error 1
make[2]: Leaving directory `/builddir/build/BUILD/recordmydesktop-0.3.3.1/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/builddir/build/BUILD/recordmydesktop-0.3.3.1'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.72949 (%build)


RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.72949 (%build)
Comment 2 manuel wolfshant 2007-03-04 20:43:24 EST
here is a short patch which fixes the above and also preserves timestamps of
installed files. Please use it to fix your package and I guess we have a winner.
#diff -Nupr recordmydesktop.spec recordmydesktop-new.spec

--- recordmydesktop.spec        2007-03-05 03:26:39.000000000 +0200
+++ recordmydesktop-new.spec    2007-03-05 03:37:59.000000000 +0200
@@ -1,6 +1,6 @@
 Name:           recordmydesktop
 Version:        0.3.3.1
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        Desktop session recorder with audio and video

 Group:          Applications/Multimedia
@@ -36,13 +36,11 @@ make %{?_smp_mflags}

 %install
 rm -rf $RPM_BUILD_ROOT
-make install DESTDIR=$RPM_BUILD_ROOT
-
+make install DESTDIR=$RPM_BUILD_ROOT INSTALL="%{__install} -c -p"

 %clean
 rm -rf $RPM_BUILD_ROOT

-
 %files
 %defattr(-,root,root,-)
 %doc AUTHORS COPYING README ChangeLog
Comment 3 Sindre Pedersen Bjørdal 2007-03-05 05:15:16 EST
Changes:

- Remove duplicate BR
- Add missing zlib-devel BR
- Applied patch to Preserve timestamps

Updated:

SPEC: http://folk.ntnu.no/sindrb/packages/recordmydesktop.spec
SRPM: http://folk.ntnu.no/sindrb/packages/recordmydesktop-0.3.3.1-2.src.rpm
Comment 4 manuel wolfshant 2007-03-05 05:58:56 EST
Good:

- rpmlint checks do not return anything on source and binary rpm; the debuginfo
package reports:
W: recordmydesktop-debuginfo spurious-executable-perm
/usr/src/debug/recordmydesktop-0.3.3.1/src/load_cache.c
W: recordmydesktop-debuginfo spurious-executable-perm
/usr/src/debug/recordmydesktop-0.3.3.1/include/rmdtypes.h
W: recordmydesktop-debuginfo spurious-executable-perm
/usr/src/debug/recordmydesktop-0.3.3.1/src/cache_frame.c
Probably a chmod -x on these three files in %prep should make rpmlint happy.
- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) is OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream, is latest version, sha1sum
3eb2ffe79a0fcf5e04962bd3d120c1e5587c178b recordmydesktop-0.3.3.1.tar.gz
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all file/directories that it creates, does not take ownership of foreign
files/directories
- no duplicate files
- permissions ok (see rpmlint output on -debuginfo above)
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- not a GUI, so no need for .desktop file
- no need for a separate -devel package
- no .la files
- no scriptlets
- no static files

 SHOULD:
- builds in mock for devel/x86_64
- works as advertised

APPROVED
Comment 5 Sindre Pedersen Bjørdal 2007-03-05 19:06:31 EST
New Package CVS Request
=======================
Package Name: recordmydesktop
Short Description: Desktop session recorder with audio and video
Owners: foolish@guezz.net
Branches: FC-5 FC-6
InitialCC: 
Comment 6 Andy Grover 2012-02-07 20:20:10 EST
hi, I'd like to get this (and gtk/qt interfaces) built for epel.

Package Change Request
======================
Package Name: recordmydesktop
New Branches: el5 el6
Owners: mbooth narasim grover
Comment 7 Jon Ciesla 2012-02-08 08:02:39 EST
Git done (by process-git-requests).

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