Bug 673404 (perl-Parse-CPAN-Meta) - Review Request: perl-Parse-CPAN-Meta - Parse META.yml and META.json CPAN metadata files
Summary: Review Request: perl-Parse-CPAN-Meta - Parse META.yml and META.json CPAN meta...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: perl-Parse-CPAN-Meta
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Marcela Mašláňová
QA Contact: Fedora Extras Quality Assurance
URL: http://search.cpan.org/dist/Parse-CPA...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-01-28 04:55 UTC by Iain Arnell
Modified: 2011-02-04 23:42 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-04 23:42:03 UTC
Type: ---
Embargoed:
mmaslano: fedora-review+


Attachments (Terms of Use)

Description Iain Arnell 2011-01-28 04:55:13 UTC
Spec URL: http://fedorapeople.org/~iarnell/review/perl-Parse-CPAN-Meta.spec
SRPM URL: http://fedorapeople.org/~iarnell/review/perl-Parse-CPAN-Meta-1.4200-1.fc15.src.rpm

Description:
Parse::CPAN::Meta is a parser for META.json and META.yml files, using
JSON::PP and/or CPAN::Meta::YAML.

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

*rt-0.10_01

Comment 1 Marcela Mašláňová 2011-02-01 15:13:49 UTC
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

ACCEPT

Comment 2 Iain Arnell 2011-02-01 16:08:16 UTC
New Package SCM Request
=======================
Package Name: perl-Parse-CPAN-Meta
Short Description: Parse META.yml and META.json CPAN metadata files
Owners: iarnell
Branches: 
InitialCC: perl-sig

Comment 3 Paul Howarth 2011-02-02 15:06:37 UTC
perl-Parse-CPAN-Meta is already in Fedora.

This is a duplicate of Bug #447079.

Comment 4 Iain Arnell 2011-02-02 16:23:14 UTC
(In reply to comment #3)
> perl-Parse-CPAN-Meta is already in Fedora.

Parse::CPAN::Meta was merged into core with 5.10.1 and retired as a separate package in Fedora in December 2009. This is a revival as a dual-lifed module.

Comment 5 Paul Howarth 2011-02-02 16:32:10 UTC
Indeed, you're right.

However, there is still an existing perl-Parse-CPAN-Meta repo with a master branch so shouldn't the "New Package SCM Request" actually be a "Package Change Request" to change ownership of the devel branch and unblock it Rawhide?

Comment 6 Iain Arnell 2011-02-02 16:39:57 UTC
Package Change Request
======================
Package Name: perl-Parse-CPAN-Meta
Short Description: Parse META.yml and META.json CPAN metadata files
Owners: iarnell
Branches:

Comment 7 Kevin Fenzi 2011-02-03 21:20:46 UTC
GIT DONE. 

You might want to talk to the old owner and see if they would like to help maintain.

Comment 8 Iain Arnell 2011-02-04 04:06:48 UTC
Thanks, all. And Steve, if you're interested, of course you're more than welcome to maintain this package again.

Comment 9 Iain Arnell 2011-02-04 23:42:03 UTC
Imported and built for rawhide.


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