[Bug 657591] Review Request: rubygem-isolate - Very simple RubyGems sandbox
bugzilla at redhat.com
bugzilla at redhat.com
Fri Nov 26 21:34:57 UTC 2010
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=657591
Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
--- Comment #3 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> 2010-11-26 16:34:57 EST ---
(In reply to comment #2)
> Package:
> * When requiring 'isolate/rake' (via README):
- Well, lib/isolate/rake.rb is perhaps a "plugin" for rake,
and actually:
--------------------------------------------------------
[tasaka1 at localhost ~]$ ruby -rubygems -e 'require "isolate/rake"'
/usr/lib/ruby/gems/1.8/gems/isolate-3.0.0/lib/isolate/rake.rb:1: undefined
method `namespace' for main:Object (NoMethodError)
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in
`gem_original_require'
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in `require'
from -e:1
[tasaka1 at localhost ~]$ ruby -rubygems -e 'require "rake" ; require
"isolate/rake"'
[tasaka1 at localhost ~]$
--------------------------------------------------------
So maybe it is better that lib/isolate/rake.rb contains "require 'rake'"
at the top, however I guess this is okay for now.
> Otherwise - looks good!
>
> ------------------------------------------------------------
> This package (rubygem-isolate) is APPROVED by goldmann
> ------------------------------------------------------------
Thank you!
New Package SCM Request
=======================
Package Name: rubygem-isolate
Short Description: Very simple RubyGems sandbox
Owners: mtasaka
Branches: f13 f14
--
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