Peter, sounds reasonable. ad 1) I've not much experience with non-Sun jvms/jres. However, referring sun.* classes in a task might not be a good practice. But isn't moving report-function to the optional category just a cosmetic operation? What do you think about moving the functionality "getBootstrapClasspathUrls" into JavaEnvUtils and do it there via reflection. We can also add tests for the both classes in JavaEnvUtils.getJrePackageTestCases. Or is there a better location for this functionality?
ad 2) I've also made this experience. Some times ago I've had a version with this feature but in the present version i abandoned it to make things easier. However, this behaviour should be switchable as there a imaginable cases where it is not desired. ad 3) Absolutely. (Something, I've never thought about.) I'm working on it but it may take a while (I've started a new project/contract yesterday). Cheers Rainer > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > Sent: Thursday, June 02, 2005 4:08 PM > To: dev@ant.apache.org > Subject: DO NOT REPLY [Bug 28228] - <classloader> task (snip) > > ------- Additional Comments From [EMAIL PROTECTED] > 2005-06-02 16:08 ------- I am looking at this again. It would > be nice to make a number of changes: > > 1) the task import sun.import sun.misc.Launcher and import > sun.misc.URLClassPath. > It would be better not to import sun.* classes in > non-optional tasks. A > solution to this would be make the report function a > seperate optional task. > > 2) the task could check if a jar or directort is already in a > in the classpath > I found when testing that it is easy to get into a > situation where the > classpath will have the same jar file many times - due to calls to > <classloader> with using netbeans > > 3) the doc could state that use of this task in ides that do not > have a separate jvm for a run of ant will cause issues - the > classpath will be retained by the ide and used in the next > ant build run > > > -- > Configure bugmail: > http://issues.apache.org/bugzilla/userprefs.cgi?tab=email > ------- You are receiving this mail because: ------- > You are the assignee for the bug, or are watching the assignee. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
