svn commit: r1368246 - /maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java

[hboutemy]

Author: hboutemy
Date: Wed Aug  1 20:58:44 2012
New Revision: 1368246

URL: http://svn.apache.org/viewvc?rev=1368246&view=rev
Log:
[MDEP-187] improved error message to describe the real cause of the problem

Modified:
    
maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java

Modified: 
maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java?rev=1368246&r1=1368245&r2=1368246&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java
 Wed Aug  1 20:58:44 2012
@@ -41,6 +41,7 @@ import org.codehaus.plexus.util.Reflecti
 import org.codehaus.plexus.util.StringUtils;
 
 import java.io.File;
+import java.io.IOException;
 import java.lang.reflect.Field;
 import java.util.List;
 
@@ -169,10 +170,16 @@ public abstract class AbstractDependency
             theLog.info(
                 "Copying " + ( this.outputAbsoluteArtifactFilename ? 
artifact.getAbsolutePath() : artifact.getName() )
                     + " to " + destFile );
-            FileUtils.copyFile( artifact, destFile );
 
+            if ( artifact.isDirectory() )
+            {
+                throw new MojoExecutionException( "Artifact has not been 
packaged yet. When used on reactor artifact, "
+                    + "copy should be executed after packaging: see MDEP-187." 
);
+            }
+
+            FileUtils.copyFile( artifact, destFile );
         }
-        catch ( Exception e )
+        catch ( IOException e )
         {
             throw new MojoExecutionException( "Error copying artifact from " + 
artifact + " to " + destFile, e );
         }