Bug 478942 - Review Request: perl-Catalyst-Authentication-Store-DBIx-Class - A storage class for Catalyst Authentication
Summary: Review Request: perl-Catalyst-Authentication-Store-DBIx-Class - A storage cla...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: manuel wolfshant
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 478940
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-01-06 05:32 UTC by Gabriel Somlo
Modified: 2009-01-15 03:10 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-01-15 02:55:01 UTC
Type: ---
Embargoed:
manuel.wolfshant: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Gabriel Somlo 2009-01-06 05:32:39 UTC
Spec URL: http://mirror.net.cmu.edu/gls/10/spec/perl-Catalyst-Authentication-Store-DBIx-Class.spec
SRPM URL: http://mirror.net.cmu.edu/gls/10/source/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-1.fc10.src.rpm
Description:
The Catalyst::Authentication::Store::DBIx::Class class provides access
to authentication information stored in a database via DBIx::Class.

Comment 1 manuel wolfshant 2009-01-06 21:16:22 UTC
Once again, pretty much standard. Most noticeable:
rpmlint of perl-Catalyst-Authentication-Store-DBIx-Class.src: W: filename-too-long-for-joliet perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-1.fc10.src.rpm
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

rpmlint of perl-Catalyst-Authentication-Store-DBIx-Class:
perl-Catalyst-Authentication-Store-DBIx-Class.noarch: W: filename-too-long-for-joliet perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-1.fc11.noarch.rpm
=> Both can be ignored but keep in mind that the package will not be able to be included in a standard CD. Source rpm name has 68 chars, binary has 71 chars and both exceed the limit of 64 specified for Joliet
sha1sum: ea63d54c6c5a5a1224d2913a795c68b537ea3733 Catalyst-Authentication-Store-DBIx-Class-0.1082.tar.gz
make test is OK.
License:
- same as perl for User.pm and Class.pm
- not specified for SimpleDB.pm


Could you please try to clarify with the author the situation of SimpleDB.pm ? Most probably it has the same license as the rest of the code (and as specified in the README) but a written proof would be better.

Comment 2 Gabriel Somlo 2009-01-08 19:34:35 UTC
http://mirror.net.cmu.edu/gls/10/spec/perl-Catalyst-Authentication-Store-DBIx-Class.spec

http://mirror.net.cmu.edu/gls/10/source/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc10.src.rpm

These now contain an extra file, 'FedoraLicense.txt' which is an email reply
from the maintainer clarifying the license on SimpleDB.pm and promising to
include a blurb within the file itself.

Comment 3 manuel wolfshant 2009-01-08 22:56:18 UTC
[wolfy@wolfy2 tmp]$ wget http://mirror.net.cmu.edu/gls/10/source/perl-Catalyst-Authent                        ication-Store-DBIx-Class-0.1082-2.fc10.src.rpm
--00:55:45--  http://mirror.net.cmu.edu/gls/10/source/perl-Catalyst-Authentication-Sto                        re-DBIx-Class-0.1082-2.fc10.src.rpm
Resolving mirror.net.cmu.edu... 128.2.4.9
Connecting to mirror.net.cmu.edu|128.2.4.9|:80... connected.
HTTP request sent, awaiting response... 403 Forbidden
00:55:45 ERROR 403: Forbidden.

chmod +r ?

Comment 4 Gabriel Somlo 2009-01-09 15:06:23 UTC
> chmod +r ?

nope, chcon something-something :)

Didn't realize mv-ing something into /var/www/blah instead of cp-ing it would annoy selinux, now I know :)

If you try again now, it should work.

Sorry for the delay
--Gabriel

Comment 5 manuel wolfshant 2009-01-10 01:15:26 UTC
Excellent. APPROVED

Comment 6 Gabriel Somlo 2009-01-10 23:04:48 UTC
New Package CVS Request
=======================
Package Name: perl-Catalyst-Authentication-Store-DBIx-Class
Short Description: A storage class for Catalyst Authentication
Owners: somlo
Branches: F-9 F-10
InitialCC:

Comment 7 Kevin Fenzi 2009-01-11 17:08:42 UTC
cvs done.

Comment 8 Fedora Update System 2009-01-12 19:33:58 UTC
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc9

Comment 9 Fedora Update System 2009-01-12 19:34:07 UTC
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc10

Comment 10 Fedora Update System 2009-01-15 02:54:58 UTC
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-01-15 03:10:36 UTC
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.


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