Bug 1089634 - Review Request: perl-Crypt-SaltedHash - Perl interface to functions that assist in working with salted hashes
Summary: Review Request: perl-Crypt-SaltedHash - Perl interface to functions that assi...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Šabata
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-21 09:54 UTC by David Dick
Modified: 2014-05-12 02:05 UTC (History)
2 users (show)

Fixed In Version: perl-Crypt-SaltedHash-0.09-1.el6
Clone Of:
Environment:
Last Closed: 2014-05-03 19:54:11 UTC
Type: ---
Embargoed:
psabata: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description David Dick 2014-04-21 09:54:21 UTC
Spec URL: http://ddick.fedorapeople.org/packages/perl-Crypt-SaltedHash.spec
SRPM URL: http://ddick.fedorapeople.org/packages/perl-Crypt-SaltedHash-0.09-1.fc20.src.rpm
Description: Perl interface to functions that assist in working with salted hashes
Fedora Account System Username: ddick

Comment 3 Petr Šabata 2014-04-22 13:29:55 UTC
TODO: Fix a typo in the description, s/agorithms/algorithms/
TODO: Convert the line endings in Changes and README.
TODO: Remove the Digest::SHA build-time dependency; it doesn't appear to be used at all during build.  Keep the runtime dependency, though!

Anyhow, no showstoppers.  Approving.

Comment 4 David Dick 2014-04-23 09:35:07 UTC
Hi Petr, all fixed except for removing Digest::SHA as a build-time dependency.  It gets used in rawhide in t/bug-localize-regex-vars.t as seen below. It's undetectable as the only place it is referenced is in Crypt::SaltedHash->new when the algorithm is not specified.

PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
t/01use.t .................... ok
t/02pod.t .................... skipped: Test::Pod 1.14 required
t/03podcoverage.t ............ skipped: Test::Pod::Coverage 1.04 required
t/04Crypt-SaltedHash.t ....... ok
#   Failed test 'validate() lived even through non-match and $1 already set!'
#   at t/bug-localize-regex-vars.t line 19.
#          got: 'Can't locate Digest/SHA.pm in @INC (you may need to install the Digest::SHA module) (@INC contains: /builddir/build/BUILD/Crypt-SaltedHash-0.09/blib/lib /builddir/build/BUILD/Crypt-SaltedHash-0.09/blib/arch /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at /usr/share/perl5/vendor_perl/Digest.pm line 41.
# '
#     expected: undef
# Looks like you failed 1 test of 3.
t/bug-localize-regex-vars.t .. 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/3 subtests 
Test Summary Report
-------------------
t/bug-localize-regex-vars.t (Wstat: 256 Tests: 3 Failed: 1)
  Failed test:  2
  Non-zero exit status: 1
Files=5, Tests=10,  0 wallclock secs ( 0.03 usr  0.01 sys +  0.11 cusr  0.01 csys =  0.16 CPU)
Result: FAIL

Comment 5 David Dick 2014-04-23 09:36:43 UTC
New Package SCM Request
=======================
Package Name: perl-Crypt-SaltedHash
Short Description: Perl interface to functions that assist in working with salted hashes
Owners: ddick
Branches: f20 el6 epel7
InitialCC: perl-sig

Thanks Petr!

Comment 6 Petr Šabata 2014-04-23 11:33:14 UTC
(In reply to David Dick from comment #4)
> Hi Petr, all fixed except for removing Digest::SHA as a build-time
> dependency.  It gets used in rawhide in t/bug-localize-regex-vars.t as seen
> below. It's undetectable as the only place it is referenced is in
> Crypt::SaltedHash->new when the algorithm is not specified.

Yes, I saw this; however, I didn't notice it was called through validate() in t/bug-localize-regex-vars.t.  My bad :)

Comment 7 Gwyn Ciesla 2014-04-24 12:52:17 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2014-04-25 00:28:35 UTC
perl-Crypt-SaltedHash-0.09-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Crypt-SaltedHash-0.09-1.fc20

Comment 9 Fedora Update System 2014-04-25 00:35:20 UTC
perl-Crypt-SaltedHash-0.09-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/perl-Crypt-SaltedHash-0.09-1.el6

Comment 10 Fedora Update System 2014-04-25 21:57:10 UTC
perl-Crypt-SaltedHash-0.09-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 11 Fedora Update System 2014-05-03 19:54:11 UTC
perl-Crypt-SaltedHash-0.09-1.fc20 has been pushed to the Fedora 20 stable repository.

Comment 12 Fedora Update System 2014-05-12 02:05:39 UTC
perl-Crypt-SaltedHash-0.09-1.el6 has been pushed to the Fedora EPEL 6 stable repository.


Note You need to log in before you can comment on or make changes to this bug.