You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2026/10/01 19:09:09 UTC

(maven-jdeprscan-plugin) branch master updated: Call ToolchainManager.getToolchains directly

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-jdeprscan-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new 918beff  Call ToolchainManager.getToolchains directly
918beff is described below

commit 918beffd70be98a0603b134a368fa4d20b459e0a
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Thu Oct 1 07:31:04 2026 +0200

    Call ToolchainManager.getToolchains directly
---
 .../plugins/jdeprscan/AbstractJDeprScanMojo.java   | 24 ++++++----------------
 1 file changed, 6 insertions(+), 18 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/jdeprscan/AbstractJDeprScanMojo.java b/src/main/java/org/apache/maven/plugins/jdeprscan/AbstractJDeprScanMojo.java
index eeac567..9d85dca 100644
--- a/src/main/java/org/apache/maven/plugins/jdeprscan/AbstractJDeprScanMojo.java
+++ b/src/main/java/org/apache/maven/plugins/jdeprscan/AbstractJDeprScanMojo.java
@@ -20,10 +20,8 @@ package org.apache.maven.plugins.jdeprscan;
 
 import java.io.File;
 import java.io.IOException;
-import java.lang.reflect.Method;
 import java.util.Collections;
 import java.util.List;
-import java.util.Map;
 import java.util.Properties;
 import java.util.StringTokenizer;
 
@@ -233,22 +231,12 @@ public abstract class AbstractJDeprScanMojo extends AbstractMojo {
             tc = toolchainManager.getToolchainFromBuildContext("jdk", session);
 
             if (tc == null) {
-                // Maven 3.3.0 has plugin execution scoped Toolchain Support
-                try {
-                    Method getToolchainsMethod = toolchainManager
-                            .getClass()
-                            .getMethod("getToolchains", MavenSession.class, String.class, Map.class);
-
-                    @SuppressWarnings("unchecked")
-                    List<Toolchain> tcs = (List<Toolchain>) getToolchainsMethod.invoke(
-                            toolchainManager, session, "jdk", Collections.singletonMap("version", "[9,)"));
-
-                    if (tcs != null && tcs.size() > 0) {
-                        // pick up latest, jdeps of JDK9 has more options compared to JDK8
-                        tc = tcs.get(tcs.size() - 1);
-                    }
-                } catch (ReflectiveOperationException | SecurityException | IllegalArgumentException e) {
-                    // ignore
+                List<Toolchain> tcs =
+                        toolchainManager.getToolchains(session, "jdk", Collections.singletonMap("version", "[9,)"));
+
+                if (tcs != null && tcs.size() > 0) {
+                    // pick up latest, jdeps of JDK9 has more options compared to JDK8
+                    tc = tcs.get(tcs.size() - 1);
                 }
             }
         }