Bug 494161 (perl-Term-Size-Perl) - Review Request: perl-Term-Size-Perl - Perl extension for retrieving terminal size
Summary: Review Request: perl-Term-Size-Perl - Perl extension for retrieving terminal ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: perl-Term-Size-Perl
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chris Weyl
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-04-05 06:59 UTC by Iain Arnell
Modified: 2010-08-24 23:48 UTC (History)
2 users (show)

Fixed In Version: 0.029-2.fc9
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-07 04:14:31 UTC
Type: ---
Embargoed:
cweyl: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Iain Arnell 2009-04-05 06:59:22 UTC
Spec URL: http://iarnell.fedorapeople.org/rpms/perl-Term-Size-Perl.spec
SRPM URL: http://iarnell.fedorapeople.org/rpms/perl-Term-Size-Perl-0.029-1.fc11.src.rpm
Description: 
Yet another implementation of Term::Size. Now in pure Perl, with the
exception of a C probe run on build time.

scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1277692

note: although the resulting rpm appears to be noarch, the build is arch-dependent and produces different code in Params.pm for ppc and x86 (and consequently, rpmlint moans about no-binary).

Comment 1 Chris Weyl 2009-04-06 05:50:54 UTC
Everything looks good...  With the exception of a missing BR on Test::Pod;

Comment 2 Chris Weyl 2009-04-06 06:00:23 UTC
Everything looks good...  With the exception of a missing BR on Test::Pod;
please add before importing.

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1277692

Spec looks sane, clean and consistent; license is correct (GPL+ or Artistic);
make test passes cleanly.

Source tarballs match upstream (sha1sum):
00699199e149f28a2d57bb1ddbb1d0fde94c9f31 Term-Size-Perl-0.029.tar.gz
00699199e149f28a2d57bb1ddbb1d0fde94c9f31 Term-Size-Perl-0.029.tar.gz.srpm

Final provides / requires are sane:

0 packages and 1 specfiles checked; 0 errors, 0 warnings.
=====> perl-Term-Size-Perl-0.029-1.fc10.src.rpm <=====
====> rpmlint
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
====> provides for perl-Term-Size-Perl-0.029-1.fc10.src.rpm
====> requires for perl-Term-Size-Perl-0.029-1.fc10.src.rpm
perl(ExtUtils::CBuilder)  
perl(ExtUtils::MakeMaker)  
perl(Test::More)  

=====> perl-Term-Size-Perl-0.029-1.fc10.x86_64.rpm <=====
====> rpmlint
perl-Term-Size-Perl.x86_64: E: no-binary
1 packages and 0 specfiles checked; 1 errors, 0 warnings.
====> provides for perl-Term-Size-Perl-0.029-1.fc10.x86_64.rpm
perl(Term::Size::Perl) = 0.029
perl(Term::Size::Perl::Params) = 0.029
perl-Term-Size-Perl = 0.029-1.fc10
perl-Term-Size-Perl(x86-64) = 0.029-1.fc10
====> requires for perl-Term-Size-Perl-0.029-1.fc10.x86_64.rpm
perl(:MODULE_COMPAT_5.10.0)  
perl(Exporter)  
perl(Term::Size::Perl::Params)  
perl(strict)  
perl(vars)  

APPROVED

Comment 3 Iain Arnell 2009-04-06 08:44:01 UTC
New Package CVS Request
=======================
Package Name: perl-Term-Size-Perl
Short Description: Perl extension for retrieving terminal size
Owners: iarnell
Branches: F-9 F-10
InitialCC: perl-sig

Comment 4 Kevin Fenzi 2009-04-07 03:30:07 UTC
cvs done.

Comment 5 Fedora Update System 2009-04-07 04:04:04 UTC
perl-Term-Size-Perl-0.029-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-Term-Size-Perl-0.029-2.fc10

Comment 6 Fedora Update System 2009-04-07 04:04:08 UTC
perl-Term-Size-Perl-0.029-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/perl-Term-Size-Perl-0.029-2.fc9

Comment 7 Fedora Update System 2009-04-13 19:38:37 UTC
perl-Term-Size-Perl-0.029-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2009-04-13 19:41:18 UTC
perl-Term-Size-Perl-0.029-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Iain Arnell 2010-08-24 11:43:37 UTC
Package Change Request
======================
Package Name: perl-Term-Size-Perl
New Branches: el6
Owners: iarnell

Comment 10 Kevin Fenzi 2010-08-24 23:48:16 UTC
Git done (by process-git-requests).


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