Bug 529569 - (mingw32-speex) Review Request: mingw32-speex - MinGW build of the Speex voice compression codec library
Review Request: mingw32-speex - MinGW build of the Speex voice compression co...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
low Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On: mingw32-libogg
  Show dependency treegraph
Reported: 2009-10-18 14:43 EDT by Mihai Limbășan
Modified: 2010-12-17 10:25 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-12-17 10:25:23 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Mihai Limbășan 2009-10-18 14:43:11 EDT
Spec URL: http://rpms.limbasan.ro/fedora/11/SPECS/mingw32-speex.spec
SRPM URL: http://rpms.limbasan.ro/fedora/11/SRMPS/mingw32-speex-1.2-2.rc1.fc11.src.rpm
Description: This is a MinGW crosscompiler port of the speex package.

The spec file was created based on the example at


and on the main Fedora speex spec, taking into account the MinGW SIG packaging
guidelines at http://fedoraproject.org/wiki/Packaging/MinGW .

rpmlint output:

[mimock@home syncup]$ rpmlint mingw32-speex-1.2-2.rc1.fc11.noarch.rpm mingw32-speex-1.2-2.rc1.fc11.src.rpm mingw32-speex-static-1.2-2.rc1.fc11.noarch.rpm mingw32-speex-tools-1.2-2.rc1.fc11.noarch.rpm
mingw32-speex-static.noarch: E: arch-independent-package-contains-binary-or-object /usr/i686-pc-mingw32/sys-root/mingw/lib/libspeex.a
mingw32-speex-static.noarch: E: arch-independent-package-contains-binary-or-object /usr/i686-pc-mingw32/sys-root/mingw/lib/libspeexdsp.a
mingw32-speex-static.noarch: W: no-documentation
mingw32-speex-tools.noarch: W: no-documentation
4 packages and 0 specfiles checked; 2 errors, 2 warnings.

which are OK accoring to the packaging guidelines.

Beside the usual static library subpackage, the command line encoder and decoder have been separated into a -tools subpackage to avoid having the speex codec libraries Require mingw32-libogg.

The -m4 patch is Fedora's -m4 patch, corrected to apply cleanly on 1.2rc1.

The -acconf patch helps in replacing the ancient config.sub, config.guess, and
libtool shipped by upstream with current versions.

The package BuildRequires, and the tools subpackage Requires the mingw32-libogg package - see bug #529548 - which is available at the same test repo ( http://rpms.limbasan.ro/fedora/11/i386/ ). mingw32-speex builds cleanly in mock. If there's any way to get Koji to perform a scratch build using third party packages, I couldn't figure it out, so there's no Koji link for the moment.
Comment 1 Richard W.M. Jones 2009-10-19 08:21:34 EDT
Package looks sane.  Same comment re static libraries.
Comment 2 Mihai Limbășan 2009-10-19 15:03:32 EDT
Improved package, bumped revision. New URLs:

Spec URL: 

Changes as follows:
- Removed -static and -tools packages.
- aclocal and pkgconfig directories under _mingw32_libdir are
  no longer installed, mingw32-filesystem already provides them.
- Removed redundant BuildRequire mingw32-binutils which is
  already Required by mingw32-gcc.
- Prepared the check build stage for whenever speex get checks.
- Cosmetic cleanup.

rpmlint says:

[mimock@home mingw32-speex]$ rpmlint -v *rpm
mingw32-speex.noarch: I: checking
mingw32-speex.src: I: checking
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Again no Koji link due to dependency on mingw32-libogg (bug #529548). Upstream does not seem to provide any tests, but it builds cleanly in mock.
Comment 4 Jason Tibbitts 2010-11-02 10:17:35 EDT
Blocking needsponsor as you do not appear to be a member of the packager group.

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