classloader for 1.7

On 8/30/06, Jesse Glick <sun.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">jesse.glicksun.com &gt; wrote:

Peter Reilly wrote:
>; So what would be the hier for the secondary class loader?
&gt; if it is bootstrap->ext->;system->;project-&gt;secondary,

Yes.

&gt; it would meant that the classes in project would not be able to see the
> secondary classes.

Correct. Why is this a problem? Ant core classes should not be referring
to tasks and such stuff.

Ok,
I have done a prototype and the results look good:

1) a new ComponentClassLoader which is a AntClassLoader, except that
 ;  a) it loads up with the classpath from Project.class.getClassLoader() and uses the paths
&nbsp; &nbsp; &nbsp;  from there to as its paths.
&nbsp;  b) it intercepts classes with "optional" or "ScriptRunner&quot; in the name and loads these from
 &nbsp; &nbsp;   itself rather than from the parent classloader.
  both of these changes are to allow ant's optional classes to be loaded for the component
  class loader rather than from Project.class.getClassLoader()
  c) set the parent classloader to Project.class.getClassLoader()

2) modify Project.java
 &nbsp; a) create a component classloader if is not created as a sub-project
 &nbsp; b) use this as a parent when Project.createClassLoader () is called
&nbsp;  c) set the component classloader of sub-projects
 &nbsp; d) provide a public accessor for the component classloader

3) modify ComponentHelper
 ;  a) use componentClassLoader when loading in tasks and types.

4) provide a task to add paths to the componentClassLoader

The end result looks like this:
<project name=";depend&quot; default=&quot;depend&quot;
 ; &nbsp; xmlns:ac=&quot;antlib:net.sf.antcontrib&quot;>
&nbsp; &nbsp; <target name=";depend&quot;>
&nbsp; &nbsp;
 &nbsp; &nbsp; &nbsp;  <!-- test for ant optional task - jdepend -->
&nbsp; &nbsp; &nbsp;   <appendcomponentpath>
&nbsp; &nbsp; &nbsp; &nbsp;   ;  <fileset dir="c:/apps/jdepend-2.9.1/lib"/&gt;
   ; &nbsp; &nbsp; </appendcomponentpath>
&nbsp; &nbsp; &nbsp;  
 &nbsp; &nbsp; &nbsp;  <jdepend>
 ; &nbsp; &nbsp; &nbsp; &nbsp;   <classespath path=";build/classes"/&gt;
 &nbsp;   ; &nbsp; </jdepend>
&nbsp; &nbsp; &nbsp; &nbsp;   ; 
 &nbsp; &nbsp; &nbsp;  <!-- test for ant lib -->
&nbsp; &nbsp;   ;  <appendcomponentpath>
&nbsp; &nbsp; &nbsp; &nbsp;   ;  <fileset dir="${user.home}/p" includes=&quot;ant-con*.jar";/>
&nbsp; &nbsp; &nbsp; &nbsp; </appendcomponentpath>

&nbsp; &nbsp;   ;  <ac:for param=&quot;param&quot; list=";a,b,c";>
&nbsp; &nbsp; &nbsp; &nbsp;   ;  <sequential>
&nbsp; &nbsp;   ; &nbsp; &nbsp; &nbsp;  <echo&gt;<;/echo>
  ; &nbsp; &nbsp; &nbsp; &nbsp;  </sequential>
&nbsp;   ; &nbsp;  </ac:for>
 ; &nbsp; &nbsp; &nbsp;
 &nbsp; &nbsp; &nbsp;  <!-- test for script -->
&nbsp; &nbsp; &nbsp;   <appendcomponentpath>
&nbsp; &nbsp; &nbsp;   ; &nbsp;  <fileset dir="${user.home}/lang/bsf&quot; includes=&quot;*.jar&quot;/>
  ; &nbsp; &nbsp; &nbsp; &nbsp;  <fileset dir="${user.home}/lang/beanshell" includes=&quot;*.jar&quot;/>
  ; &nbsp; &nbsp;  </appendcomponentpath>

&nbsp;   ; &nbsp;  <script language=&quot;beanshell"&gt;
 &nbsp; &nbsp;   ; &nbsp; &nbsp; System.out.println(&quot;Hello world";);
 &nbsp;   ; &nbsp; </script>
 ; &nbsp; </target>
</project>

A couple of notes:
&nbsp;  1) mixing using the append component loader with the .ant/lib ant $ANT_HOME/lib
 &nbsp;   ;  extensions may cause projects if the classes in .ant/lib load classes in the append component
  ; &nbsp; &nbsp; - so in the above example bsf.jar needs to be loaded with the beanshell.jar file, otherwise
  ; &nbsp; &nbsp; it will not see the beanshell classes. This can be avoided by:
   ; &nbsp; &nbsp;  a) adding more strings to intercept (fragile)
  ; &nbsp; &nbsp; &nbsp; b) getting AntLauncher to set the component class loader ( ant.jar + xml jars in project loader and the
   ; &nbsp; &nbsp; &nbsp; &nbsp; other jars in the component loader) : This would not be supported by the IDEs.
&nbsp; &nbsp; &nbsp; &nbsp;  c) using revert lookup for classes, except Project, Task etc.. (very fragile)

  2) the code is a bit like the coreloader code, but I do not use that because
&nbsp; &nbsp;   ; a) I could not get it to work
 ; &nbsp; &nbsp;  b) a public method is provided to set the coreloader.

Peter