From 42b1e6f09e00194ba2ec32e0972b91e19b9161f9 Mon Sep 17 00:00:00 2001 From: Alexander Kriegisch Date: Fri, 12 Mar 2021 10:05:07 +0700 Subject: Add missing '-options' values -1.5, -1.9 to ajc{150,190_from150}.xml When trying to find out why ajc150.xml and ajc190_from150.xml look almost identical, I found out that only in many cases the 190 version has '-option="1.9"' set where in the 150 version it was '-option="1.9"'. Unfortunately, in both files source/target versions are not set at all in many places, which looks unintentional. I tried to search & replace all relevant '' commands for 190 first, then ported back to 150. Only cases in which clearly compiling to an older version like 1.3 or 1.4 is necessary were left like before. I am expecting missing or false Java target versions in many other of the legacy XML test suites. For now, I am just committing these two. Signed-off-by: Alexander Kriegisch --- .../org/aspectj/systemtest/ajc150/ajc150.xml | 443 +++++++---------- .../aspectj/systemtest/ajc190/ajc190_from150.xml | 533 ++++++++++----------- 2 files changed, 427 insertions(+), 549 deletions(-) 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 3b4144efa..0c3f4da84 100644 --- a/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml +++ b/tests/src/test/resources/org/aspectj/systemtest/ajc150/ajc150.xml @@ -38,7 +38,7 @@ - + @@ -102,8 +102,7 @@ - - + @@ -123,7 +122,7 @@ - + @@ -265,12 +264,12 @@ - + - + @@ -288,7 +287,7 @@ - + @@ -300,11 +299,11 @@ - + - + @@ -314,8 +313,8 @@ - - + + @@ -329,7 +328,7 @@ - + @@ -349,24 +348,24 @@ - + - + - + - + @@ -400,7 +399,7 @@ - + @@ -415,17 +414,17 @@ - + - + - + @@ -458,17 +457,17 @@ - + - + - + @@ -504,7 +503,7 @@ - + @@ -542,26 +541,24 @@ - - + - - + - + - + - + @@ -639,26 +636,22 @@ - - + - - + - - + - - + @@ -666,8 +659,7 @@ (see bug 113029). If this is fixed, need to add check for this warning to this test as in test "declare annotation on non existent type - 4" --> - - + @@ -720,7 +712,7 @@ - + @@ -741,8 +733,7 @@ - - + @@ -756,36 +747,33 @@ - - + - + - + - - + - - + - + @@ -799,8 +787,7 @@ - - + @@ -861,24 +848,23 @@ - - + - + - + - + @@ -891,7 +877,7 @@ - + @@ -899,7 +885,7 @@ - + @@ -911,14 +897,12 @@ - - + - - + @@ -930,13 +914,11 @@ - - + - - + @@ -946,7 +928,7 @@ - + @@ -956,18 +938,18 @@ - + - + - + @@ -975,7 +957,7 @@ - + @@ -993,7 +975,7 @@ - + @@ -1004,9 +986,9 @@ - + - + @@ -1019,7 +1001,7 @@ - + @@ -1032,8 +1014,7 @@ - - + @@ -1048,13 +1029,13 @@ - + - + @@ -1069,30 +1050,30 @@ - + - + - + - + - + - + @@ -1101,7 +1082,7 @@ - + @@ -1121,7 +1102,7 @@ - + @@ -1136,13 +1117,13 @@ - + - + @@ -1178,19 +1159,19 @@ - + - + - + @@ -1200,11 +1181,11 @@ - + - + @@ -1249,29 +1230,26 @@ - + - - + - - + - - - + + - - + + @@ -1279,72 +1257,64 @@ - + - + - + - - + - - + - + - - + - + - - + - - + - - + - - + - - + @@ -1355,7 +1325,7 @@ - + @@ -1366,8 +1336,7 @@ - - + @@ -1382,8 +1351,7 @@ - - + @@ -1401,8 +1369,7 @@ - - + @@ -1418,8 +1385,7 @@ - - + @@ -1435,38 +1401,32 @@ - - + - - + - - + - - + - - + - - + @@ -1479,8 +1439,7 @@ - - + @@ -1557,12 +1516,12 @@ - + - + @@ -1654,50 +1613,44 @@ - + - - + - - + - - + - + - - + - - + - - + @@ -1946,8 +1899,7 @@ - - + @@ -1959,8 +1911,7 @@ - - + @@ -1971,13 +1922,11 @@ - - + - - + @@ -1990,8 +1939,7 @@ - - + @@ -2454,8 +2402,7 @@ - - + @@ -2502,7 +2449,7 @@ - + @@ -2516,7 +2463,7 @@ - + @@ -2531,12 +2478,12 @@ - + - + @@ -2546,7 +2493,7 @@ - + @@ -2558,7 +2505,7 @@ - + @@ -2571,23 +2518,19 @@ - - + - - + - - + - - + @@ -2630,7 +2573,7 @@ - + @@ -2698,26 +2641,22 @@ - - + - - + - - + - - + @@ -2808,7 +2747,7 @@ - + @@ -2847,8 +2786,7 @@ - - + @@ -2862,8 +2800,7 @@ - - + @@ -2941,8 +2878,7 @@ - - + @@ -2959,8 +2895,7 @@ - - + @@ -3398,8 +3333,7 @@ - - + @@ -3599,8 +3533,7 @@ - - + @@ -3722,15 +3655,14 @@ - - + - - + + @@ -3807,8 +3739,7 @@ - - + @@ -3894,8 +3825,7 @@ - - + @@ -3993,8 +3923,7 @@ - - + @@ -4003,8 +3932,7 @@ - - + @@ -4013,15 +3941,13 @@ - - + - - + @@ -4965,7 +4891,7 @@ - + @@ -5171,8 +5097,7 @@ - - + @@ -5375,8 +5300,7 @@ - - + @@ -5435,8 +5359,7 @@ - - + @@ -5482,8 +5405,7 @@ - - + @@ -5562,8 +5484,7 @@ - - + @@ -6090,23 +6011,23 @@ - + - + - + - + @@ -6121,12 +6042,12 @@ - + - + @@ -6230,7 +6151,7 @@ - + @@ -6244,7 +6165,7 @@ - + @@ -6258,8 +6179,8 @@ - - + + @@ -6268,8 +6189,8 @@ - - + + @@ -6278,7 +6199,7 @@ - + @@ -6288,8 +6209,8 @@ - - + + @@ -6299,8 +6220,8 @@ - - + + @@ -6309,7 +6230,7 @@ - + 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 2a0abca43..9eacce24a 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 @@ -1,6 +1,6 @@ - + @@ -38,7 +38,7 @@ - + @@ -102,8 +102,7 @@ - - + @@ -123,7 +122,7 @@ - + @@ -214,7 +213,7 @@ - + @@ -236,7 +235,7 @@ - + @@ -265,12 +264,12 @@ - + - + @@ -288,7 +287,7 @@ - + @@ -300,11 +299,11 @@ - + - + @@ -314,8 +313,8 @@ - - + + @@ -329,7 +328,7 @@ - + @@ -349,24 +348,24 @@ - + - + - + - + @@ -400,7 +399,7 @@ - + @@ -415,17 +414,17 @@ - + - + - + @@ -445,7 +444,7 @@ - + @@ -458,23 +457,23 @@ - + - + - + - - + + @@ -504,7 +503,7 @@ - + @@ -542,26 +541,24 @@ - - + - - + - + - + - + @@ -639,26 +636,22 @@ - - + - - + - - + - - + @@ -666,8 +659,7 @@ (see bug 113029). If this is fixed, need to add check for this warning to this test as in test "declare annotation on non existent type - 4" --> - - + @@ -720,7 +712,7 @@ - + @@ -741,8 +733,7 @@ - - + @@ -756,36 +747,33 @@ - - + - + - + - - + - - + - + @@ -799,8 +787,7 @@ - - + @@ -861,24 +848,23 @@ - - + - + - + - + @@ -891,7 +877,7 @@ - + @@ -899,7 +885,7 @@ - + @@ -911,14 +897,12 @@ - - + - - + @@ -930,13 +914,11 @@ - - + - - + @@ -946,7 +928,7 @@ - + @@ -956,18 +938,18 @@ - + - + - + @@ -975,7 +957,7 @@ - + @@ -993,7 +975,7 @@ - + @@ -1004,9 +986,9 @@ - + - + @@ -1019,7 +1001,7 @@ - + @@ -1032,13 +1014,12 @@ - - + - + @@ -1048,13 +1029,13 @@ - + - + @@ -1069,30 +1050,30 @@ - + - + - + - + - + - + @@ -1101,7 +1082,7 @@ - + @@ -1121,7 +1102,7 @@ - + @@ -1136,13 +1117,13 @@ - + - + @@ -1178,19 +1159,19 @@ - + - + - + @@ -1200,11 +1181,11 @@ - + - + @@ -1249,29 +1230,26 @@ - + - - + - - + - - - + + - - + + @@ -1279,72 +1257,64 @@ - + - + - + - - + - - + - + - - + - + - - + - - + - - + - - + - - + @@ -1355,7 +1325,7 @@ - + @@ -1366,8 +1336,7 @@ - - + @@ -1382,8 +1351,7 @@ - - + @@ -1401,8 +1369,7 @@ - - + @@ -1418,8 +1385,7 @@ - - + @@ -1435,38 +1401,32 @@ - - + - - + - - + - - + - - + - - + @@ -1479,8 +1439,7 @@ - - + @@ -1557,12 +1516,12 @@ - + - + @@ -1654,50 +1613,44 @@ - + - - + - - + - - + - + - - + - - + - - + @@ -1946,8 +1899,7 @@ - - + @@ -1959,8 +1911,7 @@ - - + @@ -1971,13 +1922,11 @@ - - + - - + @@ -1990,8 +1939,7 @@ - - + @@ -2454,8 +2402,7 @@ - - + @@ -2502,7 +2449,7 @@ - + @@ -2516,7 +2463,7 @@ - + @@ -2531,12 +2478,12 @@ - + - + @@ -2546,7 +2493,7 @@ - + @@ -2555,10 +2502,10 @@ - + - + @@ -2571,23 +2518,19 @@ - - + - - + - - + - - + @@ -2630,7 +2573,7 @@ - + @@ -2698,26 +2641,22 @@ - - + - - + - - + - - + @@ -2847,8 +2786,7 @@ - - + @@ -2862,8 +2800,7 @@ - - + @@ -2941,8 +2878,7 @@ - - + @@ -2959,8 +2895,7 @@ - - + @@ -3398,8 +3333,7 @@ - - + @@ -3599,8 +3533,7 @@ - - + @@ -3722,15 +3655,14 @@ - - + - - + + @@ -3807,34 +3739,69 @@ - - + - - - + + + + + + + + + - - + + + + + + - - - - + + + + + + + + + + + + - - - - - + + + + + + + + + + + + + + + - - - - + + + + + + + + + + + + @@ -3858,8 +3825,7 @@ - - + @@ -3957,8 +3923,7 @@ - - + @@ -3967,8 +3932,7 @@ - - + @@ -3977,15 +3941,13 @@ - - + - - + @@ -4929,7 +4891,7 @@ - + @@ -5135,8 +5097,7 @@ - - + @@ -5339,8 +5300,7 @@ - - + @@ -5399,8 +5359,7 @@ - - + @@ -5446,8 +5405,7 @@ - - + @@ -5526,8 +5484,7 @@ - - + @@ -6054,23 +6011,23 @@ - + - + - + - + @@ -6085,12 +6042,12 @@ - + - + @@ -6194,7 +6151,7 @@ - + @@ -6208,7 +6165,7 @@ - + @@ -6222,8 +6179,8 @@ - - + + @@ -6232,8 +6189,8 @@ - - + + @@ -6242,8 +6199,8 @@ - - + + @@ -6252,8 +6209,8 @@ - - + + @@ -6263,8 +6220,8 @@ - - + + @@ -6273,8 +6230,8 @@ - - + + -- cgit v1.2.3 From a78abecee0a116336cc9ae604c2a7fbe9d11d621 Mon Sep 17 00:00:00 2001 From: Alexander Kriegisch Date: Fri, 12 Mar 2021 10:16:36 +0700 Subject: Fix: WeaveSpec no longer ignores the '-Xlintfile' parameter Some tests in ajc150.xml and ajc190_from150.xml contain '' build steps with 'xlintfile="..."' parameters. Those parameters were passed through to Java and aspect code building steps, but not to the final weaving step, sometimes leading to spurious "type not exposed to weaver" warnings which occurred for some local and CI builds, but not always. Very strange indeed. Anyway, by making method WeaveSpec.buildWeaveArgs() pass on this parameter, the tests seem to run reliably now. TODO: Why does Ajc report that warning if the application JAR is on the inpath and the aspect JAR is on the aspectpath? Is it because a marker annotation is defined within the aspect JAR? But actually, that should not matter, especially not work once and fail at other times. I guess there is a class loading order problem or similar involved. Signed-off-by: Alexander Kriegisch --- .../test/java/org/aspectj/testing/WeaveSpec.java | 41 ++++++++++++---------- 1 file changed, 23 insertions(+), 18 deletions(-) diff --git a/testing/src/test/java/org/aspectj/testing/WeaveSpec.java b/testing/src/test/java/org/aspectj/testing/WeaveSpec.java index 9d95e79b5..41b561311 100644 --- a/testing/src/test/java/org/aspectj/testing/WeaveSpec.java +++ b/testing/src/test/java/org/aspectj/testing/WeaveSpec.java @@ -1,13 +1,13 @@ /* ******************************************************************* * Copyright (c) 2005 IBM Corporation - * All rights reserved. - * This program and the accompanying materials are made available - * under the terms of the Eclipse Public License v1.0 - * which accompanies this distribution and is available at - * http://www.eclipse.org/legal/epl-v10.html - * - * Contributors: - * Adrian Colyer, + * All rights reserved. + * This program and the accompanying materials are made available + * under the terms of the Eclipse Public License v1.0 + * which accompanies this distribution and is available at + * http://www.eclipse.org/legal/epl-v10.html + * + * Contributors: + * Adrian Colyer, * ******************************************************************/ package org.aspectj.testing; @@ -52,21 +52,21 @@ public class WeaveSpec extends CompileSpec { inTestCase.assertNoMessages(result,failMessage); File sandbox = inTestCase.getSandboxDirectory(); createJar(sandbox,"classes.jar",true); - - inTestCase.setShouldEmptySandbox(false); + + inTestCase.setShouldEmptySandbox(false); setFiles(aspectsFiles); String options = getOptions(); if (options == null) { - setOptions(""); + setOptions(""); } setClasspath("classes.jar"); args = buildArgs(); result = inTestCase.ajc(base,args); inTestCase.assertNoMessages(result,failMessage); createJar(sandbox,"aspects.jar",false); - + args = buildWeaveArgs(); - inTestCase.setShouldEmptySandbox(false); + inTestCase.setShouldEmptySandbox(false); result = inTestCase.ajc(base,args); AjcTestCase.MessageSpec messageSpec = buildMessageSpec(); inTestCase.assertMessages(result,failMessage,messageSpec); @@ -79,7 +79,7 @@ public class WeaveSpec extends CompileSpec { public void setClassesFiles(String files) { this.classesFiles = files; } - + public void setAspectsFiles(String files) { this.aspectsFiles = files; } @@ -112,7 +112,7 @@ public class WeaveSpec extends CompileSpec { jarOut.flush(); jarOut.close(); } - + private void collectClassFiles(File inDir, List inList, List toExclude) { File[] contents = inDir.listFiles(); for (File content : contents) { @@ -125,7 +125,7 @@ public class WeaveSpec extends CompileSpec { } } } - + private void copyFile(File f, OutputStream dest) throws IOException { FileInputStream fis = new FileInputStream(f); byte[] buf = new byte[4096]; @@ -135,7 +135,7 @@ public class WeaveSpec extends CompileSpec { } fis.close(); } - + private String[] buildWeaveArgs() { StringBuffer args = new StringBuffer(); if (getOptions() != null) { @@ -152,6 +152,11 @@ public class WeaveSpec extends CompileSpec { args.append(" "); args.append("-aspectpath "); args.append("aspects.jar"); + if (getXlintfile() != null) { + args.append(" -Xlintfile "); + args.append(getXlintfile()); + args.append(" "); + } String argumentString = args.toString(); StringTokenizer strTok = new StringTokenizer(argumentString," "); String[] ret = new String[strTok.countTokens()]; @@ -160,5 +165,5 @@ public class WeaveSpec extends CompileSpec { } return ret; } - + } -- cgit v1.2.3