From: wisberg Date: Sat, 24 May 2003 11:12:42 +0000 (+0000) Subject: To support the new testing ICommand wrapper for AjcTask, permit ant.jar in a testing... X-Git-Tag: V1_1_0~77 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5afb6a743ae0f722a672140ff6c4296fc904e2b9;p=aspectj.git To support the new testing ICommand wrapper for AjcTask, permit ant.jar in a testing module (ant is omitted from non-testing modules because it is normally deployed separately in ant/lib/ant.jar). This does not affect regular module or product builds. --- diff --git a/build/src/org/aspectj/internal/tools/build/Module.java b/build/src/org/aspectj/internal/tools/build/Module.java index a1ce56942..4233bf1c9 100644 --- a/build/src/org/aspectj/internal/tools/build/Module.java +++ b/build/src/org/aspectj/internal/tools/build/Module.java @@ -97,7 +97,7 @@ public class Module { for (Iterator iter = module.getLibJars().iterator(); iter.hasNext();) { File libJar = (File) iter.next(); - if (!skipLibraryJarAntecedant(libJar) + if (!skipLibraryJarAntecedant(module, libJar) && !known.contains(libJar)) { // XXX what if same referent, diff path... known.add(libJar); } @@ -113,13 +113,17 @@ public class Module { } } - /** XXX gack explicitly skip Ant */ - private static boolean skipLibraryJarAntecedant(File libJar) { + /** XXX gack explicitly skip Ant except for testing... modules */ + private static boolean skipLibraryJarAntecedant(Module module, File libJar) { if (null == libJar) { return true; } - String path = libJar.getPath().replace('\\', '/'); - return (-1 != path.indexOf("/lib/ant/lib/")); + if (!module.name.startsWith("testing")) { + String path = libJar.getPath().replace('\\', '/'); + path = path.replace(File.separatorChar, '/'); + return (-1 != path.indexOf("/lib/ant/lib/")); + } + return false; } /** XXX gack explicitly skip runtime */