You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2008/09/15 12:47:06 UTC
svn commit: r695413 -
/maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java
Author: dennisl
Date: Mon Sep 15 03:47:06 2008
New Revision: 695413
URL: http://svn.apache.org/viewvc?rev=695413&view=rev
Log:
[MONE-6] The install goal should not assume that the "pom" packaging is only only one that does not have a primary artifact
Submitted by: Jason Dillon
Reviewed by: Dennis Lundberg
Modified:
maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java
Modified: maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java?rev=695413&r1=695412&r2=695413&view=diff
==============================================================================
--- maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java (original)
+++ maven/plugins/trunk/maven-one-plugin/src/main/java/org/apache/maven/plugins/mavenone/MavenOneRepositoryInstallMojo.java Mon Sep 15 03:47:06 2008
@@ -150,10 +150,12 @@
File file = artifact.getFile();
if ( file == null )
{
- throw new MojoExecutionException(
- "The packaging for this project did not assign a file to the build artifact" );
+ getLog().warn( "The packaging for this project did not assign a file to the build artifact" );
+ }
+ else
+ {
+ installer.install( file, artifact, localRepository );
}
- installer.install( file, artifact, localRepository );
}
if ( attachedArtifacts != null && !attachedArtifacts.isEmpty() )