[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Daniel Berrange  changed:

   What|Removed |Added

 Depends on||754749

--- Comment #1 from Daniel Berrange  2011-11-17 11:21:19 
EST ---
NB requires bug 754749 to be reviewed first.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Bug 754754 depends on bug 754749, which changed state.

Bug 754749 Summary: Review Request: perl-Glib-Object-Introspection - 
Dynamically create Perl language bindings
https://bugzilla.redhat.com/show_bug.cgi?id=754749

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Mario Blättermann  changed:

   What|Removed |Added

 CC||mario.blaetterm...@gmail.co
   ||m

--- Comment #2 from Mario Blättermann  2011-11-27 
14:17:21 EST ---
Koji scratch build fails on Rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3544890

See build.log for more details:
http://koji.fedoraproject.org/koji/getfile?taskID=3544891&name=build.log

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #3 from Daniel Berrange  2011-11-28 06:57:19 
EST ---
There were some issues with the test suite. Updated srpm & spec build in brew
f17 targets succesfully:

  http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3.spec
 
http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3-0.001-2.fc17.src.rpm

Filed upstream RT ticket for actual bug

  https://rt.cpan.org/Ticket/Display.html?id=72773

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #4 from Daniel Berrange  2011-12-15 12:19:17 
EST ---
Updated to 0.002 which also fixes the RT ticket mentioned above

http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3.spec
http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3-0.002-1.fc17.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Petr Pisar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ppi...@redhat.com
 AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #5 from Petr Pisar  2012-01-04 12:38:32 EST ---
Source tar ball is original. Ok.
Summary verified from lib/Gtk3.pm.
TODO: Capitalize the `GTK+' to comply with official name (http://www.gtk.org/).

License verified from lib/Gtk3.pm and LICENSE. Ok.
URL and Source0 are usable. Ok.
Package does not contain any architecture depended code. noarch BuildArch is
Ok.
Description verified from lib/Gtk3.pm.
TODO: Capitalize the `GTK+' to comply with official name (http://www.gtk.org/).
TODO: Append slash to the URL as conforming HTTP URL must contain path.

FIX: Build-require perl(Carp) for tests (lib/Gtk3.pm:8)
TODO: Build-require perl(Exporter) for tests (lib/Gtk3.pm:10)
FIX: Build-require perl(Test::More) for tests (t/00-init.t:6)
FIX: Build-require perl(Scalar::Util) for tests (t/floating-refs.t:7)

TODO: Run tests against Xvfb to perform X11 tests
(; actually
there is easier way using xvfb-run tool).

TODO: Remove useless %defattr from %files section.

TODO: Do not package dist.ini and perl-Gtk3.doap. They do not provide any
useful data not mentioned anywhere else.

All tests pass. Ok.

$ rpmlint perl-Gtk3.spec ../SRPMS/perl-Gtk3-0.002-1.fc17.src.rpm
../RPMS/noarch/perl-Gtk3-0.002-1.fc17.noarch.rpm 
perl-Gtk3.src: W: spelling-error Summary(en_US) gtk -> Gk, gt, GTE
perl-Gtk3.src: W: spelling-error %description -l en_US gtk -> Gk, gt, GTE
perl-Gtk3.noarch: W: spelling-error Summary(en_US) gtk -> Gk, gt, GTE
perl-Gtk3.noarch: W: spelling-error %description -l en_US gtk -> Gk, gt, GTE
2 packages and 1 specfiles checked; 0 errors, 4 warnings.

rpmlint is Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-Gtk3-0.002-1.fc17.noarch.rpm 
drwxr-xr-x2 rootroot0 Jan  4 18:27
/usr/share/doc/perl-Gtk3-0.002
-rw-r--r--1 rootroot24657 Dec  9 11:37
/usr/share/doc/perl-Gtk3-0.002/LICENSE
-rw-r--r--1 rootroot  285 Dec  9 11:37
/usr/share/doc/perl-Gtk3-0.002/NEWS
-rw-r--r--1 rootroot  289 Dec  9 11:37
/usr/share/doc/perl-Gtk3-0.002/README
-rw-r--r--1 rootroot  712 Dec  9 11:37
/usr/share/doc/perl-Gtk3-0.002/dist.ini
-rw-r--r--1 rootroot  799 Dec  9 11:37
/usr/share/doc/perl-Gtk3-0.002/perl-Gtk3.doap
-rw-r--r--1 rootroot 2632 Jan  4 18:27
/usr/share/man/man3/Gtk3.3pm.gz
-rw-r--r--1 rootroot 9665 Dec  9 11:37
/usr/share/perl5/vendor_perl/Gtk3.pm
File permissions and layout Ok.

$ rpm -q --requires -p ../RPMS/noarch/perl-Gtk3-0.002-1.fc17.noarch.rpm |sort
|uniq -c
  1 gtk3  
  1 perl(Carp)  
  1 perl(Exporter)  
  1 perl(Glib) >= 1.240
  1 perl(Glib::Object::Introspection)  
  1 perl(Glib::Object::Introspection) >= 0.002
  1 perl(:MODULE_COMPAT_5.14.2)  
  1 perl(strict)  
  1 perl(warnings)  
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires Ok.
TODO: Do not export perl(Glib::Object::Introspection) without version.

$ rpm -q --provides -p ../RPMS/noarch/perl-Gtk3-0.002-1.fc17.noarch.rpm |sort
|uniq -c
  1 perl(Gtk3) = 0.002
  1 perl-Gtk3 = 0.002-1.fc17
Binary provides Ok.

$ resolvedeps rawhide ../RPMS/noarch/perl-Gtk3-0.002-1.fc17.noarch.rpm 
Binary dependencies resolvable. Ok.

Package builds in F17
(http://koji.fedoraproject.org/koji/taskinfo?taskID=3619080). Ok.

Otherwise package is in line with Fedora and Perl packaging guidelines.


Please correct all `FIX' prefixed issues, consider fixing `TODO' items, and
provide new spec file.

Resolution: Package NOT approved.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #6 from Daniel Berrange  2012-01-05 07:09:09 
EST ---
The extra 'perl(Glib::Object::Introspection)' without version, is an
automagically added dep. The versions deps are manually added, since I need to
force a minimal version to ensure the package works correctly. The unversioned
automatic dep is harmless so I don't think it is worth trying to filter it out.

Updated SRPM with all other FIX & TODO items addressed:

http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3-0.002-2.fc17.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Petr Pisar  changed:

   What|Removed |Added

Summary|Review Request: perl-Gtk3 - |Review Request: perl-Gtk3 -
   |Perl interface to the 3.x   |Perl interface to the 3.x
   |series of the gtk+ toolkit  |series of the GTK+ toolkit

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Petr Pisar  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Petr Pisar  2012-01-05 08:23:23 EST ---
Spec file changes:

--- perl-Gtk3.spec.old 2011-12-15 16:09:39.0 +0100
+++ perl-Gtk3.spec 2012-01-05 14:02:07.934998296 +0100
@@ -1,7 +1,9 @@
+%global use_x11_tests 1
+
 Name:   perl-Gtk3
 Version:0.002
-Release:1%{?dist}
-Summary:Perl interface to the 3.x series of the gtk+ toolkit
+Release:2%{?dist}
+Summary:Perl interface to the 3.x series of the GTK+ toolkit
 License:LGPLv2+
 Group:  Development/Libraries
 URL:http://search.cpan.org/dist/Gtk3/
@@ -12,14 +14,24 @@
 BuildRequires:  perl(Glib::Object::Introspection) >= 0.002
 BuildRequires:  perl(Test::More)
 BuildRequires:  gtk3
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(Exporter)
+BuildRequires:  perl(Test::More)
+BuildRequires:  perl(Scalar::Util)
 Requires:   perl(Glib) >= 1.240
 Requires:   perl(Glib::Object::Introspection) >= 0.002
 Requires:   perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
$version))
 Requires:   gtk3
+%if %{use_x11_tests}
+# X11 tests:
+BuildRequires:  xorg-x11-server-Xvfb
+BuildRequires:  xorg-x11-xinit
+BuildRequires:  font(:lang=en)
+%endif

 %description
-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/

 %prep
 %setup -q -n Gtk3-%{version}
@@ -37,17 +49,27 @@
 %{_fixperms} $RPM_BUILD_ROOT/*

 %check
-# NB, we expect most of these checks to be skipped
-# because of lack of a $DISPLAY inside koji
-make test
+%if %{use_x11_tests}
+xinit /bin/sh -c 'rm -f ok; make test && touch ok' -- /usr/bin/Xvfb :666
+test -e ok
+%else
+make test
+%endif

 %files
-%defattr(-,root,root,-)
-%doc dist.ini LICENSE NEWS perl-Gtk3.doap README
+%doc LICENSE NEWS README
 %{perl_vendorlib}/*
 %{_mandir}/man3/*

 %changelog
+* Thu Jan  5 2012 Daniel P. Berrange  - 0.002-2
+- Use xvfb to run test suite
+- Fix capitalization of GTK+
+- Remove dist.ini & perl-Gtk3.doap
+- Remove defattr from files section
+- Add missing BuildRequires for test suite
+- Add trailing / into URIs
+
 * Thu Dec 15 2011 Daniel P. Berrange  - 0.002-1
 - Update to 0.002 release


> TODO: Capitalize the `GTK+' to comply with official name 
> (http://www.gtk.org/).
-Summary:Perl interface to the 3.x series of the gtk+ toolkit
+Release:2%{?dist}
+Summary:Perl interface to the 3.x series of the GTK+ toolkit

-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/

Ok.

> TODO: Append slash to the URL as conforming HTTP URL must contain path.
-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/
TODO: You are missing full stop at the and of the description. If you hesitate
because of URL, close URL into angle brackets (.).

> FIX: Build-require perl(Carp) for tests (lib/Gtk3.pm:8)
> TODO: Build-require perl(Exporter) for tests (lib/Gtk3.pm:10)
> FIX: Build-require perl(Test::More) for tests (t/00-init.t:6)
> FIX: Build-require perl(Scalar::Util) for tests (t/floating-refs.t:7)
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(Exporter)
+BuildRequires:  perl(Test::More)
+BuildRequires:  perl(Scalar::Util)
Ok.

> TODO: Run tests against Xvfb to perform X11 tests
> (; actually
> there is easier way using xvfb-run tool).
Ok.

> TODO: Remove useless %defattr from %files section.
> TODO: Do not package dist.ini and perl-Gtk3.doap. They do not provide any
useful data not mentioned anywhere else.
 %files
-%defattr(-,root,root,-)
-%doc dist.ini LICENSE NEWS perl-Gtk3.doap README
+%doc LICENSE NEWS README
Ok.

All tests pass. Ok.

$ rpmlint perl-Gtk3.spec ../SRPMS/perl-Gtk3-0.002-2.fc17.src.rpm
../RPMS/noarch/perl-Gtk3-0.002-2.fc17.noarch.rpm 
perl-Gtk3.spec:29: W: comparison-operator-in-deptoken font(:lang=en)
perl-Gtk3.src:29: W: comparison-operator-in-deptoken font(:lang=en)
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
rpmlint Ok.

Pack

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Daniel Berrange  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Daniel Berrange  2012-01-05 10:39:15 
EST ---
New Package SCM Request
===
Package Name: perl-Gtk3
Short Description: Perl interface to the 3.x series of the GTK+ toolkit
Owners: berrange
Branches: f16
InitialCC: perl-sig

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #9 from Jon Ciesla  2012-01-05 11:54:35 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Daniel Berrange  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-06 08:05:08

--- Comment #10 from Daniel Berrange  2012-01-06 08:05:08 
EST ---
Built into rawhide

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

--- Comment #11 from Fedora Update System  
2012-01-06 08:05:47 EST ---
perl-Gtk3-0.002-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/perl-Gtk3-0.002-2.fc16

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754754

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||perl-Gtk3-0.002-2.fc16
 Resolution|RAWHIDE |ERRATA

--- Comment #12 from Fedora Update System  
2012-01-16 16:21:34 EST ---
perl-Gtk3-0.002-2.fc16 has been pushed to the Fedora 16 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review