Re: [jruby-dev] One line patch for jruby-debug

Chris Nelson wrote:
> DOH!  Attached the wrong patch... trying again.  This one is necessary 
> for jruby-debug to compile, unless I'm missing something.
> [...]
>
> Index: /home/ccnelson/workspace/jruby-svn/src/org/jruby/RubyThread.java
> ===================================================================
> --- /home/ccnelson/workspace/jruby-svn/src/org/jruby/RubyThread.java    
> (revision 4409)
> +++ /home/ccnelson/workspace/jruby-svn/src/org/jruby/RubyThread.java    
> (working copy)
> @@ -267,7 +267,7 @@
>          }
>      }
>  
> -    private RubyThread(Ruby runtime, RubyClass type) {
> +    protected RubyThread(Ruby runtime, RubyClass type) {
>          super(runtime, type);
>          this.threadService = runtime.getThreadService();
>          // set to default thread group
Yes this one is needed for us. But I would wait with sending patches to 
JRuby. Currently jruby-debug is very raw rewrite from native ruby-debug. 
Is very likely that I missed something somewhere. So I would wait until 
we get a little bit further with jruby-debug. For now we two can live 
with patching the JRuby on our machines. Note that I've added the note 
about this patch to the jruby-debug's README few days ago:
http://debug-commons.rubyforge.org/svn/jruby-debug/trunk/README
http://debug-commons.rubyforge.org/svn/jruby-debug/trunk/jruby-patch.diff

Regards,
        m.



---------------------------------------------------------------------
To unsubscribe from this list please visit:

   http://xircles.codehaus.org/manage_email