thijsrijpert commented on code in PR #244:
URL:
https://github.com/apache/maven-compiler-plugin/pull/244#discussion_r1622033060
##########
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()
Review Comment:
Yes, I think it would be set by another plugin. The project I was working on
has a lot of plugins, so I would not know which plugin exactly sets this value.
When I encountered this issue, the value was not set in the projects pom, but
it was a non-null empty string when this line was run, indicating it was set
somewhere in the compilation process.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
