cstamas commented on code in PR #2328:
URL: https://github.com/apache/maven/pull/2328#discussion_r2085418836
##########
impl/maven-executor/src/main/java/org/apache/maven/cling/executor/internal/ToolboxTool.java:
##########
@@ -142,7 +142,7 @@ private String validateOutput(boolean shave,
ByteArrayOutputStream stdout, ByteA
}
// sanity checks: stderr has any OR result is empty string (no method
should emit empty string)
if (stderr.size() > 0 || result.trim().isEmpty()) {
- throw new ExecutorException(
+ System.err.println(
Review Comment:
To not have it lost. Without this, the output is lost, this way you have it
at least in surefire report.
And with throw it would just fail (as on Java 24 mvn4 always emits warning
on stderr).
--
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
