[MRESOLVER-9] DefaultDependencyCollector does not correctly handle dependency 
management.

Resolution without MRESOLVER-9 looks like (Maven 3.3.9):

[DEBUG]    org.sonatype.sisu:sisu-guice:jar:no_aop:3.2.6:compile
[DEBUG]       javax.inject:javax.inject:jar:1:provided (scope managed from 
compile by org.apache.maven.resolver:maven-resolver:1.1.1-SNAPSHOT)
[DEBUG]       com.google.guava:guava:jar:16.0.1:compile

The 'javax.inject:javax.inject:jar:1:provided' line shows that the artifact
is in 'provided' scope. The Maven resolver without MRESOLVER-9 fixed will
incorrectly resolve that transitive provided dependency. With MRESOLVER-9 fixed,
the resolver correctly no longer resolves transitive provided dependencies. That
dependency needs to be declared directly.


Project: http://git-wip-us.apache.org/repos/asf/maven-resolver/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-resolver/commit/db4003a8
Tree: http://git-wip-us.apache.org/repos/asf/maven-resolver/tree/db4003a8
Diff: http://git-wip-us.apache.org/repos/asf/maven-resolver/diff/db4003a8

Branch: refs/heads/master
Commit: db4003a867a22b6dc66d96f83b3dcb6f364c8b5d
Parents: 9dbbd06
Author: Christian Schulte <schu...@apache.org>
Authored: Fri Dec 22 01:36:59 2017 +0100
Committer: Christian Schulte <schu...@apache.org>
Committed: Fri Dec 22 01:39:38 2017 +0100

----------------------------------------------------------------------
 maven-resolver-demos/maven-resolver-demo-snippets/pom.xml | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven-resolver/blob/db4003a8/maven-resolver-demos/maven-resolver-demo-snippets/pom.xml
----------------------------------------------------------------------
diff --git a/maven-resolver-demos/maven-resolver-demo-snippets/pom.xml 
b/maven-resolver-demos/maven-resolver-demo-snippets/pom.xml
index 32253aa..b3bfeaa 100644
--- a/maven-resolver-demos/maven-resolver-demo-snippets/pom.xml
+++ b/maven-resolver-demos/maven-resolver-demo-snippets/pom.xml
@@ -95,6 +95,10 @@
       <optional>true</optional>
     </dependency>
     <dependency>
+      <groupId>javax.inject</groupId>
+      <artifactId>javax.inject</artifactId>
+    </dependency>
+    <dependency>
       <groupId>org.sonatype.sisu</groupId>
       <artifactId>sisu-guice</artifactId>
       <classifier>no_aop</classifier>

Reply via email to