From: wisberg Date: Tue, 21 Jun 2005 18:47:46 +0000 (+0000) Subject: fix release jars including testing-* X-Git-Tag: PRE_ANDY~129 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=50ca3c97e0f3cd0bc0d73270b2d84d2d63845a19;p=aspectj.git fix release jars including testing-* --- diff --git a/build/src/org/aspectj/internal/tools/build/Result.java b/build/src/org/aspectj/internal/tools/build/Result.java index ebf61394d..bd1cbc125 100644 --- a/build/src/org/aspectj/internal/tools/build/Result.java +++ b/build/src/org/aspectj/internal/tools/build/Result.java @@ -329,7 +329,7 @@ public class Result { } private void initRequiredResults() { - Module module = getModule(); + final Module module = getModule(); final Kind kind = getKind(); if (kind.assemble) { if (kind.normal) { @@ -348,18 +348,18 @@ public class Result { } // externally-required: List modules = module.requiredModules(this); - final boolean thisIsTestingModule = isTestingModule(module); + final boolean adoptTests = !kind.normal || isTestingModule(module); for (Iterator iter = modules.iterator(); iter.hasNext();) { - module = (Module) iter.next(); - if (thisIsTestingModule || !kind.normal) { + Module required = (Module) iter.next(); + if (adoptTests) { // testing builds can rely on other release and test results - requiredResults.add(module.getResult(TEST)); - requiredResults.add(module.getResult(RELEASE)); - } else { + requiredResults.add(required.getResult(TEST)); + requiredResults.add(required.getResult(RELEASE)); + } else if (!isTestingModule(required)){ // release builds can only rely on non-testing results // from non-testing modules - requiredResults.add(module.getResult(RELEASE)); - } + requiredResults.add(required.getResult(RELEASE)); + } // else skip release dependencies on testing-* (testing-util) } } diff --git a/lib/build/build.jar b/lib/build/build.jar index ac9f2686a..8cf4d6100 100644 Binary files a/lib/build/build.jar and b/lib/build/build.jar differ