From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Fri, 15 Mar 2024 12:39:04 +0000 (+0100)
Subject: Remove BCEL files from 'lib' module
X-Git-Tag: V1_9_22~16
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1d17d255a921dfe6f41ccc529fca0486ecc7fb9f;p=aspectj.git

Remove BCEL files from 'lib' module

Both bcel.jar and bcel-verifier.jar seem to be obsolete.

Possible next step: Remove bcel-builder/verifier-src.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
---

diff --git a/lib/bcel/bcel-src.zip b/lib/bcel/bcel-src.zip
deleted file mode 100644
index a6f847d75..000000000
Binary files a/lib/bcel/bcel-src.zip and /dev/null differ
diff --git a/lib/bcel/bcel-verifier-src.zip b/lib/bcel/bcel-verifier-src.zip
deleted file mode 100644
index 916e947d0..000000000
Binary files a/lib/bcel/bcel-verifier-src.zip and /dev/null differ
diff --git a/lib/bcel/bcel-verifier.jar b/lib/bcel/bcel-verifier.jar
deleted file mode 100644
index 41471b198..000000000
Binary files a/lib/bcel/bcel-verifier.jar and /dev/null differ
diff --git a/lib/bcel/bcel.jar b/lib/bcel/bcel.jar
deleted file mode 100644
index d5cbf14eb..000000000
Binary files a/lib/bcel/bcel.jar and /dev/null differ
diff --git a/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java b/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
index b76fe4ee0..bc934815c 100644
--- a/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
+++ b/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
@@ -104,8 +104,6 @@ public abstract class AjcTestCase extends TestCase {
 	public static final String DEFAULT_CLASSPATH_ENTRIES =
 		DEFAULT_FULL_LTW_CLASSPATH_ENTRIES
 			+ Ajc.outputFolders("bridge", "util", "loadtime", "weaver", "asm", "runtime", "org.aspectj.matcher", "bcel-builder")
-			+ pathSeparator + ".." + separator + "lib" + separator + "bcel" + separator + "bcel.jar"
-			+ pathSeparator + ".." + separator + "lib" + separator + "bcel" + separator + "bcel-verifier.jar"
 			+ pathSeparator + CLASSPATH_JDT_CORE
 			+ pathSeparator + CLASSPATH_ASM
 			+ pathSeparator + CLASSPATH_ASM_COMMONS
diff --git a/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml b/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml
index 6ce13fb0d..138ca57ab 100644
--- a/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml
+++ b/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml
@@ -361,12 +361,12 @@
 
 	<ajc-test dir="java5/reflection" title="pointcut parsing with ajc compiled pointcut references">
 		<compile files="PointcutLibrary.aj,ReflectOnAjcCompiledPointcuts.java" options="-1.5"/>
-		<run class="ReflectOnAjcCompiledPointcuts" classpath="../lib/bcel/bcel.jar"/>
+		<run class="ReflectOnAjcCompiledPointcuts"/>
 	</ajc-test>
 
 	<ajc-test dir="java5/reflection" title="reflection on itds">
 		<compile files="InterTypeDeclarations.aj,ReflectOnCodeStyleITDs.java" options="-1.5 -Xlint:ignore -makeAjReflectable"/>
-		<run class="ReflectOnCodeStyleITDs" classpath="../lib/bcel/bcel.jar">
+		<run class="ReflectOnCodeStyleITDs">
 		  <stdout>
 		  	<line text="public C.new(int, int, int)"/>
 		  	<line text="C.new(int, int)"/>
@@ -400,7 +400,7 @@
 
 	<ajc-test dir="java5/reflection" title="reflection on @DeclareParents">
 		<compile files="AtAspectJDeclareParents.aj,ReflectOnAtAspectJDeclareParents.java" options="-1.5 -Xlint:ignore"/>
-		<run class="ReflectOnAtAspectJDeclareParents" classpath="../lib/bcel/bcel.jar">
+		<run class="ReflectOnAtAspectJDeclareParents">
 		  <stdout>
 		    <line text="declare parents : C implements I"/>
 		  	<line text="public int I.getX()"/>
diff --git a/tests/src/test/resources/org/aspectj/systemtest/ajc150/ataspectj/annotationgen.xml b/tests/src/test/resources/org/aspectj/systemtest/ajc150/ataspectj/annotationgen.xml
index fd9d2c111..0de24e3e0 100644
--- a/tests/src/test/resources/org/aspectj/systemtest/ajc150/ataspectj/annotationgen.xml
+++ b/tests/src/test/resources/org/aspectj/systemtest/ajc150/ataspectj/annotationgen.xml
@@ -146,7 +146,7 @@
    <ajc-test dir="java5/ataspectj/annotationGen" title="runtime pointcut resolution referencing compiled pointcuts">
         <compile files="PCLib.aj,RuntimePointcuts.java" options="-1.5">
         </compile>
-        <run class="RuntimePointcuts" classpath=".,../lib/bcel/bcel.jar" ltw=""/>
+        <run class="RuntimePointcuts"/>
     </ajc-test>
 
    <ajc-test dir="java5/ataspectj/annotationGen" title="ann gen for decp">
diff --git a/tests/src/test/resources/org/aspectj/systemtest/ajc190/ajc190_from150.xml b/tests/src/test/resources/org/aspectj/systemtest/ajc190/ajc190_from150.xml
index 79a942447..d21ecf8e6 100644
--- a/tests/src/test/resources/org/aspectj/systemtest/ajc190/ajc190_from150.xml
+++ b/tests/src/test/resources/org/aspectj/systemtest/ajc190/ajc190_from150.xml
@@ -361,12 +361,12 @@
 
 	<ajc-test dir="java5/reflection" title="pointcut parsing with ajc compiled pointcut references">
 		<compile files="PointcutLibrary.aj,ReflectOnAjcCompiledPointcuts.java" options="-1.9"/>
-		<run class="ReflectOnAjcCompiledPointcuts" classpath="../lib/bcel/bcel.jar"/>
+		<run class="ReflectOnAjcCompiledPointcuts"/>
 	</ajc-test>
 
 	<ajc-test dir="java5/reflection" title="reflection on itds">
 		<compile files="InterTypeDeclarations.aj,ReflectOnCodeStyleITDs.java" options="-1.9 -Xlint:ignore -makeAjReflectable"/>
-		<run class="ReflectOnCodeStyleITDs" classpath="../lib/bcel/bcel.jar">
+		<run class="ReflectOnCodeStyleITDs">
 		  <stdout>
 		  	<line text="public C.new(int, int, int)"/>
 		  	<line text="C.new(int, int)"/>
@@ -400,7 +400,7 @@
 
 	<ajc-test dir="java5/reflection" title="reflection on @DeclareParents">
 		<compile files="AtAspectJDeclareParents.aj,ReflectOnAtAspectJDeclareParents.java" options="-1.9 -Xlint:ignore"/>
-		<run class="ReflectOnAtAspectJDeclareParents" classpath="../lib/bcel/bcel.jar">
+		<run class="ReflectOnAtAspectJDeclareParents">
 		  <stdout>
 		    <line text="declare parents : C implements I"/>
 		  	<line text="public int I.getX()"/>
diff --git a/weaver/src/test/java/org/aspectj/weaver/bcel/ZipTestCase.java b/weaver/src/test/java/org/aspectj/weaver/bcel/ZipTestCase.java
index ccce12c41..dbdc35b46 100644
--- a/weaver/src/test/java/org/aspectj/weaver/bcel/ZipTestCase.java
+++ b/weaver/src/test/java/org/aspectj/weaver/bcel/ZipTestCase.java
@@ -99,23 +99,27 @@ public class ZipTestCase extends TestCase {
 
 	public void testBig() throws IOException {
 		System.out.println("could take 4 seconds...");
-		zipTest("../lib/bcel/bcel.jar", null);
+		// Weave a big JAR file. This is unrelated to Ant, i.e. the JAR can be replaced by another one for this test.
+		zipTest("../lib/ant/lib/ant.jar", null);
 	}
 
 	public void testBigWithEasyNoTrace() throws IOException {
 		System.out.println("could take 4 seconds...");
-		zipTest("../lib/bcel/bcel.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace0easy.jar");
+		// Weave a big JAR file. This is unrelated to Ant, i.e. the JAR can be replaced by another one for this test.
+		zipTest("../lib/ant/lib/ant.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace0easy.jar");
 	}
 
 	// this is something we test every now and again.
 	public void xtestBigWithHardNoTrace() throws IOException {
 		System.out.println("could take 24 seconds...");
-		zipTest("../lib/bcel/bcel.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace0hard.jar");
+		// Weave a big JAR file. This is unrelated to Ant, i.e. the JAR can be replaced by another one for this test.
+		zipTest("../lib/ant/lib/ant.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace0hard.jar");
 	}
 
 	public void xtestBigWithAspects() throws IOException {
 		System.out.println("could take 40 seconds...");
-		zipTest("../lib/bcel/bcel.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace.jar");
+		// Weave a big JAR file. This is unrelated to Ant, i.e. the JAR can be replaced by another one for this test.
+		zipTest("../lib/ant/lib/ant.jar", WeaverTestCase.TESTDATA_PATH + "/megatrace.jar");
 	}
 
 }