Bug 724901 - Review Request: rubygem-rails_warden - A gem that provides authentication via the Warden framework
Summary: Review Request: rubygem-rails_warden - A gem that provides authentication via...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Marcela Mašláňová
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 705520
TreeView+ depends on / blocked
 
Reported: 2011-07-22 08:46 UTC by Vít Ondruch
Modified: 2011-09-20 14:55 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-22 10:28:44 UTC
Type: ---
Embargoed:
mmaslano: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Vít Ondruch 2011-07-22 08:46:40 UTC
Spec URL: http://people.redhat.com/vondruch/rubygem-rails_warden.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-rails_warden-0.5.5-1.fc16.src.rpm
Description: A gem that provides authentication via the Warden framework

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3222030

Comment 1 Marcela Mašláňová 2011-07-22 09:06:26 UTC
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

resolvedeps-f16 rubygem-rails_warden-0.5.5-1.fc16.src.rpm 
Binary dependencies resolvable. Ok.

APPROVED

Comment 2 Vít Ondruch 2011-07-22 09:47:44 UTC
Thank you for your review!



New Package SCM Request
=======================
Package Name: rubygem-rails_warden
Short Description: A gem that provides authentication via the Warden framework
Owners: vondruch
Branches: 
InitialCC:

Comment 3 Gwyn Ciesla 2011-07-22 10:09:23 UTC
Git done (by process-git-requests).

Comment 4 Vít Ondruch 2011-09-20 05:37:30 UTC
Package Change Request
======================
Package Name: rubygem-rails_warden
New Branches: f15
Owners: vondruch clalance
InitialCC:

Comment 5 Gwyn Ciesla 2011-09-20 14:55:16 UTC
Git done (by process-git-requests).


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