Bug 467420 (mingw32-gtk2) - Review Request: mingw32-gtk2 - MinGW Windows Gtk2 library
Summary: Review Request: mingw32-gtk2 - MinGW Windows Gtk2 library
Alias: mingw32-gtk2
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Michel Lind
QA Contact: Fedora Extras Quality Assurance
Depends On: 454410 mingw32-iconv mingw32-libpng mingw32-gettext mingw32-libjpeg mingw32-glib2 mingw32-jasper mingw32-atk mingw32-cairo mingw32-pango
Blocks: mingw32-gtk-vnc
TreeView+ depends on / blocked
Reported: 2008-10-17 11:39 UTC by Richard W.M. Jones
Modified: 2009-03-23 15:57 UTC (History)
7 users (show)

Fixed In Version: 2.15.0-3.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-03-18 16:27:05 UTC
Type: ---
michel: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Comment 2 Levente Farkas 2008-11-01 13:14:50 UTC
please apply:

diff -up ./mingw32-gtk2.spec.lfarkas ./mingw32-gtk2.spec
--- ./mingw32-gtk2.spec.lfarkas	2008-11-01 14:13:22.000000000 +0100
+++ ./mingw32-gtk2.spec	2008-11-01 14:14:14.000000000 +0100
@@ -4,6 +4,8 @@
 %define __find_requires %{_mingw32_findrequires}
 %define __find_provides %{_mingw32_findprovides}
+%define glib2_version 2.6.0
 Name:           mingw32-gtk2
 Version:        2.14.4
 Release:        3%{?dist}
@@ -38,11 +40,11 @@ BuildRequires:  pkgconfig
 # Native one for msgfmt
 BuildRequires:  gettext
 # Native one for glib-genmarsjal
-BuildRequires:  glib2-devel
+BuildRequires:  glib2-devel >= %{glib2_version}
 # Native one for gtk-update-icon-cache
-BuildRequires:  gtk2
+BuildRequires:  gtk2 >= %{version}
 # Native one for gdk-pixbuf-csource
-BuildRequires:  gtk2-devel
+BuildRequires:  gtk2-devel >= %{version}

Comment 3 Richard W.M. Jones 2008-11-03 10:47:47 UTC
(In reply to comment #2)

Please see my comments:

Comment 5 Richard W.M. Jones 2009-01-25 13:18:14 UTC
Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/gtk2/mingw32-gtk2.spec
SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-gtk2-2.15.0-1.fc11.src.rpm

* Fri Jan 23 2009 Richard W.M. Jones <rjones> - 2.15.0-1
- Rebase to Fedora native version 2.15.0.
- Disable static libraries.
- Use _smp_mflags.
- Use find_lang macro.

Comment 6 Richard W.M. Jones 2009-02-06 15:21:14 UTC
Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/gtk2/mingw32-gtk2.spec
SRPM URL: http://www.annexia.org/tmp/mingw32-gtk2-2.15.0-3.fc11.src.rpm

* Fri Feb  6 2009 Richard W.M. Jones <rjones> - 2.15.0-3
- Remove documentation.
- Add license file.
- Added extra BRs suggested by auto-buildrequires.

Comment 7 Michel Lind 2009-02-09 15:49:38 UTC

? rpmlint: .defs marked executable
I went back to my mingw23-libgcrypt review:

and it looks like you ended up removing the *.def files altogether. That definitely fixes rpmlint, and I just tried rebuilding gtk-vnc after removing mingw32-gtk2's *.defs, so MinGW does not need those *.def files, but wouldn't someone trying to compile against libgcrypt on Windows need them?

What I don't know is whether they absolutely must have +x permissions. I'm guessing that chmod -x'ing them should be safe, after all, they are just header files. Should they be readded to libgcrypt?

+ 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
+ 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
+ macros used consistently
+ Package contains code

N/A large docs => -doc
N/A doc not runtime dependent

N/A headers in -devel
N/A static in -static

+   if contains *.pc, req pkgconfig
N/A if libfiles are suffixed, the non-suffixed goes to devel
N/A devel requires versioned base package

N/A desktop file uses desktop-file-install
+   clean buildroot before install
+   filenames UTF-8

-   desc and summary contain translations if available
?   package build in mock on all architectures
    Not tested -- not all dependencies in Fedora yet, thus no Koji
+   package functioned as described
+   scriplets are sane
N/A other subpackages should require versioned base
+   if main pkg is development-wise, pkgconfig can go in main package
+   require package not files

Comment 8 Richard W.M. Jones 2009-02-09 16:05:45 UTC
I don't even know enough to speculate about *.def files,
so I raised a question on the mailing list:


Thanks for the review.  I will raise a CVS request next.

Comment 9 Richard W.M. Jones 2009-02-09 16:07:42 UTC
New Package CVS Request
Package Name: mingw32-gtk2
Short Description: MinGW Windows Gtk2 library
Owners: rjones berrange
Branches: EL-5 F-10

Comment 10 Kevin Fenzi 2009-02-10 22:21:17 UTC
cvs done.

Comment 11 Richard W.M. Jones 2009-02-11 09:46:38 UTC
Imported into CVS but not built yet because of
dependencies on mingw32-pango, mingw32-jasper.

Comment 12 Michel Lind 2009-03-13 19:05:13 UTC
jasper and pango are now done, so this should probably get built soon.

Comment 13 Richard W.M. Jones 2009-03-13 19:16:05 UTC
I'm building them ... It's complicated by the fact that some
of the dependencies don't build correctly.

Comment 14 Michal Nowak 2009-03-18 13:32:24 UTC
This looks here http://koji.fedoraproject.org/koji/buildinfo?buildID=94068 to be built for Rawhide but I am unable to reach it from my box.

Comment 15 Richard W.M. Jones 2009-03-18 13:44:00 UTC
This is NOT built for F10 yet, and nor is gtk-vnc.  The bug
will be closed automatically by Bodhi when the time comes.

Comment 16 Fedora Update System 2009-03-18 16:26:31 UTC
mingw32-gtk2-2.15.0-3.fc10 has been submitted as an update for Fedora 10.

Comment 17 Fedora Update System 2009-03-23 15:56:55 UTC
mingw32-gtk2-2.15.0-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.