This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-mvnd.git

commit ad4905619ce34a6dbaf192a507aac9fa30942798
Author: Arturo Bernal <[email protected]>
AuthorDate: Wed Oct 28 19:07:51 2026 +0100

    Print "Maven home" before returning early on --version
    
    The early return added for -v/--version skipped the daemon round trip
    but also skipped the "Maven home: <path>" line the daemon used to
    print, which VersionNativeIT/VersionTest both assert on. Print it
    client-side instead, from the same DaemonParameters already in hand.
---
 client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java 
b/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java
index b8db537b..e3dcee8a 100644
--- a/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java
+++ b/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java
@@ -302,8 +302,9 @@ public class DefaultClient implements Client {
                  * -v/--version always makes Maven itself print its own 
version and return without running a
                  * build, no matter what else is on the command line, so a 
daemon round trip could not add
                  * anything here. Skip connecting to (and possibly spawning) a 
daemon just to get that same
-                 * answer back.
+                 * answer back, printing the same "Maven home" line Maven 
itself would have printed.
                  */
+                output.accept(Message.log("Maven home: " + 
parameters.mvndHome()));
                 return DefaultResult.success(argv);
             }
             /*

Reply via email to