Clojure JIRA http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 [CLASSPATH-2] Reported classpath should include classpath from parent classloaders, but not from the system classloader http://dev.clojure.org/jira/browse/CLASSPATH-2 java.classpath <p>Since a classloader delegates to its parent if it fails to provide a class, the classpath reported by java.classpath should include the classpath of parent classloaders.</p> <p>At present the reported classpath contains both system and baseloader classpaths, which I believe is incorrect, since the system classloader is not necessarily a parent of the baseloader.</p> CLASSPATH-2 Reported classpath should include classpath from parent classloaders, but not from the system classloader Defect Major Resolved Completed Stuart Sierra Hugo Duncan Tue, 13 Sep 2011 20:35:55 -0500 Thu, 15 Sep 2011 12:53:56 -0500 Thu, 15 Sep 2011 12:53:55 -0500 0 0 <p>If this patch is accepted, it will fix the issue described in <a href="http://dev.clojure.org/jira/browse/CLASSPATH-1">http://dev.clojure.org/jira/browse/CLASSPATH-1</a>. Hugo and I talked and his fix is more general than mine, so you can close <a href="http://dev.clojure.org/jira/browse/CLASSPATH-1" title="clojure.java.classpath/system-classpath is incorrect when running in a classloader other than the application classloader"><del>CLASSPATH-1</del></a>.</p> <p>Patch does not compile.</p> <p>Oops, I ran the tests, but failed to notice there weren't any...</p> <p>Updated patch, with a unit test.</p> <p>Updated patch with unit test</p> <p>Patch is good. Not sure about the test. Finding "clojure-.*jar" in the classpath is subject to the build environment. I think all we can reasonably expect is that the classpath is a seq of java.io.Files.</p> <p>The test are normally run from the project's own pom, and even if run from an external project, the classpath will include a clojure jar in the huge majority of cases. I suppose the tests could be guarded with a java property set in the project's pom.</p> <p>If the tests for the clojure jar are dropped, a test for a non-empty sequence should at least be added.</p> <p>Agreed: the seq should be non-empty and contain Files.</p> <p>Patch with unit test for classpath, testing for non-empty sequence of java.io.File objects as return value.</p> <p>Patch applied. Will release new version soon.</p> Global Rank Patch Code