You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2010/01/16 19:24:37 UTC

svn commit: r900009 - /maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java

Author: bentmann
Date: Sat Jan 16 18:24:37 2010
New Revision: 900009

URL: http://svn.apache.org/viewvc?rev=900009&view=rev
Log:
[MASSEMBLY-289] Fix bogus warning about attaching non-regular file
Submitted by: Patrick Woodworth

Modified:
    maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java

Modified: maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java?rev=900009&r1=900008&r2=900009&view=diff
==============================================================================
--- maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java (original)
+++ maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java Sat Jan 16 18:24:37 2010
@@ -418,7 +418,7 @@
                             projectHelper.attachArtifact( project, format, null, destFile );
                         }
                     }
-                    else
+                    else if ( attach )
                     {
                         getLog().warn( "Assembly file: " + destFile + " is not a regular file (it may be a directory). It cannot be attached to the project build for installation or deployment." );
                     }