gnodet commented on PR #12733:
URL: https://github.com/apache/maven/pull/12733#issuecomment-5396307051

   Thanks for the contribution! The root cause identification is correct — the 
stdout logger is hardcoded to `INFO_INT` and ignores the quiet-mode log level.
   
   However, this fix has an unintended side effect: setting the stdout logger 
to `ERROR_INT` causes the output to be **suppressed entirely**, not just 
stripped of the prefix. The output path is:
   
   ```java
   PrintStream psOut = new LoggingOutputStream(s -> stdout.info("[stdout] " + 
s)).printStream();
   ```
   
   Since the message is emitted via `stdout.info(...)`, an ERROR-level logger 
silences it completely — both the `[INFO] [stdout]` prefix *and* the actual 
value. So `mvn -q help:evaluate -DforceStdout` would produce no output at all 
instead of the clean value.
   
   The correct approach is to bypass the `LoggingOutputStream` wrapper entirely 
in quiet mode (same as `--raw-streams`), which I've implemented in #12810. 
Closing in favor of that PR.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to