Bug 2126348
Summary: | Review Request: perl-Tickit - Perl bindings for Tickit | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Petr Salaba <psalaba> |
Component: | Package Review | Assignee: | Jitka Plesnikova <jplesnik> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | jplesnik, package-review |
Target Milestone: | --- | Flags: | jplesnik:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | perl-Tickit-0.73-1.fc38 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2022-11-07 18:46:01 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Petr Salaba
2022-09-13 08:45:07 UTC
Source file is ok Summary is ok License is ok Description is ok URL and Source0 are ok All tests passed BuildRequires FIX: Please add build-requires: perl(base) - lib/Tickit/Pen.pm:356 perl(Carp) - lib/Tickit.pm:11 perl(Config) - spec file:54 perl(constant) - lib/Tickit/Pen.pm:357 perl(Errno) - t/30tickit.t:16 perl(integer) - lib/Tickit/Utils.pm:277 perl(IO::Handle) - lib/Tickit.pm:13 perl(List::Util) - lib/Tickit/RectSet.pm:11 perl(overload) - lib/Tickit/Pen.pm:345 perl(POSIX) - t/03utils.t:8 perl(Scalar::Util) - lib/Tickit.pm:15 perl(Test::Builder) - lib/Tickit/Test.pm:54 perl(Time::HiRes) - lib/Tickit.pm:16 perl(utf8) - t/03utils.t:23 perl(XSLoader) - lib/Tickit.pm:19 $ rpm -qp --requires perl-Tickit-0.73-1.fc38.x86_64.rpm | sort | uniq -c | grep -v rpmlib 1 libc.so.6()(64bit) 1 libc.so.6(GLIBC_2.2.5)(64bit) 1 libc.so.6(GLIBC_2.4)(64bit) 1 libc.so.6(GLIBC_2.7)(64bit) 1 libtickit.so.3()(64bit) 1 perl(base) 1 perl(Carp) 1 perl(constant) 1 perl(Exporter) 1 perl(IO::Handle) 1 perl(List::Util) 1 perl(:MODULE_COMPAT_5.36.0) 1 perl(overload) 1 perl(Scalar::Util) 1 perl(Struct::Dumb) >= 0.04 1 perl(Test::Builder) 1 perl(Tickit) 1 perl(Tickit::Debug) 1 perl(Tickit::Event) 1 perl(Tickit::Pen) >= 0.31 1 perl(Tickit::Rect) 1 perl(Tickit::RectSet) 1 perl(Tickit::RenderBuffer) 1 perl(Tickit::Term) 1 perl(Tickit::Test::MockTerm) 1 perl(Tickit::Utils) 1 perl(Tickit::Window) 1 perl(Time::HiRes) 1 perl(:VERSION) >= 5.14.0 1 perl(warnings) 1 rtld(GNU_HASH) FIX: Please add run-requires: perl(XSLoader) - lib/Tickit.pm:19 $ rpm -qp --requires perl-Tickit-tests-0.73-1.fc38.x86_64.rpm | sort | uniq -c | grep -v rpmlib 1 perl(Errno) 1 perl(IO::Handle) 1 perl(POSIX) 1 perl(Test::Builder::Tester) 2 perl(Test::Fatal) 1 perl(Test::Harness) 1 perl(Test::HexString) 1 perl(Test::More) 1 perl(Test::Refcount) 1 perl(Tickit) 1 perl-Tickit = 0.73-1.fc38 1 perl(Tickit::Pen) 1 perl(Tickit::Rect) 1 perl(Tickit::RectSet) 1 perl(Tickit::RenderBuffer) 1 perl(Tickit::StringPos) 1 perl(Tickit::Term) 1 perl(Tickit::Test) 1 perl(Tickit::Utils) 1 perl(Tickit::Window) 1 perl(Time::HiRes) 1 perl(utf8) 1 perl(:VERSION) >= 5.14.0 1 perl(warnings) 1 /usr/bin/perl 1 /usr/bin/sh FIX: Please remove duplicated run-require 'perl(Test::Fatal)' FIX: Please add run-requires 'perl(constant)' $ rpm -qp --provides perl-Tickit-0.73-1.fc38.x86_64.rpm | sort | uniq -c 1 perl(Tickit) = 0.73 1 perl-Tickit = 0.73-1.fc38 1 perl(Tickit::Debug) = 0.73 1 perl(Tickit::Event) = 0.73 1 perl(Tickit::Pen) = 0.73 1 perl(Tickit::Pen::Immutable) = 0.73 1 perl(Tickit::Pen::Mutable) = 0.73 1 perl(Tickit::Rect) = 0.73 1 perl(Tickit::RectSet) = 0.73 1 perl(Tickit::RenderBuffer) = 0.73 1 perl(Tickit::StringPos) = 0.73 1 perl(Tickit::Term) = 0.73 1 perl(Tickit::Test) = 0.73 1 perl(Tickit::Test::MockTerm) = 0.73 1 perl(Tickit::Utils) = 0.73 1 perl(Tickit::Window) = 0.73 1 perl-Tickit(x86-64) = 0.73-1.fc38 $ rpm -qp --provides perl-Tickit-tests-0.73-1.fc38.x86_64.rpm | sort | uniq -c 1 perl-Tickit-tests = 0.73-1.fc38 1 perl-Tickit-tests(x86-64) = 0.73-1.fc38 Binary provides are Ok. $ rpmlint ./perl-Tickit* Rpmlint is ok FIX: In section %files, - please move LICENSE from %docs to %license - please remove META.json, it is only metadata - not useful FIX: The t/99pod.t has to be removed from perl-Tickit-tests. Test::Pod check works only for lib/ in the repository. t/99pod.t ................. 1/1 # Failed test 'POD test for lib' # at /usr/share/perl5/vendor_perl/Test/Pod.pm line 184. # lib does not exist # Looks like you failed 1 test of 1. t/99pod.t ................. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/1 subtests Otherwise the package is in line with Fedora and Perl packaging guide lines. Please correct all 'FIX' items and provide a new spec file. Resolution: NOT Approved Any progress? Sorry for the late update, here's the fixed/updated files: Spec URL: https://rpm.urumasi.xyz/perl-Tickit/perl-Tickit.spec SRPM URL: https://rpm.urumasi.xyz/perl-Tickit/perl-Tickit-0.73-1.fc38.src.rpm > BuildRequires > FIX: Please add build-requires: <...> Almost all missing dependencies were added only one is missing. FIX: Add BR - perl(Config) - spec file:54 > FIX: Please add run-requires: > perl(XSLoader) - lib/Tickit.pm:19 +# Runtime +Requires: perl(XSLoader) Ok. > $ rpm -qp --requires perl-Tickit-tests-0.73-1.fc38.x86_64.rpm | sort | uniq -c | grep -v rpmlib <...> > FIX: Please remove duplicated run-require 'perl(Test::Fatal)' -Requires: perl(Test::Fatal) Ok > FIX: Please add run-requires 'perl(constant)' Still need to be fixed > FIX: In section %files, > - please move LICENSE from %docs to %license > - please remove META.json, it is only metadata - not useful -%doc Changes examples LICENSE META.json README +%doc Changes examples README +%license LICENSE Ok. > FIX: The t/99pod.t has to be removed from perl-Tickit-tests. Test::Pod check > works only for lib/ in the repository. +rm %{buildroot}%{_libexecdir}/%{name}/t/99pod.t Ok. Please correct all 'FIX' items and provide a new spec file. Resolution: NOT Approved Checked and corrected all FIXes, thanks for the review Spec URL: https://rpm.urumasi.xyz/perl-Tickit/perl-Tickit.spec SRPM URL: https://rpm.urumasi.xyz/perl-Tickit/perl-Tickit-0.73-1.fc38.src.rpm (In reply to Jitka Plesnikova from comment #4) > > BuildRequires > > FIX: Please add build-requires: > <...> > Almost all missing dependencies were added only one is missing. > FIX: Add BR - perl(Config) - spec file:54 +BuildRequires: perl(Config) Ok. > > FIX: Please add run-requires 'perl(constant)' > Still need to be fixed # Runtime +Requires: perl(constant) Requires: perl(XSLoader) Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version)) @@ -63,6 +63,7 @@ Summary: Tests for %{name} Requires: %{name} = %{?epoch:%{epoch}:}%{version}-%{release} Requires: perl(Test::Harness) +Requires: perl(constant) The run-require should be added only to sub-package perl-Tickit-tests. FIX: Remove run-requires 'perl(constant)' from the main package perl-Tickit, because it is found by generators. Please correct 'FIX' item. Otherwise the package is in line with Fedora and Perl packaging guide lines. Resolution: Approved (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-Tickit |