Spec URL: http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API-0.42-1.fc18.src.rpm Description: This is a hopefully-complete API-compliant interface to MetaCPAN (https://metacpan.org) with DWIM capabilities, to make your life easier. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3954430 *rt-0.10_02
Quick first pass comments: * Latest upstream is 0.43 * No point having BR: Test::Perl::Critic unless you also set AUTHOR_TESTING * No point having BR: Pod::Coverage::TrustPod/Test::Pod/Test::Pod::Coverage unless you also set RELEASE_TESTING * BR: Test::Script isn't really needed because there are no scripts to test * You don't need to remove empty directories from the buildroot * Does use of the %{__perl} macro buy us anything? * Similarly, %{?perl_default_filter} does nothing, but causes no harm either None of these are blockers so if you like I'll do a formal review of the package as it stands, or if you want to update to the latest upstream I'll wait until then. I just updated my local upstream release monitoring script to use this module.
Thanks, Paul. Updated to 0.43, enabled author/release tests, dropped Test::Script BR, and stopped removing empty directories. Spec URL: http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API-0.43-1.fc18.src.rpm Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=4005561
Review for perl-MetaCPAN-API ============================ - rpmlint clean - package and spec file naming OK - package meets guidelines - license is OK for Fedora, matches upstream, and LICENSE file is packaged - spec file written in English and is legible - source tarball matches upstream - package builds OK in mock for F-16..F-18 (x86_64 [noarch]) - package tested and works OK in F-16 x86_64 - --with network-tests build option works as advertised - buildreqs sufficient and not redundant - no locales, libraries, devel files to worry about - not intended to be relocatable - no duplicate files or directory ownership issues - permissions sane - macro usage is consistent - code, not content - no large docs to consider - docs don't affect runtime - not a GUI app, no .desktop file needed - all filenames are plain ASCII - no scriptlets present or needed - no file dependencies Requires -------- perl(:MODULE_COMPAT_5.14.2) perl(Any::Moose) perl(Carp) perl(HTTP::Tiny) perl(JSON) perl(Try::Tiny) perl(URI::Escape) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(PayloadIsXz) <= 5.2-1 Provides -------- perl(MetaCPAN::API) = 0.43 perl(MetaCPAN::API::Author) = 0.43 perl(MetaCPAN::API::Module) = 0.43 perl(MetaCPAN::API::POD) = 0.43 perl(MetaCPAN::API::Release) = 0.43 perl(MetaCPAN::API::Source) = 0.43 perl-MetaCPAN-API = 0.43-1.fc18 Nothing to quibble about here. APPROVED.
New Package SCM Request ======================= Package Name: perl-MetaCPAN-API Short Description: Comprehensive, DWIM-featured API to MetaCPAN Owners: iarnell Branches: f16 f17 InitialCC: perl-sig
Git done (by process-git-requests).
perl-MetaCPAN-API-0.43-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/perl-MetaCPAN-API-0.43-1.fc17
perl-MetaCPAN-API-0.43-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-MetaCPAN-API-0.43-1.fc16
perl-MetaCPAN-API-0.43-1.fc17 has been pushed to the Fedora 17 testing repository.
perl-MetaCPAN-API-0.43-1.fc16 has been pushed to the Fedora 16 stable repository.
perl-MetaCPAN-API-0.43-1.fc17 has been pushed to the Fedora 17 stable repository.