Bug 1835263
Summary: | Review Request: perl-Crypt-PWSafe3 - Read and write Passwordsafe v3 files | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Charles R. Anderson <cra> |
Component: | Package Review | Assignee: | Petr Pisar <ppisar> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | package-review, ppisar |
Target Milestone: | --- | Flags: | ppisar:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2020-05-21 01:03:46 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: | |||
Bug Depends On: | 1835258 | ||
Bug Blocks: | 1832088 |
Description
Charles R. Anderson
2020-05-13 13:55:22 UTC
This is a re-review to unretire the package. This package requires perl-Crypt-ECB, being reviewed for unretirement in #1835258. >rpmlint perl-Crypt-PWSafe3.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. >rpmlint perl-Crypt-PWSafe3-1.22-11.fc33.src.rpm perl-Crypt-PWSafe3.src: W: spelling-error %description -l en_US passwordsafe -> password safe, password-safe, password perl-Crypt-PWSafe3.src: W: strange-permission Crypt-PWSafe3-1.22.tar.gz 640 perl-Crypt-PWSafe3.src: W: strange-permission perl-Crypt-PWSafe3.spec 640 1 packages and 0 specfiles checked; 0 errors, 3 warnings. >rpmlint perl-Crypt-PWSafe3-1.22-11.fc33.noarch.rpm perl-Crypt-PWSafe3.noarch: W: spelling-error %description -l en_US passwordsafe -> password safe, password-safe, password 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Blocks kpcli installation. License: Artistic 2.0 License statement from README and all the files in the source distribution: COPYRIGHT Crypt::PWSafe3 Copyright (c) 2011-2016 by T.v.Dein <tlinden> LICENSE This program is free software; you can redistribute it and/or modify it under the same terms of the Artistic License 2.0, see: L<http://www.perlfoundation.org/artistic_license_2_0> Updated to use the latest perl packaging best practices: Spec URL: https://fedorapeople.org/~cra/perl-Crypt-PWSafe3/perl-Crypt-PWSafe3.spec SRPM URL: https://fedorapeople.org/~cra/perl-Crypt-PWSafe3/perl-Crypt-PWSafe3-1.22-12.fc33.src.rpm The previous package review was bug #1183286. Url and Source0 addresses are Ok. Source0 archive (SHA-256: f390c6d84e289544a0f7bee9a9941bb34579cbf526df7c78eb77b3e177c78eea) is original. Ok. Summary verified from lib/Crypt/PWSafe3.pm. Ok. Description verified from lib/Crypt/PWSafe3.pm. Ok. License verified from README, lib/Crypt/PWSafe3/Field.pm, lib/Crypt/PWSafe3/PasswordPolicy.pm, lib/Crypt/PWSafe3/Databaseformat.pm, lib/Crypt/PWSafe3/SHA256.pm, lib/Crypt/PWSafe3/HeaderField.pm, lib/Crypt/PWSafe3/Record.pm, lib/Crypt/PWSafe3.pm, t/run.t, and Makefile.PL. Ok. No XS code, noarch BuildArch is Ok. FIX: Do not build-require 'findutils' and 'coreutils'. They are not used anywhere. TODO: Constrain 'perl(ExtUtils::MakeMaker)' dependency with '>= 6.76' because of the NO_PACKLIST=1, NO_PERLLOCAL=1 Makefile.PL arguments. All tests pass. OK. $ rpmlint perl-Crypt-PWSafe3.spec ../SRPMS/perl-Crypt-PWSafe3-1.22-12.fc33.src.rpm ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm sh: /usr/bin/python2: No such file or directory perl-Crypt-PWSafe3.src: W: spelling-error %description -l en_US passwordsafe -> password safe, password-safe, password perl-Crypt-PWSafe3.noarch: W: spelling-error %description -l en_US passwordsafe -> password safe, password-safe, password perl-Crypt-PWSafe3.noarch: W: spelling-error %description -l en_US passwordsafe -> password safe, password-safe, password 3 packages and 1 specfiles checked; 0 errors, 3 warnings. rpmlint is Ok. $ rpm -q -lv -p ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm drwxr-xr-x 2 root root 0 May 18 13:23 /usr/share/doc/perl-Crypt-PWSafe3 -rw-r--r-- 1 root root 4396 Mar 14 2016 /usr/share/doc/perl-Crypt-PWSafe3/CHANGELOG -rw-r--r-- 1 root root 1170 Mar 14 2016 /usr/share/doc/perl-Crypt-PWSafe3/README -rw-r--r-- 1 root root 3867 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3.3pm.gz -rw-r--r-- 1 root root 8688 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::Databaseformat.3pm.gz -rw-r--r-- 1 root root 1728 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::Field.3pm.gz -rw-r--r-- 1 root root 1876 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::HeaderField.3pm.gz -rw-r--r-- 1 root root 1839 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::PasswordPolicy.3pm.gz -rw-r--r-- 1 root root 2728 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::Record.3pm.gz -rw-r--r-- 1 root root 1518 May 18 13:23 /usr/share/man/man3/Crypt::PWSafe3::SHA256.3pm.gz drwxr-xr-x 2 root root 0 May 18 13:23 /usr/share/perl5/vendor_perl/Crypt drwxr-xr-x 2 root root 0 May 18 13:23 /usr/share/perl5/vendor_perl/Crypt/PWSafe3 -rw-r--r-- 1 root root 24177 Mar 14 2016 /usr/share/perl5/vendor_perl/Crypt/PWSafe3.pm -rw-r--r-- 1 root root 20735 Nov 15 2012 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/Databaseformat.pm -rw-r--r-- 1 root root 4427 Feb 16 2015 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/Field.pm -rw-r--r-- 1 root root 4777 Feb 16 2015 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/HeaderField.pm -rw-r--r-- 1 root root 4163 Feb 12 2015 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/PasswordPolicy.pm -rw-r--r-- 1 root root 10235 May 21 2015 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/Record.pm -rw-r--r-- 1 root root 1303 Feb 16 2015 /usr/share/perl5/vendor_perl/Crypt/PWSafe3/SHA256.pm A file layout and the permissions are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.30.2) 1 perl(:VERSION) >= 5.10.0 1 perl(Bytes::Random::Secure) 1 perl(Carp) 1 perl(Carp::Heavy) 1 perl(Config) 1 perl(Crypt::CBC) >= 2.3 1 perl(Crypt::ECB) >= 1.45 1 perl(Crypt::PWSafe3::Field) 1 perl(Crypt::PWSafe3::HeaderField) 1 perl(Crypt::PWSafe3::PasswordPolicy) 1 perl(Crypt::PWSafe3::Record) 1 perl(Crypt::PWSafe3::SHA256) 1 perl(Crypt::Random) >= 1.25 1 perl(Crypt::Twofish) >= 2.14 1 perl(Data::Dumper) 1 perl(Data::UUID) >= 1.22 1 perl(Digest::HMAC) >= 1 1 perl(Digest::SHA) >= 1 1 perl(Exporter) 1 perl(File::Copy) 1 perl(File::Spec) 1 perl(File::Temp) 1 perl(FileHandle) 1 perl(strict) 1 perl(utf8) 1 perl(vars) 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 TODO: Bytes::Random::Secure is optional, Crypt::Random is a fallback. Change the 'perl(Bytes::Random::Secure)' from Requires to Recommends. $ rpm -q --provides -p ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm | sort -f | uniq -c 1 perl(Crypt::PWSafe3) = 1.22 1 perl(Crypt::PWSafe3::Field) = 1.06 1 perl(Crypt::PWSafe3::HeaderField) = 1.05 1 perl(Crypt::PWSafe3::PasswordPolicy) = 1.01 1 perl(Crypt::PWSafe3::Record) = 1.10 1 perl(Crypt::PWSafe3::SHA256) = 1.03 1 perl-Crypt-PWSafe3 = 1.22-12.fc33 Binary provides are Ok. $ resolvedeps rawhide ../RPMS/noarch/perl-Crypt-PWSafe3-1.22-12.fc33.noarch.rpm Binary dependencies are resolvable. Ok. Otherwise the package is in line with Fedora and Perl packaging guidelines. Please correct all the 'FIX' items, and consider fixing the 'TODO' items before building this package. The package builds in Fedora 33 (https://koji.fedoraproject.org/koji/taskinfo?taskID=44646874). Ok. Resolution: Package APPROVED. |