https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Bug ID: 1876259 Summary: Review Request: perl-URI-cpan - URLs that refer to things on the CPAN Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: paul@city-fan.org QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-URI-cpan/branches/fedora/... SRPM URL: http://www.city-fan.org/~paul/extras/perl-URI-cpan/perl-URI-cpan-1.007-2.fc3... Description: This module is for handling URLs that refer to things on the CPAN.
Fedora Account System Username: pghmcfc
This is needed to update perl-Perl-PrereqScanner-NotQuiteLite to the current upstream release.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1878990
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1878990 [Bug 1878990] Upgrade perl-Perl-PrereqScanner-NotQuiteLite to 0.9913
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |ppisar@redhat.com Assignee|nobody@fedoraproject.org |ppisar@redhat.com Doc Type|--- |If docs needed, set a value Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #1 from Petr Pisar ppisar@redhat.com --- The URL and Source0 addresses are Ok. The Source0 archive (SHA-256: 125a1394662e0a45e2696977496c1d027b14387ff6e39b60c07e0f96eae152d3) is original. Ok. Summary verified from lib/URI/cpan.pm. Ok. Description is Ok. License verified from lib/URI/cpan/distfile.pm, lib/URI/cpan/package.pm, lib/URI/cpan/module.pm, lib/URI/cpan/author.pm, lib/URI/cpan/dist.pm, lib/URI/cpan.pm, Makefile.PL, LICENSE, and README. Ok. No XS code, noarch BuildArch is Ok.
TODO: Consider using %{make_install} macro with Makefile.PL NO_PACKLIST=1 NO_PERLLOCAL=1. FIX: Move 'perl(URI)' BuildRequires to tests section (t/uri-cpan.t:4). It's not used at a run-time.
All tests pass. Ok.
$ rpmlint perl-URI-cpan.spec ../SRPMS/perl-URI-cpan-1.007-2.fc34.src.rpm ../RPMS/noarch/perl-URI-cpan-1.007-2.fc34.noarch.rpm sh: /usr/bin/python2: No such file or directory 2 packages and 1 specfiles checked; 0 errors, 0 warnings. rpmlint is Ok.
$ rpm -q -lv -p ../RPMS/noarch/perl-URI-cpan-1.007-2.fc34.noarch.rpm drwxr-xr-x 2 root root 0 Sep 15 15:41 /usr/share/doc/perl-URI-cpan -rw-r--r-- 1 root root 632 Apr 17 2015 /usr/share/doc/perl-URI-cpan/Changes -rw-r--r-- 1 root root 372 Apr 17 2015 /usr/share/doc/perl-URI-cpan/README drwxr-xr-x 2 root root 0 Sep 15 15:41 /usr/share/licenses/perl-URI-cpan -rw-r--r-- 1 root root 18355 Apr 17 2015 /usr/share/licenses/perl-URI-cpan/LICENSE -rw-r--r-- 1 root root 1796 Sep 15 15:41 /usr/share/man/man3/URI::cpan.3pm.gz -rw-r--r-- 1 root root 1298 Sep 15 15:41 /usr/share/man/man3/URI::cpan::author.3pm.gz -rw-r--r-- 1 root root 1296 Sep 15 15:41 /usr/share/man/man3/URI::cpan::dist.3pm.gz -rw-r--r-- 1 root root 1599 Sep 15 15:41 /usr/share/man/man3/URI::cpan::distfile.3pm.gz -rw-r--r-- 1 root root 1298 Sep 15 15:41 /usr/share/man/man3/URI::cpan::module.3pm.gz -rw-r--r-- 1 root root 1300 Sep 15 15:41 /usr/share/man/man3/URI::cpan::package.3pm.gz drwxr-xr-x 2 root root 0 Sep 15 15:41 /usr/share/perl5/vendor_perl/URI drwxr-xr-x 2 root root 0 Sep 15 15:41 /usr/share/perl5/vendor_perl/URI/cpan -rw-r--r-- 1 root root 3601 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan.pm -rw-r--r-- 1 root root 857 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan/author.pm -rw-r--r-- 1 root root 509 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan/dist.pm -rw-r--r-- 1 root root 2540 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan/distfile.pm -rw-r--r-- 1 root root 515 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan/module.pm -rw-r--r-- 1 root root 518 Apr 17 2015 /usr/share/perl5/vendor_perl/URI/cpan/package.pm
A file layout and the permissions are Ok.
$ rpm -q --requires -p ../RPMS/noarch/perl-URI-cpan-1.007-2.fc34.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.32.0) 1 perl(Carp) 1 perl(CPAN::DistnameInfo) 1 perl(parent) 1 perl(strict) 1 perl(URI::cpan) 1 perl(URI::cpan::author) 1 perl(URI::cpan::dist) 1 perl(URI::cpan::distfile) 1 perl(URI::cpan::module) 1 perl(URI::cpan::package) 1 perl(URI::_generic) 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 Binary requires are Ok.
$ rpm -q --provides -p ../RPMS/noarch/perl-URI-cpan-1.007-2.fc34.noarch.rpm | sort -f | uniq -c 1 perl(URI::cpan) = 1.007 1 perl(URI::cpan::author) = 1.007 1 perl(URI::cpan::dist) = 1.007 1 perl(URI::cpan::distfile) = 1.007 1 perl(URI::cpan::module) = 1.007 1 perl(URI::cpan::package) = 1.007 1 perl-URI-cpan = 1.007-2.fc34 Binary provides are Ok.
$ resolvedeps rawhide ../RPMS/noarch/perl-URI-cpan-1.007-2.fc34.noarch.rpm Binary dependencies are resolvable. Ok.
The package builds in F34 (https://koji.fedoraproject.org/koji/taskinfo?taskID=51510948). Ok.
Otherwise the package is in line with Fedora and Perl packaging guidelines. Please correct the 'FIX' item, and consider fixing the 'TODO' item before building this package. Resolution: Package APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #2 from Paul Howarth paul@city-fan.org --- Thanks Petr.
Repo request: https://pagure.io/releng/fedora-scm-requests/issue/29079
I'll fix the perl(URI) categorisation on import.
I'll switch to %{make_install} after branching for epel7, which doesn't have a new enough EU::MM.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #3 from Gwyn Ciesla gwync@protonmail.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-URI-cpan
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
--- Comment #4 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-bc55da4e50 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2020-bc55da4e50
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #5 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-5b408002a0 has been submitted as an update to Fedora 33. https://bodhi.fedoraproject.org/updates/FEDORA-2020-5b408002a0
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #6 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-1da58546d4 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-1da58546d4
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |perl-URI-cpan-1.007-4.fc34
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #7 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-1da58546d4 has been pushed to the Fedora 32 testing repository. In short time you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-1da58546d4 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-1da58546d4
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #8 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2020-3cdcbc56e0 has been pushed to the Fedora EPEL 7 testing repository.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2020-3cdcbc56e0
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2020-8bb1d5473c has been pushed to the Fedora EPEL 8 testing repository.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2020-8bb1d5473c
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #10 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-bc55da4e50 has been pushed to the Fedora 31 testing repository. In short time you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-bc55da4e50 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-bc55da4e50
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #11 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-5b408002a0 has been pushed to the Fedora 33 testing repository. In short time you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-5b408002a0 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-5b408002a0
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed| |2020-09-23 17:12:09
--- Comment #12 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-1da58546d4 has been pushed to the Fedora 32 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #13 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-bc55da4e50 has been pushed to the Fedora 31 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #14 from Fedora Update System updates@fedoraproject.org --- FEDORA-2020-5b408002a0 has been pushed to the Fedora 33 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #15 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2020-3cdcbc56e0 has been pushed to the Fedora EPEL 7 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1876259
--- Comment #16 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2020-8bb1d5473c has been pushed to the Fedora EPEL 8 stable repository. If problem still persists, please make note of it in this bug report.
package-review@lists.fedoraproject.org