Bug 1482867 - Review Request: perl-JSON-Parse - Read JSON into a Perl variable
Summary: Review Request: perl-JSON-Parse - Read JSON into a Perl variable
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jitka Plesnikova
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1472755
TreeView+ depends on / blocked
 
Reported: 2017-08-18 09:11 UTC by Emmanuel Seyman
Modified: 2017-09-06 10:44 UTC (History)
2 users (show)

Fixed In Version: perl-JSON-Parse-0.49-2.fc28
Clone Of:
Environment:
Last Closed: 2017-09-06 10:44:55 UTC
Type: ---
Embargoed:
jplesnik: fedora-review+


Attachments (Terms of Use)

Description Emmanuel Seyman 2017-08-18 09:11:50 UTC
Spec URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse.spec
SRPM URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse-0.49-1.fc26.src.rpm
Description:
A Perl module for parsing JSON. (JSON means "JavaScript Object Notation" and it
is specified in "RFC 7159".)

Fedora Account System Username: eseyman

Comment 1 Jitka Plesnikova 2017-08-18 12:03:59 UTC
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

$ rpm -qp --requires perl-JSON-Parse-0.49-1.fc28.x86_64.rpm | sort | uniq -c
      1 libc.so.6()(64bit)
      1 libc.so.6(GLIBC_2.14)(64bit)
      1 libc.so.6(GLIBC_2.2.5)(64bit)
      1 libc.so.6(GLIBC_2.3.4)(64bit)
      1 libc.so.6(GLIBC_2.4)(64bit)
      1 libperl.so.5.26()(64bit)
      1 perl(Carp)
      1 perl(Exporter)
      1 perl(Getopt::Long)
      1 perl(JSON::Parse)
      1 perl(lib)
      1 perl(:MODULE_COMPAT_5.26.0)
      1 perl(strict)
      1 perl(warnings)
      1 perl(XSLoader)
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
      1 rtld(GNU_HASH)
      1 /usr/bin/perl
Binary requires are Ok.

$ rpm -qp --provides perl-JSON-Parse-0.49-1.fc28.x86_64.rpm | sort | uniq -c
      1 perl(JSON::Parse) = 0.49
      1 perl-JSON-Parse = 0.49-1.fc28
      1 perl-JSON-Parse(x86-64) = 0.49-1.fc28
      1 perl(JSON::Tokenize) = 0.49
Binary provides are Ok.

$ rpmlint ./perl-JSON-Parse.spec ./perl-JSON-Parse-0.49-1.fc28.*
perl-JSON-Parse.x86_64: W: hidden-file-or-dir /usr/lib/.build-id
perl-JSON-Parse.x86_64: W: hidden-file-or-dir /usr/lib/.build-id
perl-JSON-Parse.x86_64: W: no-manual-page-for-binary validjson
2 packages and 1 specfiles checked; 0 errors, 3 warnings.
Rpmlint is ok

$ rpm -qplv perl-JSON-Parse-0.49-1.fc28.x86_64.rpm
-rwxr-xr-x    1 root    root                     1121 Aug 18 12:33 /usr/bin/validjson
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/lib/.build-id
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/lib/.build-id/8c
lrwxrwxrwx    1 root    root                       64 Aug 18 12:33 /usr/lib/.build-id/8c/049a7e2ba55b5d11475fdc41307f5b80a53869 -> ../../../../usr/lib64/perl5/vendor_perl/auto/JSON/Parse/Parse.so
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/lib64/perl5/vendor_perl/JSON
-rw-r--r--    1 root    root                     1850 Nov 26  2016 /usr/lib64/perl5/vendor_perl/JSON/Parse.pm
-rw-r--r--    1 root    root                    54160 Nov 26  2016 /usr/lib64/perl5/vendor_perl/JSON/Parse.pod
-rw-r--r--    1 root    root                      369 Nov 26  2016 /usr/lib64/perl5/vendor_perl/JSON/Tokenize.pm
-rw-r--r--    1 root    root                     3496 Nov 26  2016 /usr/lib64/perl5/vendor_perl/JSON/Tokenize.pod
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/lib64/perl5/vendor_perl/auto/JSON
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/lib64/perl5/vendor_perl/auto/JSON/Parse
-rwxr-xr-x    1 root    root                    87832 Aug 18 12:33 /usr/lib64/perl5/vendor_perl/auto/JSON/Parse/Parse.so
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse
-rw-r--r--    1 root    root                     3955 Nov 26  2016 /usr/share/doc/perl-JSON-Parse/Changes
-rw-r--r--    1 root    root                     3448 Nov 26  2016 /usr/share/doc/perl-JSON-Parse/README
drwxr-xr-x    2 root    root                        0 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples
-rw-r--r--    1 root    root                      133 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/array.pl
-rw-r--r--    1 root    root                      231 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/assert.pl
-rw-r--r--    1 root    root                      363 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/bad-utf8.pl
-rw-r--r--    1 root    root                      376 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/bad-utf8.txt
-rw-r--r--    1 root    root                      285 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/chr.pl
-rw-r--r--    1 root    root                      221 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/collide.pl
-rw-r--r--    1 root    root                      229 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/ebi.pl
-rw-r--r--    1 root    root                      814 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/first-bit.pl
-rw-r--r--    1 root    root                      132 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/hash.pl
-rw-r--r--    1 root    root                     1055 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/json-tiny-round-trip-demo.pl
-rw-r--r--    1 root    root                      313 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/kani.pl
-rw-r--r--    1 root    root                      217 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/key-collision.pl
-rw-r--r--    1 root    root                      198 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/long-number.pl
-rw-r--r--    1 root    root                      222 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/sasori.pl
-rw-r--r--    1 root    root                      186 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/synopsis.pl
-rwxr-xr-x    1 root    root                      627 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/tokenize-synopsis.pl
-rw-r--r--    1 root    root                      221 Aug 18 12:33 /usr/share/doc/perl-JSON-Parse/examples/true-subs.pl
-rw-r--r--    1 root    root                    18966 Aug 18 12:33 /usr/share/man/man3/JSON::Parse.3pm.gz
-rw-r--r--    1 root    root                     2494 Aug 18 12:33 /usr/share/man/man3/JSON::Tokenize.3pm.gz

FIX: Please remove execute permission from examples/tokenize-synopsis.pl,
     it is part of documentation.

BuildRequires are almost ok.
FIX: Please add perl-devel and gcc, because it is XS package

Please correct all 'FIX' issues.

Otherwise, the package looks good.
Approved.

Comment 2 Emmanuel Seyman 2017-08-18 14:20:08 UTC
(In reply to Jitka Plesnikova from comment #1)
>
> FIX: Please remove execute permission from examples/tokenize-synopsis.pl,
>      it is part of documentation.

Done.

> BuildRequires are almost ok.
> FIX: Please add perl-devel and gcc, because it is XS package

Done.

Spec URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse.spec
SRPM URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse-0.49-2.fc26.src.rpm

> Otherwise, the package looks good.
> Approved.

\o/
Thanks for the review, Jitka.

Repo requested : https://pagure.io/releng/fedora-scm-requests/issue/130

Comment 3 Gwyn Ciesla 2017-08-18 14:31:22 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-JSON-Parse


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