Bug 484202 - Review Request: perl-Test-JSON - Test JSON data
Summary: Review Request: perl-Test-JSON - Test JSON data
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Miroslav Suchý
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-02-05 12:33 UTC by Allisson Azevedo
Modified: 2009-02-27 00:57 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-02-27 00:57:48 UTC
Type: ---
Embargoed:
msuchy: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Allisson Azevedo 2009-02-05 12:33:06 UTC
Spec URL: http://allisson.fedorapeople.org/packages/perl-Test-JSON/perl-Test-JSON.spec

SRPM URL: http://allisson.fedorapeople.org/packages/perl-Test-JSON/perl-Test-JSON-0.06-1.fc10.src.rpm

Description: JavaScript Object Notation (JSON) is a lightweight data interchange format.
Test::JSON makes it easy to verify that you have built valid JSON and that
it matches your expected output.

Comment 1 Miroslav Suchý 2009-02-24 11:22:04 UTC
I will look on this today.

Comment 2 Miroslav Suchý 2009-02-24 12:32:46 UTC
+ package builds in mock (rawhide i386).
koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1156435
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
9d8f4c2e724f67e66317f06efaa53e03 for both Test-JSON-0.06.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
not tested - package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
N/A no headers or static libraries.
N/A no .pc file present.
N/A no -devel subpackage
N/A no .la files.
N/A no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
t/00-load.........# Testing Test::JSON 0.06, Perl 5.010000, /usr/bin/perl
ok
t/10testjson......ok
t/pod-coverage....ok
t/pod.............ok
All tests successful.
Files=4, Tests=39,  0 wallclock secs ( 0.02 usr  0.01 sys +  0.21 cusr  0.05 csys =  0.29 CPU)
Result: PASS
+ Package perl-Test-JSON:
Provides: perl(Test::JSON) = 0.06
Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(VersionedDependencies) <= 3.0.3-1
Requires: perl(Carp) perl(JSON::Any) perl(Test::Builder) perl(Test::Differences) perl(strict)


APPROVED

Comment 3 Allisson Azevedo 2009-02-24 16:23:19 UTC
New Package CVS Request
=======================
Package Name: perl-Test-JSON
Short Description: Test JSON data
Owners: allisson
Branches: F-10 F-9
InitialCC: perl-sig

Comment 4 Kevin Fenzi 2009-02-24 20:51:00 UTC
cvs done.


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