Spec URL: http://bkabrda.fedorapeople.org/pkgs/jquery-rails/rubygem-jquery-rails.spec SRPM URL: http://bkabrda.fedorapeople.org/pkgs/jquery-rails/rubygem-jquery-rails-2.0.2-1.fc17.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4323052 Description: This gem provides jQuery and the jQuery-ujs driver for your Rails 3 application. Fedora Account System Username: bkabrda
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).