Bug 606064 - Review Request: libfc14audiodecoder - C wrapper library for Future Composer audio decoding
Review Request: libfc14audiodecoder - C wrapper library for Future Composer a...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Michel Alexandre Salim
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2010-06-20 06:27 EDT by Michael Schwendt
Modified: 2010-06-26 06:40 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-06-26 06:40:53 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
michel: fedora‑review+
tibbs: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Michael Schwendt 2010-06-20 06:27:17 EDT
Spec URL: http://mschwendt.fedorapeople.org/libfc14audiodecoder.spec
SRPM URL: http://mschwendt.fedorapeople.org/libfc14audiodecoder-1.0.0-1.fc13.src.rpm
Description: This library provides a C API for a Future Composer audio decoder, which
has been used in several plug-ins for versatile audio players like XMMS,
BMP, Audacious and GStreamer.

This library will be needed for future updates of gstreamer-plugins-fc and audacious-plugin-fc.
Comment 1 Michel Alexandre Salim 2010-06-25 07:23:09 EDT
Everything looks in good shape. APPROVED

* DONE Review [100%]
** DONE Names [2/2]
*** DONE Package name
*** DONE Spec name
** DONE Meets [[http://fedoraproject.org/wiki/Packaging/Guidelines][guidelines]]
** DONE source files match upstream
   sha1sum: f11f7aff4f59c669a07a5f3481cb69505270b008
** DONE License [3/3]
*** DONE License is Fedora-approved
*** DONE License field accurate
    all C++ source files have the "later version" provision
*** DONE License included iff packaged by upstream
** DONE rpmlint [2/2]
*** DONE on src.rpm
    libfc14audiodecoder.src: E: unknown-key GPG#b8af1c54
    -> private signing key, ignoring "error"
    libfc14audiodecoder.src: W: no-buildroot-tag
    -> fine for F-13+

*** DONE on x86_64.rpm
    Spurious warnings from rpmlint spellcheck; can be ignored

** DONE Language & locale [3/3]
*** DONE Spec in US English
*** DONE Spec legible
*** N/A Use %find_lang to handle locale files
** DONE Build [3/3]
*** DONE Koji results
*** DONE BRs complete
*** DONE Directory ownership
** DONE Spec inspection [8/8]
*** DONE ldconfig for libraries
*** DONE No duplicate files
*** DONE File permissions
*** DONE Filenames must be UTF-8
*** DONE Has %clean section
    (except F-13+:
*** DONE %buildroot cleaned on %install
2*** DONE Macro usage consistent
*** DONE Documentation [2/2]
**** N/A If large docs, separate -doc
**** DONE %doc files are non-essential
*** DONE Development [5/5]
**** DONE Headers in -devel
**** DONE If versioned .so's, unversioned in -devel
**** N/A Static only if necessary, put in -static
**** DONE -devel, -static requires main
**** DONE No .la
Comment 2 Michael Schwendt 2010-06-25 13:39:07 EDT
Great! Thank you!

New Package CVS Request
Package Name: libfc14audiodecoder
Short Description: C wrapper library for Future Composer audio decoding
Owners: mschwendt
Branches: F-12 F-13
Comment 3 Jason Tibbitts 2010-06-26 04:02:46 EDT
CVS done (by process-cvs-requests.py).

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