https://bugzilla.redhat.com/show_bug.cgi?id=1954004
Bug ID: 1954004 Summary: Review Request: perl-JSON-Create - Create JSON Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: ppisar@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://ppisar.fedorapeople.org/perl-JSON-Create/perl-JSON-Create.spec SRPM URL: https://ppisar.fedorapeople.org/perl-JSON-Create/perl-JSON-Create-0.32-1.fc3... Description: JSON::Create encodes Perl variables into JSON. The basic routine "create_json" gives common defaults. The stricter version "create_json_strict" accepts only unambiguous inputs. For more customization, an object created with "new" and run with "create" allows specifying behavior in more detail.
Fedora Account System Username: ppisar
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
Emmanuel Seyman emmanuel@seyman.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |emmanuel@seyman.fr Assignee|nobody@fedoraproject.org |emmanuel@seyman.fr Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
--- Comment #1 from Emmanuel Seyman emmanuel@seyman.fr --- The tests included in the distribution fail to run because the spec file does not build-require perl(Test::More). Petr, can you fix this before I continue with the review?
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
--- Comment #2 from Petr Pisar ppisar@redhat.com --- Mea culpa. The corrected package is on the same address.
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
--- Comment #3 from Petr Pisar ppisar@redhat.com --- I've just added also File::Temp and File::Spec required by a patch.
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
Emmanuel Seyman emmanuel@seyman.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #4 from Emmanuel Seyman emmanuel@seyman.fr ---
Description verified from lib/JSON/Create.pod. Ok. License verified from Makefile.PL, and lib/JSON/Create.pod. Ok. Build-time dependencies are Ok. All tests pass. Ok.
$ rpmlint *.spec /var/lib/mock/fedora-rawhide-x86_64/result/ perl-JSON-Create-tests.noarch: W: no-documentation 5 packages and 1 specfiles checked; 0 errors, 1 warnings. rpmlint output is Ok.
File layout and permissions are Ok.
$ rpm -q --requires -p perl-JSON-Create-0.32-1.fc35.x86_64.rpm | sort -f | uniq -c 1 libc.so.6()(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.32()(64bit) 1 libpthread.so.0()(64bit) 1 libpthread.so.0(GLIBC_2.2.5)(64bit) 1 perl(:MODULE_COMPAT_5.32.1) 1 perl(B) 1 perl(Carp) 1 perl(Exporter) 1 perl(JSON::Parse) >= 0.60 1 perl(parent) 1 perl(Scalar::Util) 1 perl(strict) 1 perl(Unicode::UTF8) >= 0.62 1 perl(utf8) 1 perl(warnings) 1 rpmlib(CompressedFileNames) <= 3.0.4-1 1 rpmlib(FileDigests) <= 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 1 rpmlib(PayloadIsZstd) <= 5.4.18-1 1 rtld(GNU_HASH) Binary Requires are Ok.
$ rpm -q --provides -p ../RPMS/noarch/perl-Test-Lib-0.002-1.fc35.noarch.rpm | sort -f | uniq -c 1 perl(JSON::Create) = 0.32 1 perl(JSON::Create::Bool) = 0.32 1 perl(JSON::Create::PP) = 0.32 1 perl-JSON-Create = 0.32-1.fc35 1 perl-JSON-Create(x86-64) = 0.32-1.fc35 Binary Provides are Ok.
The package builds in F35 (https://koji.fedoraproject.org/koji/taskinfo?taskID=67220547). Ok.
The package is in line with Fedora and Perl packaging guidelines. Resolution: Package APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
--- Comment #5 from Petr Pisar ppisar@redhat.com --- https://pagure.io/releng/fedora-scm-requests/issue/33805
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED Fixed In Version| |perl-JSON-Create-0.32-1.fc3 | |5
--- Comment #6 from Petr Pisar ppisar@redhat.com --- Thank you for the review and the repository.
https://bugzilla.redhat.com/show_bug.cgi?id=1954004
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE Last Closed| |2021-05-05 14:16:58
package-review@lists.fedoraproject.org