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 2011/02/13 16:02:27 UTC

svn commit: r1070247 - /maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java

Author: bentmann
Date: Sun Feb 13 15:02:27 2011
New Revision: 1070247

URL: http://svn.apache.org/viewvc?rev=1070247&view=rev
Log:
o Revised error handling to properly notify caller of some serious trouble

Modified:
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java?rev=1070247&r1=1070246&r2=1070247&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java Sun Feb 13 15:02:27 2011
@@ -360,10 +360,7 @@ public class MavenProject
                 }
                 catch ( ProjectBuildingException e )
                 {
-                    if ( logger != null )
-                    {
-                        logger.debug( "Failed to build parent project for " + getId(), e );
-                    }
+                    throw new IllegalStateException( "Failed to build parent project for " + getId(), e );
                 }
             }
             else if ( model.getParent() != null )
@@ -378,10 +375,7 @@ public class MavenProject
                 }
                 catch ( ProjectBuildingException e )
                 {
-                    if ( logger != null )
-                    {
-                        logger.debug( "Failed to build parent project for " + getId(), e );
-                    }
+                    throw new IllegalStateException( "Failed to build parent project for " + getId(), e );
                 }
             }
         }
@@ -1856,10 +1850,8 @@ public class MavenProject
                 }
                 catch ( InvalidRepositoryException e )
                 {
-                    if ( logger != null )
-                    {
-                        logger.debug( "Failed to create release distribution repository for " + getId(), e );
-                    }
+                    throw new IllegalStateException( "Failed to create release distribution repository for " + getId(),
+                                                     e );
                 }
             }
         }
@@ -1886,10 +1878,9 @@ public class MavenProject
                 }
                 catch ( InvalidRepositoryException e )
                 {
-                    if ( logger != null )
-                    {
-                        logger.debug( "Failed to create snapshot distribution repository for " + getId(), e );
-                    }
+                    throw new IllegalStateException(
+                                                     "Failed to create snapshot distribution repository for " + getId(),
+                                                     e );
                 }
             }
         }