You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/09/03 08:18:30 UTC

svn commit: r267425 - in /maven/components/trunk/maven-artifact-ant: sample.build.xml src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java src/main/resources/META-INF/plexus/components.xml

Author: brett
Date: Fri Sep  2 23:18:23 2005
New Revision: 267425

URL: http://svn.apache.org/viewcvs?rev=267425&view=rev
Log:
PR: MNG-795
create an empty fileset if there were no dependencies downloaded

Modified:
    maven/components/trunk/maven-artifact-ant/sample.build.xml
    maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java
    maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml

Modified: maven/components/trunk/maven-artifact-ant/sample.build.xml
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-ant/sample.build.xml?rev=267425&r1=267424&r2=267425&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-ant/sample.build.xml (original)
+++ maven/components/trunk/maven-artifact-ant/sample.build.xml Fri Sep  2 23:18:23 2005
@@ -28,6 +28,14 @@
     </typedef>
   </target>
 
+  <target name="test-no-deps" depends="initTaskDefs">
+     <artifact:dependencies filesetId="runtime.fileset">
+     </artifact:dependencies> 
+     <copy todir="target/test-no-deps" verbose="true">
+       <fileset refid="runtime.fileset" />
+     </copy>
+  </target>
+
   <target name="test-bad-dep" depends="initTaskDefs">
     <artifact:dependencies pathId="dependency.classpath" filesetId="dependency.fileset">
       <dependency groupId="foo" artifactId="foo" version="1.0-alpha-2"/>

Modified: maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java?rev=267425&r1=267424&r2=267425&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java (original)
+++ maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/DependenciesTask.java Fri Sep  2 23:18:23 2005
@@ -32,6 +32,7 @@
 import org.apache.maven.project.MavenProjectBuilder;
 import org.apache.maven.project.artifact.MavenMetadataSource;
 import org.apache.tools.ant.BuildException;
+import org.apache.tools.ant.Project;
 import org.apache.tools.ant.types.FileList;
 import org.apache.tools.ant.types.FileSet;
 import org.apache.tools.ant.types.Path;
@@ -102,6 +103,11 @@
             pom = createDummyPom();
         }
 
+        if ( dependencies.isEmpty() )
+        {
+            log( "There were no dependencies specified", Project.MSG_WARN );
+        }
+
         Set artifacts;
         try
         {
@@ -165,17 +171,24 @@
         FileSet fileSet = new FileSet();
         fileSet.setDir( fileList.getDir( getProject() ) );
 
-        for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
+        if ( result.getArtifacts().isEmpty() )
         {
-            Artifact artifact = (Artifact) i.next();
-            String filename = localRepo.pathOf( artifact );
-
-            FileList.FileName file = new FileList.FileName();
-            file.setName( filename );
-
-            fileList.addConfiguredFile( file );
-
-            fileSet.createInclude().setName( filename );
+            fileSet.createExclude().setName( "**/**" );
+        }
+        else
+        {
+            for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
+            {
+                Artifact artifact = (Artifact) i.next();
+                String filename = localRepo.pathOf( artifact );
+    
+                FileList.FileName file = new FileList.FileName();
+                file.setName( filename );
+    
+                fileList.addConfiguredFile( file );
+    
+                fileSet.createInclude().setName( filename );
+            }
         }
 
         if ( pathId != null )

Modified: maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml?rev=267425&r1=267424&r2=267425&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml (original)
+++ maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml Fri Sep  2 23:18:23 2005
@@ -192,11 +192,6 @@
           <role>org.apache.maven.artifact.transform.ArtifactTransformation</role>
           <field-name>artifactTransformations</field-name>
         </requirement>
-        <requirement>
-          <role>org.apache.maven.artifact.transform.ArtifactTransformation</role>
-          <role-hint>snapshot</role-hint>
-          <field-name>snapshotTransformation</field-name>
-        </requirement>
       </requirements>
     </component>
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org