[Bug 487] Review request: libmimic - Encoding/decoding library for Mimic V2.x

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=487





--- Comment #8 from Andrea Musuruane <musuruan at gmail.com>  2009-08-07 10:17:59 ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistent macro usage.
OK - Meets Packaging Guidelines.
OK - License (LGPLv2+)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
94f0dbb1d3c253201553a4069555fb84  libmimic-1.0.4.tar.gz
94f0dbb1d3c253201553a4069555fb84  libmimic-1.0.4.tar.gz.orig
NA - Package needs ExcludeArch
see below - BuildRequires correct
NA - Spec handles locales/find_lang
NA - Package is relocatable and has a reason to be.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
OK, RPM Fusion - Package is code or permissible content.
NA - Doc subpackage needed/used.
OK - Packages %doc files don't affect runtime.

OK - Headers/static libs in -devel subpackage.
OK - Spec has needed ldconfig in post and postun
OK - .pc files in -devel subpackage/requires pkgconfig
OK - .so files in -devel subpackage.
OK - -devel package Requires: %{name} = %{version}-%{release}
OK - .la files are removed.

NA - Package is a GUI app and has a .desktop file

see below - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
couldn't check - No rpmlint output.
couldn't check - final provides and requires are sane

SHOULD Items:

see below - Should build in mock.
see below - Should build on all supported archs
see below - Should function as described.
OK - Should have sane scriptlets.
OK - Should have subpackages require base package with fully versioned depend.
OK - Should have dist tag
OK - Should package latest version
NA - check for outstanding bugs on package. (For core merge reviews)

Issues:

1. Build in mock failed under:
fedora-10-x86_64
fedora-11-i386
fedora-11-x86_64

[...]
 cat .libs/libmimic.exp | sed -e "s/\(.*\)/\1;/" >> .libs/libmimic.ver
 echo "local: *; };" >> .libs/libmimic.ver
 gcc -shared  .libs/mimic.o .libs/encode.o .libs/decode.o .libs/bitstring.o
.lib
s/vlc_common.o .libs/vlc_encode.o .libs/vlc_decode.o .libs/fdct_quant.o
.libs/id
ct_dequant.o .libs/colorspace.o .libs/deblock.o  -lglib -lm  -m64
-mtune=generic
 -Wl,-soname -Wl,libmimic.so.0 -Wl,-version-script -Wl,.libs/libmimic.ver -o
.li
bs/libmimic.so.0.0.1
/usr/bin/ld: cannot find -lglib
collect2: ld returned 1 exit status
make[2]: *** [libmimic.la] Error 1
make[2]: Leaving directory `/builddir/build/BUILD/libmimic-1.0.4/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/builddir/build/BUILD/libmimic-1.0.4'
make: *** [all] Error 2

2. How can I test this library? Is there any program that can already use it?


-- 
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.