Bug 842328
Summary: | Review Request: rubygem-jquery-rails - Use jQuery with Rails 3 | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Bohuslav "Slavek" Kabrda <bkabrda> |
Component: | Package Review | Assignee: | Vít Ondruch <vondruch> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | notting, package-review, vondruch |
Target Milestone: | --- | Flags: | vondruch:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2012-07-24 14:10:35 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Bohuslav "Slavek" Kabrda
2012-07-23 13:46:15 UTC
I'll take it for a review. * More specific license - Although it seems that the license field is correct, could you please elaborate a bit more what license is for what library? * Smoke test failed? - I did just small smoketest and it failed. I believe that it depends on Rails application configuration, but anyway, could you please doublecheck? $ irb reqirb(main):001:0> require 'jquery-rails' NameError: uninitialized constant Rails from /usr/share/gems/gems/jquery-rails-2.0.2/lib/jquery/rails/engine.rb:1:in `<top (required)>' from /usr/share/rubygems/rubygems/custom_require.rb:36:in `require' from /usr/share/rubygems/rubygems/custom_require.rb:36:in `require' from /usr/share/gems/gems/jquery-rails-2.0.2/lib/jquery/rails.rb:1:in `<top (required)>' from /usr/share/rubygems/rubygems/custom_require.rb:36:in `require' from /usr/share/rubygems/rubygems/custom_require.rb:36:in `require' from /usr/share/gems/gems/jquery-rails-2.0.2/lib/jquery-rails.rb:1:in `<top (required)>' from /usr/share/rubygems/rubygems/custom_require.rb:60:in `require' from /usr/share/rubygems/rubygems/custom_require.rb:60:in `rescue in require' from /usr/share/rubygems/rubygems/custom_require.rb:35:in `require' from (irb):1 from /usr/bin/irb:12:in `<main>' Otherwise I see no issue => APPROVED. But please, make sure that you understand the above mentioned error and better describe the licenses before import. Thanks for the review! Yes, I'll be more verbose about the licenses. jquery-rails depends on rails being required before, which is what is done using bundler. This is the expected behaviour, but good catch anyway :) New Package SCM Request ======================= Package Name: rubygem-jquery-rails Short Description: Use jQuery with Rails 3 Owners: bkabrda Branches: InitialCC: Git done (by process-git-requests). |