Skip to content

Commit d900082

Browse files
authored
fix Java15 build (#89)
* fix java15 build Signed-off-by: olivier lamy <olamy@apache.org> * avoid travis spam Signed-off-by: olivier lamy <olamy@apache.org>
1 parent 671ad54 commit d900082

File tree

3 files changed

+12
-9
lines changed

3 files changed

+12
-9
lines changed

.travis.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,3 @@ cache:
1313
branches:
1414
except:
1515
- gh-pages
16-
notifications:
17-
email:
18-
- olamy@apache.org

plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ public String[] createCommandLine( CompilerConfiguration config )
210210

211211
public static String[] buildCompilerArguments( CompilerConfiguration config, String[] sourceFiles )
212212
{
213-
List<String> args = new ArrayList<String>();
213+
List<String> args = new ArrayList<>();
214214

215215
// ----------------------------------------------------------------------
216216
// Set output
@@ -668,7 +668,7 @@ private static CompilerResult compileInProcess0( Class<?> javacClass, String[] a
668668
static List<CompilerMessage> parseModernStream( int exitCode, BufferedReader input )
669669
throws IOException
670670
{
671-
List<CompilerMessage> errors = new ArrayList<CompilerMessage>();
671+
List<CompilerMessage> errors = new ArrayList<>();
672672

673673
String line;
674674

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

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,10 @@ protected String getRoleHint()
6161
protected int expectedErrors()
6262
{
6363
String javaVersion = getJavaVersion();
64-
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")){
64+
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||javaVersion.contains("15")){
6565
// lots of new warnings about obsoletions for future releases
6666
return 5;
6767
}
68-
6968
// javac output changed for misspelled modifiers starting in 1.6...they now generate 2 errors per occurrence, not one.
7069
if ( "1.5".compareTo( javaVersion ) < 0 )
7170
{
@@ -80,7 +79,7 @@ protected int expectedErrors()
8079
protected int expectedWarnings()
8180
{
8281
String javaVersion = getJavaVersion();
83-
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")){
82+
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||javaVersion.contains("15")){
8483
return 1;
8584
}
8685
if (javaVersion.contains("9.0")){
@@ -115,6 +114,9 @@ public String getTargetVersion()
115114
if (javaVersion.contains("14")){
116115
return "14";
117116
}
117+
if (javaVersion.contains("15")){
118+
return "15";
119+
}
118120
return super.getTargetVersion();
119121
}
120122

@@ -134,13 +136,17 @@ public String getSourceVersion()
134136
{
135137
return "14";
136138
}
139+
if (javaVersion.contains("15"))
140+
{
141+
return "15";
142+
}
137143
return super.getTargetVersion();
138144
}
139145

140146
protected Collection<String> expectedOutputFiles()
141147
{
142148
String javaVersion = getJavaVersion();
143-
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14"))
149+
if (javaVersion.contains("9.0")||javaVersion.contains("11")||javaVersion.contains("14")||javaVersion.contains("15"))
144150
{
145151
return Arrays.asList( new String[]{ "org/codehaus/foo/Deprecation.class", "org/codehaus/foo/ExternalDeps.class",
146152
"org/codehaus/foo/Person.class"} );

0 commit comments

Comments
 (0)