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

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


The following commit(s) were added to refs/heads/master by this push:
     new 0a60ed4  Provide actual toolchain name during "current JDK" log 
messages
0a60ed4 is described below

commit 0a60ed448c00d48f6ee400362c912c395356b142
Author: Nikolas Grottendieck <g...@nikolasgrottendieck.com>
AuthorDate: Sun Apr 20 18:29:14 2025 +0200

    Provide actual toolchain name during "current JDK" log messages
    
    This change makes the info and debug statements during the toolchain 
selection
    process more useful to users. As a user the actually selected toolchain 
name is
    usually interesting/important information and required when debugging 
problems.
    
    Closes #126.
---
 .../apache/maven/plugins/toolchain/jdk/SelectJdkToolchainMojo.java  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/toolchain/jdk/SelectJdkToolchainMojo.java
 
b/src/main/java/org/apache/maven/plugins/toolchain/jdk/SelectJdkToolchainMojo.java
index c76f80f..7a1276a 100644
--- 
a/src/main/java/org/apache/maven/plugins/toolchain/jdk/SelectJdkToolchainMojo.java
+++ 
b/src/main/java/org/apache/maven/plugins/toolchain/jdk/SelectJdkToolchainMojo.java
@@ -185,7 +185,8 @@ public class SelectJdkToolchainMojo extends AbstractMojo {
                 currentJdkToolchainModel != null ? 
factory.createToolchain(currentJdkToolchainModel) : null;
 
         if (useJdk == JdkMode.IfMatch && currentJdkToolchain != null && 
matches(currentJdkToolchain, requirements)) {
-            getLog().info("Not using an external toolchain as the current JDK 
matches the requirements.");
+            getLog().info("Not using an external toolchain as the current JDK 
'" + currentJdkToolchain
+                    + "' matches the requirements.");
             return;
         }
 
@@ -222,7 +223,8 @@ public class SelectJdkToolchainMojo extends AbstractMojo {
         if (useJdk == JdkMode.IfSame
                 && currentJdkToolchain != null
                 && Objects.equals(getJdkHome(currentJdkToolchain), 
getJdkHome(toolchain))) {
-            getLog().debug("Not using an external toolchain as the current JDK 
has been selected.");
+            getLog().debug("Not using an external toolchain as the current JDK 
'" + currentJdkToolchain
+                    + "' has been selected.");
             return;
         }
 

Reply via email to