Bug 503867 (perl-Time-Warp) - Review Request: perl-Time-Warp - Change the start and speed of Event time
Summary: Review Request: perl-Time-Warp - Change the start and speed of Event time
Keywords:
Status: CLOSED NEXTRELEASE
Alias: perl-Time-Warp
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL: http://search.cpan.org/dist/Time-Warp
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-06-03 05:00 UTC by Chris Weyl
Modified: 2009-06-04 04:19 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-06-04 04:19:41 UTC
Type: ---
Embargoed:
panemade: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Chris Weyl 2009-06-03 05:00:02 UTC
Spec URL: http://fedorapeople.org/~cweyl/review/perl-Time-Warp.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Time-Warp-0.5-1.fc10.src.rpm

Description:
Our external experience unfolds in 3 1/2 dimensions (time has a
dimensionality of 1/2). The Time::Warp module offers developers control
over the measurement of time.

This module is redundant if you're from Gallifrey, and not recommended
for use at high speeds near very massive objects.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1390415

Additional Comment:

This is a BR of a new testing BR of DBIx::Class::DateTime::Epoch.

*rt-0.10

Comment 1 Parag AN(पराग) 2009-06-03 05:14:45 UTC
Review:
+ package builds in mock (rawhide i586).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=1390415
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
0fd3e16c80a41228364d08978cc45b6391c7d02d  Time-Warp-0.5.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
All tests successful.
Files=1, Tests=8,  2 wallclock secs ( 0.02 usr  0.01 sys +  0.01 cusr  0.01 csys =  0.05 CPU)
+ Package perl-Time-Warp-0.5-1.fc12.i586 =>
Provides: perl(Time::Warp) = 0.5
Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) perl(DynaLoader) perl(Exporter) perl(strict) perl(vars) rpmlib(VersionedDependencies) <= 3.0.3-1 rtld(GNU_HASH)

+ Not a GUI application

APPROVED.

Comment 2 Chris Weyl 2009-06-03 05:19:20 UTC
New Package CVS Request
=======================
Package Name: perl-Time-Warp
Short Description: Change the start and speed of Event time
Owners: cweyl
Branches: F-9 F-10 F-11 devel
InitialCC: perl-sig

Comment 3 Jason Tibbitts 2009-06-03 17:05:55 UTC
CVS done.

Comment 4 Chris Weyl 2009-06-04 04:19:41 UTC
Thanks for the review! :-)


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