You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by jd...@apache.org on 2005/11/23 23:27:44 UTC

svn commit: r348584 - /maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java

Author: jdcasey
Date: Wed Nov 23 14:27:41 2005
New Revision: 348584

URL: http://svn.apache.org/viewcvs?rev=348584&view=rev
Log:
Adding more detailed comment for optional artifact inclusion.

Modified:
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java?rev=348584&r1=348583&r2=348584&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java Wed Nov 23 14:27:41 2005
@@ -81,7 +81,8 @@
 
                     if ( node.filterTrail( filter ) )
                     {
-                        // If it was optional, we don't add it or its children, just allow the update of the version and scope
+                        // If it was optional and not a direct dependency, 
+                        // we don't add it or its children, just allow the update of the version and scope
                         if ( node.isChildOfRootNode() || !artifact.isOptional() )
                         {
                             artifact.setDependencyTrail( node.getDependencyTrail() );