Bug 1242724
Summary: | Review Request: perl-File-Find-Rule-Age - Rule to match on file age | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tim Orling <TicoTimo> |
Component: | Package Review | Assignee: | Petr Šabata <psabata> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | package-review, psabata |
Target Milestone: | --- | Flags: | psabata:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | perl-File-Find-Rule-Age-0.302-3.fc21 perl-File-Find-Rule-Age-0.302-3.el6 perl-File-Find-Rule-Age-0.302-3.el7 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-08-17 20:20:36 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
Tim Orling
2015-07-14 04:04:09 UTC
Nothing really new here so I'll be brief. I explained all these in the other review :) I'll assume this one is going into EL6+ as well. * The BuildRoot tag can be dropped. * The perl version constraint can be dropped. * Missing buildtime dependencies: coreutils, findutils, make All of these are called in the specfile. * Missing perl buildtime dependencies: - base, lib/File/Find/Rule/Age.pm:8 - Carp, lib/File/Find/Rule/Age.pm:10 - File::stat, lib/File/Find/Rule/Age.pm:12 - FindBin, t/01-legacy.t:6 t/02-accessed.t:6 t/02-created.t:6 t/02-modified.t:6 - strict, Makefile.PL:1 - warnings, Makefile.PL:2 The lib/* and t/* deps are needed because you run the test suite (good!). * You should filter out the autogenerated unversioned runtime dependencies on DateTime, File::Find::Rule and Params::Util. For example like so: %global __requires_exclude ^perl\\(DateTime|File::Find::Rule|Params::Util\\)$ * There's no need to clean the buildroot (line 40), remove the emtpy directories (line 45), for the %clean section (lines 52-53) or for %defattr (line 56). * You may substitute PERL_INSTALL_ROOT with DESTDIR. * Don't package upstream distribution metadata, META.json, in %doc. * Correct the changelog header (missing e-mail address). Spec URL: https://ttorling.fedorapeople.org/perl-File-Find-Rule-Age/perl-File-Find-Rule-Age.spec SRPM URL: https://ttorling.fedorapeople.org/perl-File-Find-Rule-Age/perl-File-Find-Rule-Age-0.302-1.fc23.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=10494895 This package has also built successfully on epel6 and epel7 on copr. http://copr.fedoraproject.org/coprs/ttorling/perl-meta-cpan-support/build/103860/ http://copr.fedoraproject.org/coprs/ttorling/perl-meta-cpan-support/build/105870/ Okay, that's better. A few more notes, however: * Sort your dependencies; they will be easier to read and you'll be less likely to miss something or list it twice. (`base' and `Carp', I'm looking at you!) * You still needlessly remove empty directories in %install, now on line 54. * You added some trailing whitespace, lines 36 and 69. That's just bad taste ;) * It looks like you forgot to bump the release number; now you have two changelog entries for one VR. SPEC URL: https://ttorling.fedorapeople.org/perl-File-Find-Rule-Age/perl-File-Find-Rule-Age.spec SRPM URL: https://ttorling.fedorapeople.org/perl-File-Find-Rule-Age/perl-File-Find-Rule-Age-0.302-3.fc23.src.rpm Git URL: https://fedorapeople.org/cgit/ttorling/public_git/perl-File-Find-Rule-Age.git/ koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=10518459 copr: https://copr.fedoraproject.org/coprs/ttorling/perl-meta-cpan-support/build/106070/ https://copr.fedoraproject.org/coprs/ttorling/perl-meta-cpan-support/build/106071/ copr still failing for rawhide ppc: https://copr-be.cloud.fedoraproject.org/results/ttorling/perl-meta-cpan-support//fedora-rawhide-ppc64le/perl-File-Find-Rule-Age-0.302-3.fc23 This failure does not appear to be package dependent, but rather build system. (In reply to Tim Orling from comment #4) > copr still failing for rawhide ppc: > https://copr-be.cloud.fedoraproject.org/results/ttorling/perl-meta-cpan- > support//fedora-rawhide-ppc64le/perl-File-Find-Rule-Age-0.302-3.fc23 > > This failure does not appear to be package dependent, but rather build > system. I agree. It doesn't even get to the build phase. You've adressed everything I've pointed out. I'm approving this package. New Package SCM Request ======================= Package Name: perl-File-Find-Rule-Age Short Description: Rule to match on file age Upstream URL: http://search.cpan.org/dist/File-Find-Rule-Age/ Owners: ttorling psabata Branches: f21 f22 f23 el6 epel7 InitialCC: perl-sig Git done (by process-git-requests). perl-File-Find-Rule-Age-0.302-3.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/perl-File-Find-Rule-Age-0.302-3.el7 perl-File-Find-Rule-Age-0.302-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-File-Find-Rule-Age-0.302-3.el6 perl-File-Find-Rule-Age-0.302-3.fc23 has been submitted as an update for Fedora 23. https://admin.fedoraproject.org/updates/perl-File-Find-Rule-Age-0.302-3.fc23 perl-File-Find-Rule-Age-0.302-3.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/perl-File-Find-Rule-Age-0.302-3.fc22 perl-File-Find-Rule-Age-0.302-3.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/perl-File-Find-Rule-Age-0.302-3.fc21 perl-File-Find-Rule-Age-0.302-3.fc22 has been pushed to the Fedora 22 stable repository. perl-File-Find-Rule-Age-0.302-3.fc23 has been pushed to the Fedora 23 stable repository. perl-File-Find-Rule-Age-0.302-3.fc21 has been pushed to the Fedora 21 stable repository. perl-File-Find-Rule-Age-0.302-3.el7 has been pushed to the Fedora EPEL 7 stable repository. perl-File-Find-Rule-Age-0.302-3.el6 has been pushed to the Fedora EPEL 6 stable repository. |