Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 201480 - Review Request: perl-DateTime-Format-Builder
Summary: Review Request: perl-DateTime-Format-Builder
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: 201476 201477 201478 201479
Blocks: FE-ACCEPT 201481
TreeView+ depends on / blocked
 
Reported: 2006-08-06 00:15 UTC by Chris Weyl
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-08-08 22:07:22 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

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

Description:
DateTime::Format::Builder creates DateTime parsers. Many string formats of
dates and times are simple and just require a basic regular expression to
extract the relevant information. Builder provides a simple way to do this
without writing reams of structural code.

Builder provides a number of methods, most of which you'll never need, or at
least rarely need. They're provided more for exposing of the module's innards
to any subclasses, or for when you need to do something slightly beyond what
is expected.

Comment 1 Patrice Dumas 2006-08-08 12:10:34 UTC
For tests, missing BR:
perl(Test::Pod) perl(Test::More) 
maybe
perl(DTFB::OnFailTest) perl(DTFB::Quick)

maybe Params::Validate => 0.72
since it is in Build.PL, but is it required for build? I guess so
for tests?

README and LICENCE missing from %doc

Comment 2 Chris Weyl 2006-08-08 15:19:26 UTC
BR's and %doc updated; perl(DTFB::*) are actually packages defined in and as
part of the test suites, so they were not included.

SRPM URL:
http://home.comcast.net/~ckweyl/perl-DateTime-Format-Builder-0.7807-2.fc5.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-DateTime-Format-Builder.spec

Comment 3 Patrice Dumas 2006-08-08 19:10:26 UTC
* rpmlint is silent
* package named according to guidelines
* free software, licences included
* meets packaging guidelines
* spec legible
* source match upstream
4f6ee670cab944db0492e70ca8df3be3  ./DateTime-Format-Builder-0.7807.tar.gz
* sane provides
Provides: perl(DateTime::Format::Builder) = 0.7807
perl(DateTime::Format::Builder::Parser) = 0.77
perl(DateTime::Format::Builder::Parser::Dispatch) = 0.78
perl(DateTime::Format::Builder::Parser::Quick) = 0.77
perl(DateTime::Format::Builder::Parser::Regex) = 0.77
perl(DateTime::Format::Builder::Parser::Strptime) = 0.77
perl(DateTime::Format::Builder::Parser::generic) = 0.77
* %files section right

APPROVED

Comment 4 Patrice Dumas 2006-08-08 19:16:12 UTC
You could add a comment to 

mv LICENCE LICENSE

And 
perl -pi -e 's/E<copy>/(C)/' `find lib/ -type f`
could be more classically something along
find lib/ -type f -exec sed -i -e 's/E<copy>/(C)/' {} \;
but it is a perl module, so there is more than one way
to do it ;-)

Comment 5 Chris Weyl 2006-08-08 22:07:22 UTC
+Import to CVS
+Add to owners.list
+Bump release, build for devel
+devel build succeeds
+Request branching 
+Close bug

Thanks for the review!

(In reply to comment #4)
> perl -pi -e 's/E<copy>/(C)/' `find lib/ -type f`
> could be more classically something along
> find lib/ -type f -exec sed -i -e 's/E<copy>/(C)/' {} \;
> but it is a perl module, so there is more than one way
> to do it ;-)

TMTOWTDI, even outside perl :)



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