Spec URL: http://sailer.fedorapeople.org/mingw32-libxml++.spec SRPM URL: http://sailer.fedorapeople.org/mingw32-libxml++-2.24.2-3.fc11.src.rpm Description: MinGW Windows C++ wrapper for libxml2. Approved MinGW packaging guidelines are here: http://fedoraproject.org/wiki/Packaging/MinGW
This package needs mingw32-glibmm24, which isn't included in Fedora and has to be submitted for review first. Besides that, you'll also need: BuildRequires: mingw32-gcc-c++ and pkgconfig because you have libxml++-2.6.pc in the package: Requires: pkgconfig Examples, manual, and reference shouldn't be included in the package because they duplicate docs in the native Fedora package.
You are right, mingw32-glibmm24 should go in first, I unfortunately noticed this only after submitting the review request. Thank you for your intial review comments, I updated the package according to your comments: http://sailer.fedorapeople.org/mingw32-libxml++.spec http://sailer.fedorapeople.org/mingw32-libxml++-2.24.2-4.fc11.src.rpm
Please note that we have packaged mingw32-glibmm24: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/file/tip/glibmm24 It needs to be turned into a Review Request.
mingw32-glibmm24 is now in Fedora and we can go on with this review. - In Rawhide the native Fedora libxml++ package has been updated to 2.26.0, and according to MinGW packaging guidelines the versions should match native package if possible. - Please clean up the %install section. It only makes the spec file unreadable if most of a section contains leftovers that aren't actually used. - Removing *.a in %install section only matches the (needed) import library, because static libraries aren't built (configure --disable-static). It seems unnecessary to first delete the import lib and then manually copy it over from .libs.
Thanks! I removed the docs cruft... Spec URL: http://sailer.fedorapeople.org/mingw32-libxml++.spec SRPM URL: http://sailer.fedorapeople.org/mingw32-libxml++-2.26.0-1.fc11.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1303634 Rpmlint is quiet: $ rpmlint mingw32-libxml++-2.26.0-1.fc11.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint mingw32-libxml++-2.26.0-1.fc11.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
Fedora review mingw32-libxml++-2.26.0-1.fc11.src.rpm 2008-04-17 + OK ! needs attention + rpmlint output + Package is named according to Fedora MinGW packaging guidelines + Specfile name matches the package base name + Package follows the Fedora MinGW packaging guidelines + License meets guidelines and is acceptable to Fedora LGPLv2+ + License matches the actual package license It is also the same as in the corresponding Fedora libxml++ package + The package contains the license file (COPYING) + Spec file is written in American English + Spec file is legible + Upstream sources match sources in the srpm a52fc7e6e44bb5cc187672930b843f72 libxml++-2.26.0.tar.bz2 a52fc7e6e44bb5cc187672930b843f72 SRPM/libxml++-2.26.0.tar.bz2 + Package builds in mock (Fedora Rawhide i586) n/a ExcludeArch bugs filed + BuildRequires list all build dependencies n/a %find_lang instead of %{_datadir}/locale/* n/a binary RPM with shared library files must call ldconfig in %post and %postun + Does not use Prefix: /usr + Package owns all directories it creates + No duplicate files in %files + %files has %defattr + %clean contains rm -rf $RPM_BUILD_ROOT + Consistent use of macros + Package must contain code or permissible content n/a Large documentation files should go in -doc subpackage + Files marked %doc should not affect package n/a Header files should be in -devel Fedora MinGW guidelines allow headers in main package n/a Static libraries should be in -static + Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig' n/a libfoo.so must go in -devel n/a -devel must require the fully versioned base n/a Packages should not contain libtool .la files Fedora MinGW guidelines allow .la files n/a Packages containing GUI apps must include %{name}.desktop file + Packages must not own files or directories owned by other packages + %install begins with rm -rf $RPM_BUILD_ROOT + Filenames must be valid UTF-8 After the %install cleanup dos2unix is no longer needed in BuildRequires, but that can be fixed without having to send an updated version for me to approve. Package APPROVED.
New Package CVS Request ======================= Package Name: mingw32-libxml++ Short Description: MinGW Windows C++ wrapper for libxml2 Owners: sailer rjones Branches: F-10 F-11 InitialCC:
cvs done.
mingw32-libxml++-2.26.0-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/mingw32-libxml++-2.26.0-1.fc11
mingw32-libxml++-2.24.2-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/mingw32-libxml++-2.24.2-1.fc10
mingw32-libxml++-2.24.2-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
mingw32-libxml++-2.26.0-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.