Bug 201478 - Review Request: perl-DateTime-Format-IBeat
Summary: Review Request: perl-DateTime-Format-IBeat
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Patrice Dumas
QA Contact: Fedora Package Reviews List
URL: http://search.cpan.org/dist/DateTime-...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 201480
TreeView+ depends on / blocked
 
Reported: 2006-08-06 00:08 UTC by Chris Weyl
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-08-06 16:57:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Chris Weyl 2006-08-06 00:08:33 UTC
SRPM URL: http://home.comcast.net/~ckweyl/perl-DateTime-Format-IBeat-0.161-1.fc5.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-DateTime-Format-IBeat.spec

Description:
No Time Zones, No Geographical Borders

How long is a Swatch .beat? In short, we have divided up the virtual and real
day into 1000 beats. One Swatch beat is the equivalent of 1 minute 26.4
seconds. That means that 12 noon in the old time system is the equivalent of
500 Swatch .beats.

Comment 1 Patrice Dumas 2006-08-06 11:54:05 UTC
2 remarks (not blockers):

- Since it is a noarch package, OPTIMIZE="%{optflags}" isn't needed
- no need to mv LICENSE, the spec file must be in american 
  english, not the files

* rpmlint is silent
* package named according to guidelines
* free software, licences included
* meets packaging guidelines
* spec legible
* source match upstream
53e1b71382b074c85e12c6b0bf855444  DateTime-Format-IBeat-0.161.tar.gz
* sane provides
Provides: perl(DateTime::Format::IBeat) = 0.161
* BuildRequires constructed according to Makefile.PL
* %files section right

APPROVED

Comment 2 Chris Weyl 2006-08-06 16:57:33 UTC
(In reply to comment #1)
> - Since it is a noarch package, OPTIMIZE="%{optflags}" isn't needed

Dropped those bits post-import.

+Import to CVS
+Add to owners.list
+Bump release, build for devel
+devel build succeeds
+Request branching (FC-4,FC-5)
+Close bug

Thanks for the review! :)


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