Bug 1076516 - Review Request: rubygem-spring - Rails application preloader
Summary: Review Request: rubygem-spring - Rails application preloader
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Vít Ondruch
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-14 13:32 UTC by Josef Stribny
Modified: 2016-01-04 05:52 UTC (History)
3 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2014-03-18 13:57:23 UTC
vondruch: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Josef Stribny 2014-03-14 13:32:38 UTC
Spec URL: http://data-strzibny.rhcloud.com/rubygem-spring.spec
SRPM URL: http://kojipkgs.fedoraproject.org//work/tasks/2580/6632580/rubygem-spring-1.1.2-1.fc21.src.rpm
Description: Spring is a Rails application preloader. It speeds up development by keeping your application running in the background so you don't need to boot it every time you run a test, rake task or migration.
Fedora Account System Username: jstribny

I am packaging rubygem-spring since Rails 4.1 suggests using spring in development and includes it in default Gemfile.

Comment 1 Vít Ondruch 2014-03-17 16:14:12 UTC
I'll take this for a review.

Comment 2 Vít Ondruch 2014-03-18 12:02:03 UTC
Just from the beginning, please do not link the SRPM from Koji, since it will get garbage collected soon, i.e. the link will become invalid.

* Bundled OkJson library
  - This library has bundling exception, but you should mark it properly via
    virtual provide [1]

* Test suite
  - I failed to execute the test suite as well. Listen is only one piece, but
    there is also extensively used Bundler, which might bring another issues.
  - However, could you fix the comment? rubygem-listen actually *is* in Fedora,
    but in old version. Unfortunately looking in direction in which Listen
    developers go, I am not sure it will be ever updated. I wouldn't be
    surprised that this might be the reason why spring does not depend on recent
    version if listen.

* Missing 'ps' dependency
  - It seems that '$ spring status' requires 'ps' command to work:

    /usr/share/gems/gems/spring-1.1.2/lib/spring/client/status.rb:20:in ``':
      No such file or directory - ps (Errno::ENOENT)

  - Please comment the require appropriately, in case the 'ps' requirement is
    dropped from code base one day.

* Hidden file
  - There is shipped unnecessary hidden file:

    rubygem-spring-doc.noarch: E: version-control-internal-file
      /usr/share/gems/gems/spring-1.1.2/test/apps/.gitignore

Otherwise the package looks sand => APPROVED

Please fix the OkJson and 'ps' dependency prior import.


[1] https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Packages_granted_exceptions

Comment 3 Josef Stribny 2014-03-18 12:46:29 UTC
Thank you. Will do.

New Package SCM Request
=======================
Package Name: rubygem-spring
Short Description: Rails application preloader
Owners: jstribny
Branches:
InitialCC:

Comment 4 Gwyn Ciesla 2014-03-18 12:55:53 UTC
Git done (by process-git-requests).


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