Skip to content

Commit 01e93cc

Browse files
committed
Support jdk 21
1 parent 2804681 commit 01e93cc

File tree

4 files changed

+30
-16
lines changed

4 files changed

+30
-16
lines changed

.github/workflows/maven.yml

+2-8
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,8 @@ jobs:
2727
uses: codehaus-plexus/.github/.github/workflows/maven.yml@master
2828
with:
2929
jdk-fast-fail-build: '11'
30-
jdk-matrix: '["11", "17", "20"]'
31-
jdk-distribution-matrix: '["zulu", "temurin", "microsoft", "liberica","corretto"]'
30+
jdk-matrix: '["11", "17", "21"]'
31+
jdk-distribution-matrix: '["zulu", "temurin", "microsoft", "liberica", "corretto"]'
3232
os-matrix: '["ubuntu-latest","windows-latest", "macOS-latest"]'
33-
matrix-exclude: '[
34-
{ "jdk": "21-ea", "distribution": "corretto" },
35-
{ "jdk": "21-ea", "distribution": "liberica" },
36-
{ "jdk": "20", "distribution": "microsoft" },
37-
{ "jdk": "21-ea", "distribution": "temurin" }
38-
]'
3933
maven_args: 'install javadoc:javadoc -e -B -V -fae -Pno-tests-if-not-on-osx'
4034

plexus-compilers/plexus-compiler-javac-errorprone/src/test/java/org/codehaus/plexus/compiler/javac/JavacErrorProneCompilerTest.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@ protected int expectedWarnings()
2121
String javaVersion = getJavaVersion();
2222
if (javaVersion.startsWith("1.8")) {
2323
return 1;
24-
} else {
25-
if (javaVersion.contains("18") || javaVersion.contains("19") || javaVersion.contains("20")) {
26-
return 5;
27-
}
28-
return 2;
24+
} else if (javaVersion.contains("18") || javaVersion.contains("19") || javaVersion.contains("20")) {
25+
return 5;
26+
} else if (javaVersion.contains("21")) {
27+
return 6;
2928
}
29+
return 2;
3030
}
3131

3232
@Override

plexus-compilers/plexus-compiler-javac/src/test/java/org/codehaus/plexus/compiler/javac/AbstractJavacCompilerTest.java

+12-3
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,8 @@ protected int expectedErrors()
6969
String javaVersion = getJavaVersion();
7070
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||
7171
javaVersion.contains("15")||javaVersion.contains("16")||javaVersion.contains("17")||
72-
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")){
72+
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")||
73+
javaVersion.contains("21")){
7374
return 5;
7475
}
7576
// javac output changed for misspelled modifiers starting in 1.6...they now generate 2 errors per occurrence, not one.
@@ -89,7 +90,8 @@ protected int expectedWarnings()
8990
String javaVersion = getJavaVersion();
9091
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||
9192
javaVersion.contains("15")||javaVersion.contains("16")||javaVersion.contains("17")||
92-
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")){
93+
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")||
94+
javaVersion.contains("21")){
9395
return 1;
9496
}
9597
if (javaVersion.contains("1.8")){
@@ -137,6 +139,9 @@ public String getTargetVersion()
137139
if (javaVersion.contains("20")){
138140
return "20";
139141
}
142+
if (javaVersion.contains("21")){
143+
return "21";
144+
}
140145
return super.getTargetVersion();
141146
}
142147

@@ -175,6 +180,9 @@ public String getSourceVersion()
175180
if (javaVersion.contains("20")){
176181
return "20";
177182
}
183+
if (javaVersion.contains("21")){
184+
return "21";
185+
}
178186
return super.getTargetVersion();
179187
}
180188

@@ -184,7 +192,8 @@ protected Collection<String> expectedOutputFiles()
184192
String javaVersion = getJavaVersion();
185193
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||
186194
javaVersion.contains("15")||javaVersion.contains("16")||javaVersion.contains("17")||
187-
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")
195+
javaVersion.contains("18")||javaVersion.contains("19")||javaVersion.contains("20")||
196+
javaVersion.contains("21")
188197
){
189198
return Arrays.asList( "org/codehaus/foo/Deprecation.class", "org/codehaus/foo/ExternalDeps.class",
190199
"org/codehaus/foo/Person.class" );

plexus-compilers/plexus-compiler-javac/src/test/java/org/codehaus/plexus/compiler/javac/JavaxToolsCompilerTest.java

+11
Original file line numberDiff line numberDiff line change
@@ -25,4 +25,15 @@ public class JavaxToolsCompilerTest
2525
extends AbstractJavacCompilerTest
2626
{
2727
// no op default is to javax.tools if available
28+
29+
@Override
30+
protected int expectedWarnings() {
31+
String javaVersion = getJavaVersion();
32+
if (javaVersion.contains("21")) {
33+
return 8;
34+
} else {
35+
return super.expectedWarnings();
36+
}
37+
}
38+
2839
}

0 commit comments

Comments
 (0)