Bug 979665

Summary: Review Request: perl-Data-MessagePack - MessagePack serialising/deserialising
Product: [Fedora] Fedora Reporter: Christopher Meng <i>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: notting, package-review, panemade
Target Milestone: ---Flags: panemade: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: perl-Data-MessagePack-0.47-1.fc18 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-15 00:59:01 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 979666    

Description Christopher Meng 2013-06-29 08:37:28 UTC
Spec URL: http://cicku.me/perl-Data-MessagePack.spec
SRPM URL: http://cicku.me/perl-Data-MessagePack-0.47-1.fc20.src.rpm
Description: This module converts Perl data structures to MessagePack and vice versa.
Fedora Account System Username: cicku

Comment 1 Parag AN(पराग) 2013-07-03 11:00:42 UTC
Review:

+ mock build is successful for f20

+ rpmlint output on rpms gave
perl-Data-MessagePack.src: W: spelling-error Summary(en_US) serialising -> serializing, serialization
perl-Data-MessagePack.src: W: spelling-error Summary(en_US) deserialising -> serializing
perl-Data-MessagePack.src: W: spelling-error %description -l en_US versa -> avers, verse, verso
perl-Data-MessagePack.src: W: invalid-url URL: http://search.cpan.org/dist/Data-MessagePack/ <urlopen error [Errno -2] Name or service not known>
perl-Data-MessagePack.src: W: invalid-url Source0: http://www.cpan.org/authors/id/G/GF/GFUJI/Data-MessagePack-0.47.tar.gz <urlopen error [Errno -2] Name or service not known>
perl-Data-MessagePack.x86_64: W: spelling-error Summary(en_US) serialising -> serializing, serialization
perl-Data-MessagePack.x86_64: W: spelling-error Summary(en_US) deserialising -> serializing
perl-Data-MessagePack.x86_64: W: spelling-error %description -l en_US versa -> avers, verse, verso
perl-Data-MessagePack.x86_64: W: invalid-url URL: http://search.cpan.org/dist/Data-MessagePack/ <urlopen error [Errno -2] Name or service not known>
perl-Data-MessagePack-debuginfo.x86_64: W: invalid-url URL: http://search.cpan.org/dist/Data-MessagePack/ <urlopen error [Errno -2] Name or service not known>
3 packages and 0 specfiles checked; 0 errors, 10 warnings.
==> looks clean

+ Source verified with upstream as (sha256sum)
srpm tarball     : 47225ee51263bd78324a0f2a3de800bf53344dc597cb64d58661e711bbab9ca8
upstream tarball : 47225ee51263bd78324a0f2a3de800bf53344dc597cb64d58661e711bbab9ca8

+ License "GPL+ or Artistic" is valid and included in source files.

+ rest follows packaging guidelines

+ make test is successful with output
All tests successful.

+ Package perl-Data-MessagePack-0.47-1.fc20
Provides: perl(Data::MessagePack) = 0.47 perl(Data::MessagePack::Boolean) perl(Data::MessagePack::PP) perl-Data-MessagePack = 0.47-1.fc20 perl-Data-MessagePack(x86-64) = 0.47-1.fc20

Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.14)(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.4)(64bit) perl >= 0:5.008001 perl(B) perl(Carp) perl(Config) perl(strict) perl(warnings) rtld(GNU_HASH)

suggestions:
1) I don't think you need BuildRequires: perl, it will get pulled into build environment.

2) you don't need versions in BR: or Requires:

APPROVED.

Comment 2 Christopher Meng 2013-07-03 12:12:17 UTC
New Package SCM Request
=======================
Package Name: perl-Data-MessagePack
Short Description: MessagePack serialising/deserialising
Owners: cicku
Branches: f18 f19
InitialCC:

Comment 3 Gwyn Ciesla 2013-07-03 13:15:53 UTC
Git done (by process-git-requests).

Comment 4 Fedora Update System 2013-07-05 12:17:28 UTC
perl-Data-MessagePack-0.47-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/perl-Data-MessagePack-0.47-1.fc19

Comment 5 Fedora Update System 2013-07-05 12:18:22 UTC
perl-Data-MessagePack-0.47-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/perl-Data-MessagePack-0.47-1.fc18

Comment 6 Fedora Update System 2013-07-06 00:52:42 UTC
perl-Data-MessagePack-0.47-1.fc18 has been pushed to the Fedora 18 testing repository.

Comment 7 Fedora Update System 2013-07-15 00:59:01 UTC
perl-Data-MessagePack-0.47-1.fc19 has been pushed to the Fedora 19 stable repository.

Comment 8 Fedora Update System 2013-07-15 01:08:51 UTC
perl-Data-MessagePack-0.47-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 9 Christopher Meng 2014-01-24 01:34:30 UTC
Package Change Request
======================
Package Name: nbd
New Branches: epel7
Owners: cicku

Comment 10 Gwyn Ciesla 2014-01-24 12:43:57 UTC
Git done (by process-git-requests).

Comment 11 Christopher Meng 2014-02-20 03:11:25 UTC
Package Change Request
======================
Package Name: perl-Data-MessagePack
New Branches: epel7
Owners: cicku

Just found comment 9 was a mistake...

Comment 12 Gwyn Ciesla 2014-02-20 13:46:28 UTC
Git done (by process-git-requests).