Bug 752858 (perl-Qt) - Review Request: perl-Qt - Perl bindings for Qt
Summary: Review Request: perl-Qt - Perl bindings for Qt
Keywords:
Status: CLOSED ERRATA
Alias: perl-Qt
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL: http://search.cpan.org/dist/Qt/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-11-10 15:55 UTC by Iain Arnell
Modified: 2011-11-26 23:09 UTC (History)
3 users (show)

Fixed In Version: perl-Qt-0.96.0-1.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-11-26 23:09:20 UTC
Type: ---
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Iain Arnell 2011-11-10 15:55:29 UTC
Spec URL: http://fedorapeople.org/~iarnell/review/perl-Qt.spec
SRPM URL: http://fedorapeople.org/~iarnell/review/perl-Qt-0.96.0-1.fc17.src.rpm

Description:
This module provides Perl bindings for the Qt 4 libraries.

*rt-0.10_02

Comment 1 Iain Arnell 2011-11-10 15:57:17 UTC
Koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=3504521

Comment 2 Parag AN(पराग) 2011-11-16 09:32:04 UTC
Review:-

+ koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3504521

+ rpmlint on a package gave
perl-Qt-devel.x86_64: W: file-not-utf8 /usr/share/doc/perl-Qt-devel-0.96.0/examples/tools/codecs/encodedfiles/iso-8859-1.txt
perl-Qt-devel.x86_64: E: wrong-script-interpreter /usr/share/doc/perl-Qt-devel-0.96.0/examples/linguist/hellotr/hellotr.pro [0]
perl-Qt-devel.x86_64: W: file-not-utf8 /usr/share/doc/perl-Qt-devel-0.96.0/examples/tools/codecs/encodedfiles/iso-8859-15.txt
perl-Qt-devel.x86_64: W: no-manual-page-for-binary puic4
perl-Qt-devel.x86_64: W: no-manual-page-for-binary prcc4_bin
perl-Qt-devel.x86_64: W: no-manual-page-for-binary qdbusxml2perl
3 packages and 0 specfiles checked; 1 errors, 5 warnings.

+ Source verified with upstream as (sha1sum)
147428a10da852e82c0da06307c3c45ac8dada88  Qt-0.96.0.tar.gz
147428a10da852e82c0da06307c3c45ac8dada88  ../SOURCES/Qt-0.96.0.tar.gz

+ make test gave
100% tests passed, 0 tests failed out of 26
Total Test time (real) =   6.11 sec

+ Follows packaging guidelines

suggestions:
1) Fix the rpmlint warnings for file-not-utf8 for iso-8859-1.txt and iso-8859-15.txt. See http://fedoraproject.org/wiki/Packaging_tricks#Convert_encoding_to_UTF-8

2) Fix the rpmlint error as given above.

APPROVED.

Comment 3 Iain Arnell 2011-11-16 11:34:54 UTC
(In reply to comment #2)
>
> suggestions:
> 1) Fix the rpmlint warnings for file-not-utf8 for iso-8859-1.txt and
> iso-8859-15.txt. 

That would ruin the point of the files - the example code demonstrates how to read and write files using various encodings.

> 
> 2) Fix the rpmlint error as given above.

That's not a script - and it's not executable. It just a text file that happens to start with a shebang.

> APPROVED.

Thanks!

Comment 4 Iain Arnell 2011-11-16 11:36:03 UTC
New Package SCM Request
=======================
Package Name: perl-Qt
Short Description: Perl bindings for Qt
Owners: iarnell
Branches: f15 f16
InitialCC: perl-sig

Comment 5 Gwyn Ciesla 2011-11-16 12:10:23 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2011-11-17 03:58:37 UTC
perl-Qt-0.96.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/perl-Qt-0.96.0-1.fc16

Comment 7 Fedora Update System 2011-11-19 06:08:49 UTC
perl-Qt-0.96.0-1.fc16 has been pushed to the Fedora 16 testing repository.

Comment 8 Fedora Update System 2011-11-26 23:09:20 UTC
perl-Qt-0.96.0-1.fc16 has been pushed to the Fedora 16 stable repository.


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