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 2009/07/19 23:24:44 UTC

svn commit: r795622 - /maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java

Author: bentmann
Date: Sun Jul 19 21:24:44 2009
New Revision: 795622

URL: http://svn.apache.org/viewvc?rev=795622&view=rev
Log:
o Revised error handling, API misuse is really not a model/network problem

Modified:
    maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java

Modified: maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java?rev=795622&r1=795621&r2=795622&view=diff
==============================================================================
--- maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java (original)
+++ maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java Sun Jul 19 21:24:44 2009
@@ -488,10 +488,8 @@
 
         if ( modelResolver == null )
         {
-            problems.add( new ModelProblem( "Non-resolvable parent POM " + toId( parent ) + " for POM "
-                + toSourceHint( childModel ) + ": " + "No model resolver provided", ModelProblem.Severity.FATAL,
-                                            toSourceHint( childModel ) ) );
-            throw new ModelBuildingException( problems );
+            throw new IllegalArgumentException( "no model resolver provided, cannot resolve parent POM "
+                + toId( parent ) + " for POM " + toSourceHint( childModel ) );
         }
 
         ModelSource modelSource;