dependency:tree omits batik-js

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: maven-dependency-tree-1.2
Fix Version/s: maven-dependency-tree-2.0
Component/s: maven-dependency-tree
Labels:
None
Environment:
Maven 3.0, Ubuntu, JDK 6.

Testcase included:
yes
Number of attachments :
1

Description

(Not sure what the right place to file this is. maven-dependency-tree 1.2 gives MNG as the JIRA component by inheritance.)

mvn dependency:tree on the attached project lists batik-script by way of batik-bridge as expected, but then fails to show batik-js as a dependency of that. If you list batik-script as a direct dependency, batik-js is correctly shown.

Possibly related is that Maven 2.2.1 also fails to find batik-js in MavenProject.getRuntimeArtifacts, so clean package fails, whereas this works fine under Maven 3.0. Yet just running the dependency plugin under M3 does not suffice to pick up this fix, wherever it was - ~~MNG-4690~~?

Although the Batik JARs are built from an odd source tree, the artifacts as present in the local repository look normal enough; all of the dependencies involved are simple compile scope without exclusions etc.

Marking "major" since this seems to cause ~~MNBMODULE-102~~ (producing build errors for certain projects); also I have noticed that the dependency graph feature in the NB IDE omits batik-js, and there may be other user-visible effects of this bug.

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Hide
test_test-dependency-tree_jar_1.0-SNAPSHOT_101018-185145.zip

18/Oct/10 6:09 PM

2 kB

Jesse Glick
test-dependency-tree/pom.xml 1.0 kB

test-dependency-tree/src/.../C.java 0.3 kB

Download Zip
Show

test_test-dependency-tree_jar_1.0-SNAPSHOT_101018-185145.zip

18/Oct/10 6:09 PM

2 kB

Jesse Glick

Issue Links

is depended upon by

MPIR-237 Unexpected download

MNBMODULE-102 Transitive dependencies incorrectly processed in Maven 3

MDEP-339 tree mojo doesn't work consistently with maven3

MENFORCER-140 DependencyConvergence: upgrade maven-dependency-tree dependency to 2.0 for better Maven 3 support

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Benjamin Bentmann added a comment - 18/Oct/10 6:23 PM

The tree calculated by Maven 3.0 core is correct:

[DEBUG] test:test-dependency-tree:jar:1.0-SNAPSHOT
[DEBUG]    org.apache.xmlgraphics:batik-swing:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-awt-util:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-bridge:jar:1.7:compile
[DEBUG]          org.apache.xmlgraphics:batik-anim:jar:1.7:compile
[DEBUG]          org.apache.xmlgraphics:batik-parser:jar:1.7:compile
[DEBUG]          org.apache.xmlgraphics:batik-script:jar:1.7:compile
[DEBUG]             org.apache.xmlgraphics:batik-js:jar:1.7:compile
[DEBUG]          org.apache.xmlgraphics:batik-xml:jar:1.7:compile
[DEBUG]          xalan:xalan:jar:2.6.0:compile
[DEBUG]       org.apache.xmlgraphics:batik-css:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-dom:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-ext:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-gui-util:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-gvt:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-svg-dom:jar:1.7:compile
[DEBUG]       org.apache.xmlgraphics:batik-util:jar:1.7:compile
[DEBUG]       xml-apis:xml-apis:jar:1.3.04:compile
[DEBUG]       xml-apis:xml-apis-ext:jar:1.3.04:compile

.
As mentioned in the compat notes, maven-dependency-tree couldn't be briged onto Aether and this is one of those corner cases where the discrepancy gets visible.

Show

Benjamin Bentmann added a comment - 18/Oct/10 6:23 PM The tree calculated by Maven 3.0 core is correct: [DEBUG] test:test-dependency-tree:jar:1.0-SNAPSHOT [DEBUG] org.apache.xmlgraphics:batik-swing:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-awt-util:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-bridge:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-anim:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-parser:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-script:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-js:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-xml:jar:1.7:compile [DEBUG] xalan:xalan:jar:2.6.0:compile [DEBUG] org.apache.xmlgraphics:batik-css:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-dom:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-ext:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-gui-util:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-gvt:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-svg-dom:jar:1.7:compile [DEBUG] org.apache.xmlgraphics:batik-util:jar:1.7:compile [DEBUG] xml-apis:xml-apis:jar:1.3.04:compile [DEBUG] xml-apis:xml-apis-ext:jar:1.3.04:compile . As mentioned in the compat notes, maven-dependency-tree couldn't be briged onto Aether and this is one of those corner cases where the discrepancy gets visible.

Hide

Permalink

Jesse Glick added a comment - 19/Oct/10 9:58 AM

I see. Should maven-dependency-tree then be considered fully deprecated? If so, is there some plan to release a fixed maven-dependency-plugin; and is there some documentation on which API in Maven core (or Aether) should be used in place of the deprecated component?

Show

Jesse Glick added a comment - 19/Oct/10 9:58 AM I see. Should maven-dependency-tree then be considered fully deprecated? If so, is there some plan to release a fixed maven-dependency-plugin; and is there some documentation on which API in Maven core (or Aether) should be used in place of the deprecated component?

Hide

Permalink

Jesse Glick added a comment - 01/Nov/10 1:50 PM

The above tree seems to be printed by:

	at org.apache.maven.project.DefaultProjectDependenciesResolver$GraphLogger.visitEnter(DefaultProjectDependenciesResolver.java:223)
	at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:173)
	at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177)
	at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177)
	at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177)
	at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177)
	at org.apache.maven.project.DefaultProjectDependenciesResolver.resolve(DefaultProjectDependenciesResolver.java:145)

Show

Jesse Glick added a comment - 01/Nov/10 1:50 PM The above tree seems to be printed by: at org.apache.maven.project.DefaultProjectDependenciesResolver$GraphLogger.visitEnter(DefaultProjectDependenciesResolver.java:223) at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:173) at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177) at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177) at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177) at org.sonatype.aether.impl.internal.GraphEdge.accept(GraphEdge.java:177) at org.apache.maven.project.DefaultProjectDependenciesResolver.resolve(DefaultProjectDependenciesResolver.java:145)

Hide

Permalink

Jesse Glick added a comment - 27/Jul/11 1:00 PM

Possibly related to ~~MNG-4982~~?

Show

Jesse Glick added a comment - 27/Jul/11 1:00 PM Possibly related to MNG-4982 ?

Hide

Permalink

David Tombs added a comment - 13/Sep/11 9:52 AM

This issue just caused me some grief. Could this discrepancy between dependency:tree and the actual tree used possibly be documented in the dependency:tree docs?

Show

David Tombs added a comment - 13/Sep/11 9:52 AM This issue just caused me some grief. Could this discrepancy between dependency:tree and the actual tree used possibly be documented in the dependency:tree docs?

Hide

Permalink

Jesse Glick added a comment - 14/Oct/11 12:03 PM

I do not have commit privileges, so:

Index: src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java	(revision 1183422)
+++ src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java	(working copy)
@@ -65,7 +65,7 @@
 
 /**
  * Displays the dependency tree for this project.
- *
+ * Beware that due to <a href="https://jira.codehaus.org/browse/MSHARED-167">MSHARED-167</a> the result may be incorrect in obscure cases.
  * @author <a href="mailto:markhobson@gmail.com">Mark Hobson</a>
  * @version $Id$
  * @since 2.0-alpha-5

Show

Jesse Glick added a comment - 14/Oct/11 12:03 PM I do not have commit privileges, so: Index: src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java =================================================================== --- src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java (revision 1183422) +++ src/main/java/org/apache/maven/plugin/dependency/TreeMojo.java (working copy) @@ -65,7 +65,7 @@ /** * Displays the dependency tree for this project. - * + * Beware that due to <a href= "https: //jira.codehaus.org/browse/MSHARED-167" >MSHARED-167</a> the result may be incorrect in obscure cases. * @author <a href= "mailto:markhobson@gmail.com" >Mark Hobson</a> * @version $Id$ * @since 2.0-alpha-5

Hide

Permalink

Herve Boutemy added a comment - 10/Jun/12 4:20 PM

new DependencyGraphBuilder component created in r1348663

this component has a Maven 2 implementation using "usual" DependencyTreeBuilder and a Maven 3 implementation using Maven 3 with Aether
Default implementation detects current Maven version and automatically switches between the 2

Show

Herve Boutemy added a comment - 10/Jun/12 4:20 PM new DependencyGraphBuilder component created in r1348663 this component has a Maven 2 implementation using "usual" DependencyTreeBuilder and a Maven 3 implementation using Maven 3 with Aether Default implementation detects current Maven version and automatically switches between the 2

Hide

Permalink

Herve Boutemy added a comment - 19/Apr/13 12:33 AM

IT added in r1469697: clearly shows how M2 has wrong resolution where M3 has a fixed one

Show

Herve Boutemy added a comment - 19/Apr/13 12:33 AM IT added in r1469697 : clearly shows how M2 has wrong resolution where M3 has a fixed one

People

Assignee:

Herve Boutemy

Reporter:

Jesse Glick

Vote (1)

Watch (4)

Dates

Created:

18/Oct/10 6:09 PM

Updated:

19/Apr/13 12:33 AM

Resolved:

10/Jun/12 4:20 PM