diff --git a/build.xml b/build.xml index 5c532d3..5865aa5 100644 --- a/build.xml +++ b/build.xml @@ -578,6 +578,9 @@ limitations under the License.-->'> + + + @@ -590,6 +593,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.jdk14.CustomConfigTestCase"/> + + + @@ -602,6 +608,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.jdk14.CustomConfigTestCase"/> + + + @@ -612,6 +621,9 @@ limitations under the License.-->'> + + + @@ -624,6 +636,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.jdk14.CustomConfigTestCase"/> + + + @@ -636,6 +651,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.jdk14.CustomConfigTestCase"/> + + + @@ -705,6 +723,9 @@ limitations under the License.-->'> + + + @@ -718,6 +739,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -731,6 +755,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -742,6 +769,9 @@ limitations under the License.-->'> + + + @@ -755,6 +785,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -768,6 +801,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -895,6 +931,9 @@ limitations under the License.-->'> + + + @@ -908,6 +947,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -921,6 +963,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -932,6 +977,9 @@ limitations under the License.-->'> + + + @@ -945,6 +993,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + @@ -958,6 +1009,9 @@ limitations under the License.-->'> value="org.apache.commons.logging.log4j.CustomConfigTestCase"/> + + + diff --git a/src/test/org/apache/commons/logging/Wrapper.java b/src/test/org/apache/commons/logging/Wrapper.java index 9818271..11521b9 100644 --- a/src/test/org/apache/commons/logging/Wrapper.java +++ b/src/test/org/apache/commons/logging/Wrapper.java @@ -82,10 +82,22 @@ public class Wrapper { // Construct URLs for the various JAR files File target = new File(System.getProperty("wrapper.target")); - URL commonsLogging = - (new File(target, "commons-logging.jar")).toURL(); - URL commonsLoggingApi = - (new File(target, "commons-logging-api.jar")).toURL(); + + // the test invoker is required to set these system properties + // to point to the appropriate jar files + String jcljar = System.getProperty("commons.logging.jar"); + String jclapijar = System.getProperty("commons.logging.api.jar"); + String jclappjar = System.getProperty("commons.logging.appenders.jar"); + if ((jcljar==null) || (jclapijar==null) || (jclappjar==null)) { + throw new Exception( + "Required system property for test is missing. " + + "One or more of the following is not defined: " + + "commons.logging.jar, commons.logging.api.jar, " + + "commons.logging.appenders.jar"); + } + + URL commonsLogging = (new File(target, jcljar)).toURL(); + URL commonsLoggingApi = (new File(target, jclapijar)).toURL(); URL commonsLoggingTests = (new File(target, "commons-logging-tests.jar")).toURL(); URL junit = @@ -96,9 +108,12 @@ public class Wrapper { log4j = (new File(System.getProperty("wrapper.log4j"))).toURL(); appender = - (new File(target, "commons-logging-appender.jar")).toURL(); + (new File(target, jclappjar)).toURL(); } + System.out.println("commonsLogging lib is at [" + commonsLogging + "]"); + System.out.println("commonsLoggingApi lib is at [" + commonsLoggingApi + "]"); + // Construct class loader repository lists for supported scenarios if ("API".equals(System.getProperty("wrapper.hierarchy"))) { parentList.add(commonsLoggingApi);