Author: bentmann
Date: Fri Aug 28 08:55:27 2009
New Revision: 808819

URL: http://svn.apache.org/viewvc?rev=808819&view=rev
Log:
o Fail properly when mojo requires project but there is none

Modified:
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java?rev=808819&r1=808818&r2=808819&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
 Fri Aug 28 08:55:27 2009
@@ -417,11 +417,17 @@
     {
         MojoDescriptor mojoDescriptor = mojoExecution.getMojoDescriptor();
 
+        if ( mojoDescriptor.isProjectRequired() && 
!session.isUsingPOMsFromFilesystem() )
+        {
+            throw new MojoExecutionException( "Goal " + mojoDescriptor.getId()
+                + " requires a project to execute but there is no POM in this 
build." );
+        }
+
         if ( mojoDescriptor.isOnlineRequired() && session.isOffline() )
         {
             if ( MojoExecution.Source.CLI.equals( mojoExecution.getSource() ) )
             {
-                throw new MojoExecutionException( "Goal " + 
mojoDescriptor.getGoal()
+                throw new MojoExecutionException( "Goal " + 
mojoDescriptor.getId()
                     + " requires online mode for execution but Maven is 
currently offline." );
             }
             else


Reply via email to