Bug 1186725 - Review Request: perl-Apache-Session-NoSQL - NoSQL implementation of Apache::Session
Summary: Review Request: perl-Apache-Session-NoSQL - NoSQL implementation of Apache::S...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Dick
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1181654
TreeView+ depends on / blocked
 
Reported: 2015-01-28 12:08 UTC by Paul Howarth
Modified: 2015-02-18 08:57 UTC (History)
2 users (show)

Fixed In Version: perl-Apache-Session-NoSQL-0.1-2.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-18 08:57:26 UTC
Type: ---
Embargoed:
ddick: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Paul Howarth 2015-01-28 12:08:37 UTC
Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Apache-Session-NoSQL/branches/fedora/perl-Apache-Session-NoSQL.spec
SRPM URL: http://www.city-fan.org/~paul/extras/perl-Apache-Session-NoSQL/perl-Apache-Session-NoSQL-0.1-1.fc22.src.rpm

Description:
NoSQL implementation of Apache::Session. Sessions are stored in NoSQL
bases, either Redis or Cassandra.

Fedora Account System Username: pghmcfc

This package contains old spec file elements for compatibility with EPEL-5, even though an EPEL-5 build is not currently possible due to missing driver back-ends. Either Redis or Net::Cassandra is required. I have split the package up such that it requires at least one driver back-end to be present, and appropriate dependencies are in place for each driver but the main package does not require any back-end driver that is not needed.

Net::Cassandra is not currently available in Fedora/EPEL but I have added support for it (off by default) such that it can easily be enabled if and when someone packages it.

Comment 1 David Dick 2015-01-29 20:04:24 UTC
License is correct

Package builds in rawhide http://koji.fedoraproject.org/koji/taskinfo?taskID=8773114

rpmlint is clean

perl(base) should be put in as a BuildRequire for all three packages

you should put in the Change, README and examples doc files.

isn't it worth building Net-Cassandra in rawhide at least?

if you wanted to build Net-Cassandra in the EPEL branches, for EPEL6 and 7 at least it looks like the only thing stopping you is perl(MooseX::StrictConstructor) which builds cleanly in EPEL6 and 7.

http://koji.fedoraproject.org/koji/taskinfo?taskID=8773086
http://koji.fedoraproject.org/koji/taskinfo?taskID=8773093

Comment 2 Paul Howarth 2015-01-29 20:38:24 UTC
(In reply to David Dick from comment #1)
> License is correct
> 
> Package builds in rawhide
> http://koji.fedoraproject.org/koji/taskinfo?taskID=8773114
> 
> rpmlint is clean
> 
> perl(base) should be put in as a BuildRequire for all three packages
> 
> you should put in the Change, README and examples doc files.

Fixed in -2:

Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Apache-Session-NoSQL/branches/fedora/perl-Apache-Session-NoSQL.spec
SRPM URL: http://www.city-fan.org/~paul/extras/perl-Apache-Session-NoSQL/perl-Apache-Session-NoSQL-0.1-2.fc22.src.rpm

> isn't it worth building Net-Cassandra in rawhide at least?

I'd be delighted if someone did (and once it was in Rawhide, it could probably go to F-20, F-21 and EPEL too). I'm just not sure that I'm the right person to maintain it as I've no interest in it at all personally, and it's a big package that might need a fair bit of maintenance down the line.

Comment 3 David Dick 2015-01-30 21:13:02 UTC
Yeah, fair enough.  I can't say i have a lot of interest in it either. :)

Package APPROVED!

Comment 4 Paul Howarth 2015-01-31 13:46:59 UTC
New Package SCM Request
=======================
Package Name: perl-Apache-Session-NoSQL
Short Description: NoSQL implementation of Apache::Session
Upstream URL: http://search.cpan.org/dist/Apache-Session-NoSQL/
Owners: pghmcfc
Branches: f20 f21 el6 epel7
InitialCC: perl-sig

Thanks for the review David.

Comment 5 Gwyn Ciesla 2015-02-01 15:44:26 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2015-02-01 19:21:27 UTC
perl-Apache-Session-NoSQL-0.1-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/perl-Apache-Session-NoSQL-0.1-2.el6

Comment 7 Fedora Update System 2015-02-01 19:21:35 UTC
perl-Apache-Session-NoSQL-0.1-2.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/perl-Apache-Session-NoSQL-0.1-2.el7

Comment 8 Fedora Update System 2015-02-01 19:21:41 UTC
perl-Apache-Session-NoSQL-0.1-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Apache-Session-NoSQL-0.1-2.fc20

Comment 9 Fedora Update System 2015-02-01 19:21:48 UTC
perl-Apache-Session-NoSQL-0.1-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/perl-Apache-Session-NoSQL-0.1-2.fc21

Comment 10 Fedora Update System 2015-02-15 03:07:50 UTC
perl-Apache-Session-NoSQL-0.1-2.fc21 has been pushed to the Fedora 21 stable repository.

Comment 11 Fedora Update System 2015-02-15 03:08:00 UTC
perl-Apache-Session-NoSQL-0.1-2.fc20 has been pushed to the Fedora 20 stable repository.

Comment 12 Fedora Update System 2015-02-17 18:58:18 UTC
perl-Apache-Session-NoSQL-0.1-2.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 13 Fedora Update System 2015-02-17 19:01:32 UTC
perl-Apache-Session-NoSQL-0.1-2.el7 has been pushed to the Fedora EPEL 7 stable repository.


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