Thank you for your explanation, Assaf. I still find the whole rjb/jruby business tricky to understand, but I'm not going to complain since it's one of the feature I like a lot in buildr. :-)
Lacton On Tue, Sep 23, 2008 at 11:23 PM, Assaf Arkin <[EMAIL PROTECTED]> wrote: > On Tue, Sep 23, 2008 at 1:52 PM, lacton <[EMAIL PROTECTED]> wrote: >> Assaf, >> >> Will you tell me what was wrong with the way I tried to factor common >> code between commands.rb, jruby.rb and rjb.rb? > > Because I prefer to tightly couple the tools_jar and load methods of > each module, then to tightly couple the modules together. > > Generally when I troubleshoot code, I'm either working against rjb.rb > or jruby.rb, depending on which platform I'm using. It's much more > convenient to have all the code in one place, then spread it out. > > And it hides a distinction that exists between these modules: RJB's > tool_jars method brings with it a dependency on setting JAVA_HOME, > JRuby doesn't. By having two separate methods to begin with, it's > easier when looking at rjb.rb to notice that JAVA_HOME is only > referenced once, and only by tools_jar, and to move it there from the > load method, where it was before. > > Assaf