Bug 771295

Summary: Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now
Product: [Fedora] Fedora Reporter: Bohuslav "Slavek" Kabrda <bkabrda>
Component: Package ReviewAssignee: Vít Ondruch <vondruch>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: notting, package-review, vondruch
Target Milestone: ---Flags: vondruch: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-01-03 14:20:57 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Bohuslav "Slavek" Kabrda 2012-01-03 07:35:44 UTC 
Delorean lets you travel in time with Ruby by mocking Time.now.

SPEC: http://bkabrda.fedorapeople.org/pkgs/delorean/rubygem-delorean.spec
SRPM: http://bkabrda.fedorapeople.org/pkgs/delorean/rubygem-delorean-1.2.0-1.fc17.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3615002
Comment 1 Vít Ondruch 2012-01-03 12:06:05 UTC 
I'll take it for a review.
Comment 2 Vít Ondruch 2012-01-03 12:55:25 UTC 
* Test suite
  - Please consider execution of the test suite in the %check section? It seems
    there are specs available upstream.

* rpmlint output
  - rpmlint complains about permissions of .spec file:

    rubygem-delorean.src: W: strange-permission rubygem-delorean.spec 0666L
    A file that you listed to include in your package has strange permissions.
    Usually, a file should have 0644 permissions.

Otherwise, I have no other comments, so I APPROVE the package. Please fix the rpmlint before import of the package and consider execution of the test suite.
Comment 3 Bohuslav "Slavek" Kabrda 2012-01-03 13:05:08 UTC 
Thank you for your review. I'll fix these things before making a SCM request.
Comment 4 Bohuslav "Slavek" Kabrda 2012-01-03 13:41:21 UTC 
New Package SCM Request
=======================
Package Name: rubygem-delorean
Short Description: Delorean lets you travel in time with Ruby by mocking Time.now
Owners: bkabrda
Branches: 
InitialCC:
Comment 5 Gwyn Ciesla 2012-01-03 13:49:05 UTC 
Git done (by process-git-requests).