You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by br...@apache.org on 2007/09/24 06:01:45 UTC

svn commit: r578641 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java

Author: brett
Date: Sun Sep 23 21:01:44 2007
New Revision: 578641

URL: http://svn.apache.org/viewvc?rev=578641&view=rev
Log:
handle a missing ~/.m2/settings.xml more gracefull

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java?rev=578641&r1=578640&r2=578641&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/settings/DefaultMavenSettingsBuilder.java Sun Sep 23 21:01:44 2007
@@ -134,7 +134,14 @@
     {
         if ( settingsFile == null )
         {
-            getLogger().debug( "Settings file is null. Returning." );
+            getLogger().debug( "Settings file is null. Returning null." );
+
+            return null;
+        }
+
+        if ( !settingsFile.exists() )
+        {
+            getLogger().debug( "Settings file doesn't exist. Returning null." );
 
             return null;
         }