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

[jdcasey]

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() );