https://bugzilla.redhat.com/show_bug.cgi?id=1482867
Bug ID: 1482867 Summary: Review Request: perl-JSON-Parse - Read JSON into a Perl variable Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: emmanuel@seyman.fr QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse.spe... SRPM URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse-0.4... Description: A Perl module for parsing JSON. (JSON means "JavaScript Object Notation" and it is specified in "RFC 7159".)
Fedora Account System Username: eseyman
https://bugzilla.redhat.com/show_bug.cgi?id=1482867
Jitka Plesnikova jplesnik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |jplesnik@redhat.com Assignee|nobody@fedoraproject.org |jplesnik@redhat.com Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1482867
Jitka Plesnikova jplesnik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #1 from Jitka Plesnikova jplesnik@redhat.com --- 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.
https://bugzilla.redhat.com/show_bug.cgi?id=1482867
Emmanuel Seyman emmanuel@seyman.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1472755
--- Comment #2 from Emmanuel Seyman emmanuel@seyman.fr --- (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.spe... SRPM URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse-0.4...
Otherwise, the package looks good. Approved.
\o/ Thanks for the review, Jitka.
Repo requested : https://pagure.io/releng/fedora-scm-requests/issue/130
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1472755 [Bug 1472755] perl-JSON-Path-0.411 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1482867
--- Comment #3 from Gwyn Ciesla limburgher@gmail.com --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-JSON-Parse
https://bugzilla.redhat.com/show_bug.cgi?id=1482867
Jitka Plesnikova jplesnik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |perl-JSON-Parse-0.49-2.fc28 Resolution|--- |RAWHIDE Last Closed| |2017-09-06 06:44:55
package-review@lists.fedoraproject.org