Bug 785663 - Review Request: rubygem-journey - Journey is a router
Summary: Review Request: rubygem-journey - Journey is a router
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bohuslav "Slavek" Kabrda
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-30 09:00 UTC by Vít Ondruch
Modified: 2012-01-30 14:14 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-30 14:14:24 UTC
Type: ---
bkabrda: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Comment 1 Bohuslav "Slavek" Kabrda 2012-01-30 09:15:53 UTC
I'll take this for a review.

Comment 2 Bohuslav "Slavek" Kabrda 2012-01-30 09:41:13 UTC
- README.md should be present in the main package, as it contains licensing information.
- Otherwise the package is OK (although, as FPC [1], I have a general feel of distaste about the bundled javascript libraries - but since they are currently allowed, it is not a blocker).

[1] https://fedorahosted.org/fpc/ticket/113


So after you fix the issue, this package is APPROVED.

Comment 3 Vít Ondruch 2012-01-30 09:55:47 UTC
(In reply to comment #2)
> - README.md should be present in the main package, as it contains licensing
> information.

Done

> - Otherwise the package is OK (although, as FPC [1], I have a general feel of
> distaste about the bundled javascript libraries - but since they are currently
> allowed, it is not a blocker).
> 
> [1] https://fedorahosted.org/fpc/ticket/113
> 
> 
> So after you fix the issue, this package is APPROVED.


Thank you for your review!

New Package SCM Request
=======================
Package Name: rubygem-journey
Short Description: Journey is a router
Owners: vondruch
Branches: 
InitialCC:

Comment 4 Gwyn Ciesla 2012-01-30 13:31:24 UTC
Git done (by process-git-requests).


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