Bug 491617 (mingw32-libxml++)

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.