[Bug 743007] Review Request: rubygem-introspection - Dynamic inspection of the hierarchy of method definitions on a Ruby object
bugzilla at redhat.com
bugzilla at redhat.com
Tue Oct 4 09:48:07 UTC 2011
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=743007
Bohuslav Kabrda <bkabrda at redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-review+
--- Comment #6 from Bohuslav Kabrda <bkabrda at redhat.com> 2011-10-04 05:48:06 EDT ---
(In reply to comment #5)
> (In reply to comment #4)
> > * %{geminstdir}/README.md should be marked as %doc.
>
> Moved into -doc subpackage and marked as %doc
>
> > * The way you run tests doesn't actually run them (0 tests, 0 assertions, 0
> > failures, 0 errors). When you run them this way: 'testrb -Ilib
> > ./test/*_test.rb', you can discover, that you need to include BR:
> > rubygem(metaclass) and BR: rubygem(builder) (the second one to satisfy the
> > require 'blankslate' in snapshot_test.rb). Please clarify these BRs.
>
> Good catch!
Great, the tests are run now.
>
> > * Consider including license file from the upstream in the package (maybe as a
> > separate source).
>
> I'll include the license file as soon as it will be available in official
> release. In the meantime, I have added comment into the .spec file. It is not
> hard requirement, so it should not be blocker.
>
Sounds fine.
> > * Could you explain if there currently is a particular reason for R:
> > rubygem-metaclass < 0.1?
>
> It follows the upstream suggested dependency, i.e. the original ~> 0.0.1
> dependency transform into > 0.0.1 and <= 0.1, see [1]
>
Yes, sorry about that.
>
> Spec URL: http://people.redhat.com/vondruch/rubygem-introspection.spec
> SRPM URL:
> http://people.redhat.com/vondruch/rubygem-introspection-0.0.2-3.fcf17.src.rpm
>
>
>
> [1] http://docs.rubygems.org/read/chapter/16
Package seems OK now, APPROVED.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the package-review
mailing list