Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-Term-Size - Simple way to get terminal size
https://bugzilla.redhat.com/show_bug.cgi?id=469750
Summary: Review Request: perl-Term-Size - Simple way to get terminal size Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/Term-Size OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: cweyl@alumni.drew.edu QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://fedorapeople.org/~cweyl/review/perl-Term-Size.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Term-Size-0.2-1.fc9.src.rpm
Description: *Term::Size* is a Perl module which provides a straightforward way to retrieve the terminal size.
Both functions take an optional filehandle argument, which defaults to '*STDIN{IO}'. They both return a list of two values, which are the current width and height, respectively, of the terminal associated with the specified filehandle.
'Term::Size::chars' returns the size in units of characters, whereas 'Term::Size::pixels' uses units of pixels.
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=916588
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=469750
Chris Weyl cweyl@alumni.drew.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Alias| |perl-Term-Size
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=469750
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |panemade@gmail.com AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Flag| |fedora-review+
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2008-11-04 05:21:43 EDT --- Review: + package builds in mock (rawhide i386). koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=916867 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 982b5df8351e7654a42b7bffc0bf1d57 Term-Size-0.2.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave None as they are not used. + Package perl-Term-Size-0.2-1.fc10 => Provides: Size.so perl(Term::Size) = 0.2 Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) perl(AutoLoader) perl(Carp) perl(DynaLoader) perl(Exporter) perl(strict) perl(vars) rtld(GNU_HASH) APPROVED.
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=469750
--- Comment #2 from Chris Weyl cweyl@alumni.drew.edu 2008-11-04 11:31:23 EDT --- New Package CVS Request ======================= Package Name: perl-Term-Size Short Description: Simple way to get terminal size Owners: cweyl Branches: F-8 F-9 devel InitialCC: perl-sig
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=469750
Chris Weyl cweyl@alumni.drew.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
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=469750
--- Comment #3 from Chris Weyl cweyl@alumni.drew.edu 2008-11-04 17:03:17 EDT --- New Package CVS Request ======================= Package Name: perl-Term-Size Short Description: Simple way to get terminal size Owners: cweyl Branches: F-8 F-9 F-10 devel InitialCC: perl-sig
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=469750
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #4 from Kevin Fenzi kevin@tummy.com 2008-11-05 17:48:56 EDT --- cvs done.
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=469750
Chris Weyl cweyl@alumni.drew.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
package-review@lists.fedoraproject.org