[Bug 543337] Review Request: rubygem-rcov - Code coverage analysis tool for Ruby
bugzilla at redhat.com
bugzilla at redhat.com
Wed Dec 2 16:17:52 UTC 2009
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=543337
Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
AssignedTo|nobody at fedoraproject.org |mtasaka at ioa.s.u-tokyo.ac.jp
Flag| |fedora-review?
--- Comment #1 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> 2009-12-02 11:17:48 EDT ---
Some notes:
? Requires
- Would you explain the necessity of "R: rubygem(rake)"?
( Maybe for lib/rcov/rcovtask.rb ? If so, I wonder if this
script is used by default or not because no other scripts
in rcov gem seems to require this script )
! Note that I am not objecting to adding "R: rubygem(rake)".
* ext/
- Files under ext/ are to generate C extension module
(rcovrt.so) and should not be needed to be included into
binary rpm.
( The trouble is that once we try to execute "rake check"
under %geminstdir, rake task tries to regenerate .so file.
In such cases files under ext/ are needed, however anyway
this won't work but for root because write permission for
%geminstdir is needed.
So while you may want to add files under ext to -doc
subpackage, these files should not be in main package. )
--
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 Fedora-package-review
mailing list