Bug 786808 - Review Request: rubygem-ammeter - Write specs for your Rails 3+ generators
Summary: Review Request: rubygem-ammeter - Write specs for your Rails 3+ generators
Keywords:
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:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-02 12:59 UTC by Bohuslav "Slavek" Kabrda
Modified: 2012-02-15 07:18 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-15 07:18:40 UTC
Type: ---
Embargoed:
vondruch: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Comment 1 Vít Ondruch 2012-02-02 13:57:39 UTC
I'll take it for a review since I need it for rubygem-rspec-rails.

Comment 2 Vít Ondruch 2012-02-02 16:13:52 UTC
* Move features into -doc
 - If I am not mistaken, the '%{gem_instdir}/features' folder should be moved
   into -doc, since it is cucumber test suite.

* Please consider moving of Gemfile and .gemspec into -doc
  - Not sure if it is not better to preserve such files, although there are
    no guidelines for such files.

* rpmlint

  rubygem-ammeter.noarch: W: hidden-file-or-dir /usr/share/gems/gems/ammeter-
      0.2.2/lib/.DS_Store

  - Please remove the specified file. Is is OS X specific file, not needed for
    Fedora

* Please fix the .gemspec file.
  - The gemspec needs to be update (see rspec-rails gem).
  - When requiring the library, following error appears:

Gem::LoadError: Could not find rspec (>= 2.2) amongst [abstract-1.0.0, actionpack-3.0.11, activemodel-3.0.11, activesupport-3.0.11, ammeter-0.2.2, bigdecimal-1.1.0, builder-2.1.2, diff-lcs-1.1.2, erubis-2.6.6, i18n-0.5.0, io-console-0.3, minitest-2.10.1, rack-1.4.0, rack-mount-0.7.1, rack-test-0.6.0, railties-3.0.11, rake-0.9.2.2, rdoc-3.9.4, regin-0.3.8, rspec-core-2.8.0, rspec-expectations-2.8.0, rspec-mocks-2.8.0, rspec-rails-2.8.1, ruby2ruby-1.2.4, ruby_parser-2.0.4, sexp_processor-3.0.4, thor-0.14.6, tzinfo-0.3.30]
	from /usr/share/rubygems/rubygems/dependency.rb:247:in `to_specs'
	from /usr/share/rubygems/rubygems/specification.rb:771:in `block in activate_dependencies'
	from /usr/share/rubygems/rubygems/specification.rb:760:in `each'
	from /usr/share/rubygems/rubygems/specification.rb:760:in `activate_dependencies'
	from /usr/share/rubygems/rubygems/specification.rb:744:in `activate'
	from /usr/share/rubygems/rubygems.rb:209:in `rescue in try_activate'
	from /usr/share/rubygems/rubygems.rb:206:in `try_activate'
	from /usr/share/rubygems/rubygems/custom_require.rb:58: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>'

Comment 3 Bohuslav "Slavek" Kabrda 2012-02-03 06:51:42 UTC
(In reply to comment #2)
> * Move features into -doc
>  - If I am not mistaken, the '%{gem_instdir}/features' folder should be moved
>    into -doc, since it is cucumber test suite.
> 

You are right, done.

> * Please consider moving of Gemfile and .gemspec into -doc
>   - Not sure if it is not better to preserve such files, although there are
>     no guidelines for such files.
> 

Done,

> * rpmlint
> 
>   rubygem-ammeter.noarch: W: hidden-file-or-dir /usr/share/gems/gems/ammeter-
>       0.2.2/lib/.DS_Store
> 
>   - Please remove the specified file. Is is OS X specific file, not needed for
>     Fedora
> 

Done.

> * Please fix the .gemspec file.
>   - The gemspec needs to be update (see rspec-rails gem).
>   - When requiring the library, following error appears:
> 
> Gem::LoadError: Could not find rspec (>= 2.2) amongst [abstract-1.0.0,
> actionpack-3.0.11, activemodel-3.0.11, activesupport-3.0.11, ammeter-0.2.2,
> bigdecimal-1.1.0, builder-2.1.2, diff-lcs-1.1.2, erubis-2.6.6, i18n-0.5.0,
> io-console-0.3, minitest-2.10.1, rack-1.4.0, rack-mount-0.7.1, rack-test-0.6.0,
> railties-3.0.11, rake-0.9.2.2, rdoc-3.9.4, regin-0.3.8, rspec-core-2.8.0,
> rspec-expectations-2.8.0, rspec-mocks-2.8.0, rspec-rails-2.8.1,
> ruby2ruby-1.2.4, ruby_parser-2.0.4, sexp_processor-3.0.4, thor-0.14.6,
> tzinfo-0.3.30]
>  from /usr/share/rubygems/rubygems/dependency.rb:247:in `to_specs'
>  from /usr/share/rubygems/rubygems/specification.rb:771:in `block in
> activate_dependencies'
>  from /usr/share/rubygems/rubygems/specification.rb:760:in `each'
>  from /usr/share/rubygems/rubygems/specification.rb:760:in
> `activate_dependencies'
>  from /usr/share/rubygems/rubygems/specification.rb:744:in `activate'
>  from /usr/share/rubygems/rubygems.rb:209:in `rescue in try_activate'
>  from /usr/share/rubygems/rubygems.rb:206:in `try_activate'
>  from /usr/share/rubygems/rubygems/custom_require.rb:58: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>'

Fixed.

SPEC: http://bkabrda.fedorapeople.org/pkgs/ammeter/rubygem-ammeter.spec
SRPM: http://bkabrda.fedorapeople.org/pkgs/ammeter/rubygem-ammeter-0.2.2-2.fc16.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3758401

Comment 4 Vít Ondruch 2012-02-03 14:26:44 UTC
I can't see any other obstacles => APPROVED.

Comment 5 Bohuslav "Slavek" Kabrda 2012-02-06 06:45:57 UTC
Thank you for your review!


New Package SCM Request
=======================
Package Name: rubygem-ammeter
Short Description: Write specs for your Rails 3+ generators
Owners: bkabrda
Branches: 
InitialCC:

Comment 6 Gwyn Ciesla 2012-02-06 13:22:52 UTC
Git done (by process-git-requests).


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