You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/05/04 15:42:54 UTC

svn commit: r168123 - /maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java

Author: brett
Date: Wed May  4 06:42:51 2005
New Revision: 168123

URL: http://svn.apache.org/viewcvs?rev=168123&view=rev
Log:
destinationFile was not necessarily the resultant file in the parent method. Check the return status instead

Modified:
    maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java

Modified: maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java?rev=168123&r1=168122&r2=168123&view=diff
==============================================================================
--- maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java (original)
+++ maven/components/trunk/maven-mboot2/src/main/java/download/ArtifactDownloader.java Wed May  4 06:42:51 2005
@@ -92,9 +92,7 @@
                     continue;
                 }
 
-                getRemoteArtifact( dep, destinationFile );
-
-                if ( !destinationFile.exists() )
+                if ( !getRemoteArtifact( dep, destinationFile ) )
                 {
                     throw new DownloadFailedException( "Failed to download " + dep );
                 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org