Apache
/
maven
mirror of https://github.com/apache/maven.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

add a comment about why the fix missing optional artifacts method is required 

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@332364 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Fabrizio Giustina 2005-11-10 20:18:16 +00:00
parent 2ac4000946
commit cd0f56165b
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipseUtils.java
View File

@ -299,9 +299,12 @@ public class EclipseUtils
* @todo MNG-1384 optional dependencies not resolved while compiling from a master project * @todo MNG-1384 optional dependencies not resolved while compiling from a master project
* Direct optional artifacts are not included in the list returned by project.getTestArtifacts() * Direct optional artifacts are not included in the list returned by project.getTestArtifacts()
* .classpath should include ANY direct dependency, and optional dependencies are required to compile * .classpath should include ANY direct dependency, and optional dependencies are required to compile
* This is fixed in mvn 2.0.1 but this method is needed for compatibility with the 2.0 release. Do not remove!
*/ */
public static void fixMissingOptionalArtifacts( Collection artifacts, Collection depArtifacts, ArtifactRepository localRepository, public static void fixMissingOptionalArtifacts( Collection artifacts, Collection depArtifacts,
ArtifactResolver artifactResolver, List remoteArtifactRepositories,Log log ) ArtifactRepository localRepository,
ArtifactResolver artifactResolver, List remoteArtifactRepositories,
Log log )
{ {
for ( Iterator it = depArtifacts.iterator(); it.hasNext(); ) for ( Iterator it = depArtifacts.iterator(); it.hasNext(); )
{ {