Author: vsiveton
Date: Fri Jan  4 11:16:47 2008
New Revision: 608961

URL: http://svn.apache.org/viewvc?rev=608961&view=rev
Log:
MNG-3288: Invalid systemPath allows build to continue--failing in later phase.

o verify that the system-scoped artifact is a file

Modified:
    
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java

Modified: 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
URL: 
http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=608961&r1=608960&r2=608961&view=diff
==============================================================================
--- 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
 (original)
+++ 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
 Fri Jan  4 11:16:47 2008
@@ -98,15 +98,20 @@
                     "System artifact: " + artifact + " has no file attached", 
artifact );
             }
 
-            if ( !systemFile.exists() )
+            if ( !systemFile.isFile() )
             {
-                throw new ArtifactNotFoundException(
-                    "System artifact: " + artifact + " not found in path: " + 
systemFile, artifact );
+                throw new ArtifactNotFoundException( "System artifact: " + 
artifact
+                    + " is not a file: " + systemFile, artifact );
             }
-            else
+
+            if ( !systemFile.exists() )
             {
-                artifact.setResolved( true );
+                throw new ArtifactNotFoundException(
+                    "System artifact: " + artifact + " not found in path: " + 
systemFile,
+                    artifact );
             }
+
+            artifact.setResolved( true );
         }
         else if ( !artifact.isResolved() )
         {


Reply via email to