[MCOMPILER-592] - IndexOutOfBoundsException when project.build.outputTimestamp is empty (#244)

thijsrijpert · olamy · commit fdebd58ff69c · 2024-08-23T18:34:36.000+10:00
diff --git a/src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java b/src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java
@@ -1239,7 +1239,9 @@ public void execute() throws MojoExecutionException, CompilationFailureException
             }
         }
 
-        if (outputTimestamp != null && (outputTimestamp.length() > 1 || Character.isDigit(outputTimestamp.charAt(0)))) {
+        if (outputTimestamp != null
+                && !outputTimestamp.isEmpty()
+                && (outputTimestamp.length() > 1 || Character.isDigit(outputTimestamp.charAt(0)))) {
             // if Reproducible Builds mode, apply workaround
             patchJdkModuleVersion(compilerResult, sources);
         }

Original file line number	Diff line number	Diff line change
`@@ -1239,7 +1239,9 @@ public void execute() throws MojoExecutionException, CompilationFailureException`
`1239`	`1239`	`}`
`1240`	`1240`	`}`
`1241`	`1241`
`1242`		`- if (outputTimestamp != null && (outputTimestamp.length() > 1 \|\| Character.isDigit(outputTimestamp.charAt(0)))) {`
	`1242`	`+ if (outputTimestamp != null`
	`1243`	`+ && !outputTimestamp.isEmpty()`
	`1244`	`+ && (outputTimestamp.length() > 1 \|\| Character.isDigit(outputTimestamp.charAt(0)))) {`
`1243`	`1245`	`// if Reproducible Builds mode, apply workaround`
`1244`	`1246`	`patchJdkModuleVersion(compilerResult, sources);`
`1245`	`1247`	`}`