Spec URL: https://www.box.com/s/as9w085d9tluv7x7u4ni SRPM URL: https://www.box.com/s/4vkjsrgbkgy37r0xo99t Description: Contains runtime shared libraries for libgsm, an implementation of the European GSM 06.10 provisional standard for full-rate speech transcoding, prI-ETS 300 036, which uses RPE/LTP (residual pulse excitation/long term prediction) coding at 13 kbit/s. This is the MinGW version of a package that Fedora already has in its repository. Fedora Account System Username: ulatekh
Issues found: * build dependencies: - missing mingw64 tools - it doesn't build because of that - gzip not needed (http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires_2) - C++ not needed - mingwXX-binutils can be removed (will be picked by mingwXX-gcc) * manpages are not needed in MinGW packages * mingw patch: - there is added -fno-stack-protector to CFLAGS, is it needed? (and why?) - is it needed to comment out *_INSTALL* things and tools? - it could be more generic (using variable for adding .exe to program names, ...) - (it can be compared with other patches "in the wild": http://lrn.no-ip.info/other/mingw/mingw32/gsm/1.0.13-2/) - more generic version could be sent to upstream * RHEL5 stuff can be removed (Group, BuildRoot, %clean, rm -rf, %defattr, ...) * %prep: - better to use 'cp -a' to preserve timestamps - gsm doesn't support the out of source build; I haven't found problem in the way used in this .spec file, but I don't know the standard recommended way of doing it in Fedora MinGW packages... * cosmetic: mixed tab/space rpmlint warning * library name could be rather libgsm-1.dll? But I'm not sure, Windows libraries naming is not so strict IMHO...
There were no response for more than one month. This ticket could be closed as stalled within one week as described in http://fedoraproject.org/wiki/Policy_for_stalled_package_reviews .
*** This bug has been marked as a duplicate of bug 1110027 ***