Bug 1033494

Summary: Review Request:perl-Cache-Memory-Simple - Yet another on memory cache
Product: [Fedora] Fedora Reporter: Mathieu Bridon <bochecha>
Component: Package ReviewAssignee: Petr Šabata <psabata>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: bochecha, package-review, psabata
Target Milestone: ---Flags: psabata: fedora-review?
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-05-15 09:04:21 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 1033481    
Bug Blocks:    

Description Mathieu Bridon 2013-11-22 08:47:57 UTC
Spec URL: http://bochecha.fedorapeople.org/packages/perl-Cache-Memory-Simple.spec
SRPM URL: http://bochecha.fedorapeople.org/packages/perl-Cache-Memory-Simple-1.02-1.fc20.src.rpm

Description:
Cache::Memory::Simple is yet another on memory cache implementation.

Fedora Account System Username: bochecha

Comment 1 Mathieu Bridon 2013-11-26 09:43:07 UTC
Based on your feedback on other review requests, here is a new package that adds missing build requirements and removes unneeded explicit requirements.

Spec URL: http://bochecha.fedorapeople.org/packages/perl-Cache-Memory-Simple.spec
SRPM URL: http://bochecha.fedorapeople.org/packages/perl-Cache-Memory-Simple-1.02-2.fc20.src.rpm

Comment 2 Petr Šabata 2014-09-05 10:28:31 UTC
I'm sorry for the long delay.  Finally getting to this then.

Your package is missing some buildtime deps, namely:
- constant, lib/Cache/Memory/Simple/Scalar.pm:9, via t/03_scalar.t:6
- strict, various files, including Build.PL itself
- utf8, ditto
- warnings, ditto

Both Summary and %description appear to be missing some words.  Consider "Yet another take on memory cache implementation" as Summary.  Change the %description in a similar way.

Also, there's been a new release meanwhile.

Comment 3 Petr Šabata 2015-05-14 08:50:06 UTC
Ping.  Are you still interested in this?

Comment 4 Mathieu Bridon 2015-05-15 09:04:21 UTC
I'm not interested any more, no.

Sorry for not letting you know earlier, I hope I didn't cause any work for you. :(