Bug 223039 - Review Request: libgimme-codec - Helper library for automated codecs installation
Review Request: libgimme-codec - Helper library for automated codecs installa...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Matthias Clasen
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2007-01-17 12:16 EST by Bastien Nocera
Modified: 2007-11-30 17:11 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-02-19 11:01:57 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
updated spec (1.69 KB, text/plain)
2007-01-19 21:17 EST, Matthias Clasen
no flags Details

  None (edit)
Description Bastien Nocera 2007-01-17 12:16:52 EST
Spec URL: http://people.redhat.com/bnocera/libgimme-codec/libgimme-codec.spec
SRPM URL: http://people.redhat.com/bnocera/libgimme-codec/libgimme-codec-0.0.0-1.src.rpm
libgimme-codec allows programs that use it to call a helper script when
a video or audio codec is unhandled by the media framework in use.
Comment 1 Bastien Nocera 2007-01-17 12:17:42 EST
libgimme-codec is an optional dependency of Totem, which is in Core.
Comment 2 Christopher Stone 2007-01-17 12:21:33 EST
buildroot does not match preferred tag:
Comment 4 Matthias Clasen 2007-01-17 12:41:43 EST
Currently libgimme-codec requires cvs gstreamer ( Bastian says that a
new gstreamer release is expected soon.
Comment 5 Bastien Nocera 2007-01-17 12:48:43 EST
(In reply to comment #4)
> Currently libgimme-codec requires cvs gstreamer ( Bastian says that a
> new gstreamer release is expected soon.

Bit of a confusion there I guess. libgimme-codec support in Totem requires
gstreamer core, libgimme-codec needs nothing but a libc to compile.
Comment 6 Matthias Clasen 2007-01-17 20:48:03 EST
Oh, my bad
Comment 7 Matthias Clasen 2007-01-18 21:37:59 EST
Some more comments before starting a real review:

I have to wonder if this single 200 line source file + header is really 
worth packaging as a shared library. Wouldn't it be much easier to just
copy these two files into the applications where it is needed ?

Anyway, if we want to go ahead with this micro-package, rpmlint has some complaints:

rpmlint libgimme-codec-0.0.0-1.i386.rpm 
E: libgimme-codec no-ldconfig-symlink /usr/lib/libgimme-codec.so.0.0.0
W: libgimme-codec no-documentation

rpmlint libgimme-codec-devel-0.0.0-1.i386.rpm 
W: libgimme-codec-devel no-documentation
E: libgimme-codec-devel script-without-shebang /usr/include/gimme-codec.h


Requires: %name = %{version}

should be %{version}-%{release}
Comment 8 Bastien Nocera 2007-01-19 05:20:23 EST
Totem wouldn't be the only application using this library, Rhythmbox, and other
GStreamer-based applications would be using this library as well. It would also
probably be the place where the Fedora/RHEL specific download-scripts would be
Comment 9 Matthias Clasen 2007-01-19 21:17:13 EST
Created attachment 146050 [details]
updated spec

Here is a slightly updated spec that addresses the rpmlint errors. it installs 

a .0 symlink for the library and installs the headers with proper permissions.
Comment 10 Matthias Clasen 2007-01-19 21:57:04 EST
Starting a formal review based on the modified spec:

+ rpmlint only warns about missing docs, which is fine
+ package is named according to the naming guidelines
+ spec name equal package name
+ spec follows packaging guidelines
+ license is LGPL
+ license tag matches actual license
+ tarball doesn't include license file
+ spec file is in English
+ spec file is legible
+ md5sums of upstream source matches
+ package built fine in mock
+ BuildRequires are complete
+ no locale data
+ ldconfig called in %post and %postun
+ package not relocatable
+ package owns all created directories
+ file lists contain no dupes
+ files have proper permissions, all file lists have %defattr
+ %clean is there
+ macros are used consistently
+ package contains code
+ headers are in -devel
+ -devel requires base
+ no .pc files 
+ no .la files
+ no desktop files
+ package does not own standard directories

Comment 11 Bastien Nocera 2007-01-26 12:16:01 EST
Putting this on hold, as it's pretty much agreed that GStreamer should provide
this functionality directly:

Comment 12 Bastien Nocera 2007-02-19 11:01:57 EST
The functionality has been moved to GStreamer itself, closing.

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