[MNG-8295] Dependency Manager Transitivity (now default) handles depe…

…ndency management inconsistently (#1788) Actually use TransitiveDependencyManager when maven.resolver.dependencyManagerTransitivity=true Co-authored-by: DidierLoiseau <[email protected]>
apache · Oct 15, 2024 · d742fd6 · d742fd6
1 parent 651a4b2
commit d742fd6
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...pl/src/main/java/org/apache/maven/internal/impl/resolver/MavenSessionBuilderSupplier.java b/...pl/src/main/java/org/apache/maven/internal/impl/resolver/MavenSessionBuilderSupplier.java
@@ -43,6 +43,7 @@
 import org.eclipse.aether.resolution.ArtifactDescriptorPolicy;
 import org.eclipse.aether.util.artifact.DefaultArtifactTypeRegistry;
 import org.eclipse.aether.util.graph.manager.ClassicDependencyManager;
+import org.eclipse.aether.util.graph.manager.TransitiveDependencyManager;
 import org.eclipse.aether.util.graph.selector.AndDependencySelector;
 import org.eclipse.aether.util.graph.selector.ExclusionDependencySelector;
 import org.eclipse.aether.util.graph.transformer.ChainedDependencyGraphTransformer;
@@ -84,7 +85,10 @@ protected DependencyManager getDependencyManager() {
     }
 
     public DependencyManager getDependencyManager(boolean transitive) {
-        return new ClassicDependencyManager(transitive, getScopeManager());
+        if (transitive) {
+            return new TransitiveDependencyManager(getScopeManager());
+        }
+        return new ClassicDependencyManager(getScopeManager());
     }
 
     protected DependencySelector getDependencySelector() {