Spec URL: http://hg.et.redhat.com/misc/fedora-mingw--devel/?cmd=manifest;manifest=3e4efcc61ee75e92533c944912e3ea317da288fa;path=/SDL/ SRPM URL: http://www.annexia.org/tmp/mingw/fedora-9/src/SRPMS/mingw32-SDL-1.2.13-2.fc9.src.rpm Description: MinGW Windows port of SDL cross-platform multimedia library
Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/file/tip/SDL/mingw32-SDL.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-10/src/SRPMS/mingw32-SDL-1.2.13-2.fc10.src.rpm
REVIEW: [+] = OK [-] = NOT OK [1] = SEE COMMENTS [?] = WTF? ============================= [+] source files match upstream: sha1: http://www.libsdl.org/release/SDL-1.2.13.tar.gz [+] package meets naming and versioning guidelines. [+] specfile is properly named, is cleanly written and uses macros consistently. [+] dist tag is present. [+] build root is correct. [+] license field matches the actual license. [+] license is open source-compatible. LGPLv2+ [-] license text included in package. [+] latest version is being packaged. [+] BuildRequires are proper. [+] compiler flags are appropriate. [+] %clean is present. [+] package builds in koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1037809 [+] package installs properly. [1] rpmlint is silent. [+] no shared libraries are added to the regular linker search paths. [+] owns the directories it creates. [+] doesn't own any directories it shouldn't. [+] no duplicates in %files. [2] file permissions are appropriate. [+] no scriptlets present. [+] code, not content. [+] documentation is small, so no -docs subpackage is necessary. [+] %docs are not necessary for the proper functioning of the package. [3] no headers. [+] no pkgconfig files. ================================ COMMENTS: Please package the COPYING file. [1]: no there are warings like this: "mingw32-SDL.noarch: W: manpage-not-gzipped /usr/i686-pc-mingw32/sys-root/mingw/share/man/man3/SDL_SetColorKey.3" Those can be fixed with a line like this: find $RPM_BUILD_ROOT%{_mingw32_mandir}/man3/ -type f -not -name '*.gz' -print | xargs gzip -9f "mingw32-SDL.noarch: E: arch-independent-package-contains-binary-or-object /usr/i686-pc-mingw32/sys-root/mingw/lib/libSDLmain.a" Package used for cross-compiling so this is a non issue. "mingw32-SDL.noarch: W: non-standard-dir-in-usr i686-pc-mingw32" Used in all mingw32 packges. "mingw32-SDL.noarch: W: spurious-executable-perm /usr/i686-pc-mingw32/sys-root/mingw/lib/libSDL.dll.a" Why is this executeable ? "mingw32-SDL.noarch: W: devel-file-in-non-devel-package /usr/i686-pc-mingw32/sys-root/mingw/include/SDL/SDL_events.h" Fine, because package is only used for cross compiling. [2]: See comment in [1] regarding /usr/i686-pc-mingw32/sys-root/mingw/lib/libSDL.dll.a [3]: Is OK, see comment in [1] =================================== Package looks good, please fix the issues noted above and I will approve it.
> "mingw32-SDL.noarch: W: spurious-executable-perm > /usr/i686-pc-mingw32/sys-root/mingw/lib/libSDL.dll.a" > > Why is this executable ? See: https://bugzilla.redhat.com/show_bug.cgi?id=467397#c4 Here is an updated package which should fix everything else: Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/file/tip/SDL/mingw32-SDL.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-10/src/SRPMS/mingw32-SDL-1.2.13-4.fc10.src.rpm * Tue Jan 13 2009 Richard W.M. Jones <rjones> - 1.2.13-4 - Verify we are still up to date with Fedora release. - Include COPYING in documentation. - Build with dlfcn. - List all BRs. - No need to package the man pages, don't duplicate what's in the base Fedora package already. - Requires pkgconfig.
Looks good now. => APPROVED
"- Build with dlfcn." missed this one, removed the "+" for now until this package is in fedora.
dlfcn (bug 478640) has now been approved.
(In reply to comment #6) > dlfcn (bug 478640) has now been approved. I know, because I was the one who reviewed it ;)
New Package CVS Request ======================= Package Name: mingw32-SDL Short Description: MinGW Windows port of SDL cross-platform multimedia library Owners: rjones berrange lfarkas Branches: EL-5 F-10 InitialCC:
cvs done.
mingw32-SDL-1.2.13-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/mingw32-SDL-1.2.13-4.fc10
mingw32-SDL-1.2.13-4.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.