Bug 467419 (mingw32-pango)

Spec URL:
http://hg.et.redhat.com/misc/fedora-mingw--devel/?cmd=manifest;manifest=9731189432a7424689da169dd731fbff451bd746;path=/pango/
SRPM URL:
http://www.annexia.org/tmp/mingw/fedora-9/src/SRPMS/mingw32-pango-1.22.1-1.fc9.src.rpm

Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/file/tip/pango/mingw32-pango.spec
SRPM URL: http://www.annexia.org/tmp/mingw/fedora-10/src/SRPMS/mingw32-pango-1.22.1-1.fc10.src.rpm

Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/pango/mingw32-pango.spec
SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-pango-1.22.1-2.fc11.src.rpm

* Fri Jan 23 2009 Richard W.M. Jones <rjones> - 1.22.1-2
- Disable static libraries.
- Use _smp_mflags.

Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/pango/mingw32-pango.spec
SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-pango-1.22.1-4.fc11.src.rpm

* Fri Jan 30 2009 Richard W.M. Jones <rjones> - 1.22.1-4
- Requires pkgconfig.

* Tue Jan 27 2009 Levente Farkas <lfarkas> - 1.22.1-3
- Include license file in documentation section.

Reviewing the Hg version (1.22.1-6)

- homes.merjis.com is down at the moment, so I cannot pull expat and fontconfig.
- download link needs updating: it points to the 1.21 directory, not 1.22
- F-10 has newer version (1.22.3); upstream has 1.22.4. Should not be a big deal,
  we can just get 1.22.1 reviewed and update later

Any reason for the patch naming? (patch1000 !)

(In reply to comment #5)
> Reviewing the Hg version (1.22.1-6)
> 
> - homes.merjis.com is down at the moment, so I cannot pull expat and
> fontconfig.

Should be up again now.

> - download link needs updating: it points to the 1.21 directory, not 1.22
> - F-10 has newer version (1.22.3); upstream has 1.22.4. Should not be a big
> deal,
>   we can just get 1.22.1 reviewed and update later
> 
> Any reason for the patch naming? (patch1000 !)

The convention we've been using is that native patches
use the same patch numbers as in the native package, and
any mingw32-specific patches start counting at 1000.

I can't build, it appears to be a mingw32-cairo bug:

pangocairo-fcfont.c:24:22: error: cairo-ft.h: No such file or directory

$ rpm -ql mingw32-cairo | grep ft.h
$ rpm -ql cairo-devel | grep ft.h
/usr/include/cairo/cairo-ft.h

How did you build this package and what is the package version of mingw32-cairo you have installed? I just tried to compile it locally (rpmbuild) and the compilation of pango succeeded without any errors.

Ah, my mistake. I might have been accidentally using the F-10 version of mingw32-fontconfig (from the MinGW test repo) together with the rest of the already-merged Rawhide stack. I've been unwell for the past few days too, so sorry for the delay.

MUST

- rpmlint
  manpage-not-gzipped: pango-querymodules.1

  this should probably not be packaged, as MinGW developers should have the
  docs from the native pango anyway

+ package name
+ spec file name
+ package guideline-compliant
+ license complies with guidelines
+ license field accurate
+ license file not deleted
+ spec in US English
+ spec legible
+ source matches upstream
+ builds under >= 1 archs, others excluded
+ build dependencies complete
N/A locales handled using %find_lang, no %{_datadir}/locale
N/A library -> ldconfig
N/A relocatable: give reason
+ own all directories
+ no dupes in %files
+ permission
+ %clean RPM_BUILD_ROOT
+ macros used consistently
+ Package contains code
+ static in -static
+ if contains *.pc, req pkgconfig
+ clean buildroot before install
+ filenames UTF-8

SHOULD
- if license text missing, ask upstream to include it
- desc and summary contain translations if available
? package build in mock on all architectures
+ package functioned as described
+ scriplets are sane
+ other subpackages should require versioned base
+ if main pkg is development-wise, pkgconfig can go in main package
+ require package not files

Spec URL: http://www.annexia.org/tmp/mingw32-pango.spec
SRPM URL: http://www.annexia.org/tmp/mingw32-pango-1.23.0-1.fc11.src.rpm

* Mon Mar  9 2009 Richard W.M. Jones <rjones> - 1.23.0-1
- Remove man page which duplicates what is in base Fedora.
- Rebase to 1.23.0 to match Fedora.
- +BR mingw32-dlfcn.

Changes look good.

APPROVED

New Package CVS Request
=======================
Package Name: mingw32-pango
Short Description: MinGW Pango library
Owners: rjones berrange
Branches: F-10 EL-5
InitialCC:

CVS Done

Built for Rawhide, EPEL-5, F-10.

mingw32-pango-1.23.0-1.fc10.2 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-pango-1.23.0-1.fc10.2

mingw32-pango-1.23.0-1.fc10.2 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.