Author: brett
Date: Mon Feb  6 22:07:13 2006
New Revision: 375504

URL: http://svn.apache.org/viewcvs?rev=375504&view=rev
Log:
[MNG-1908] merge r375497 to branch

Modified:
    
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
    
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
    
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/manager/DefaultWagonManager.java
URL: 
http://svn.apache.org/viewcvs/maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java?rev=375504&r1=375503&r2=375504&view=diff
==============================================================================
--- 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
 (original)
+++ 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
 Mon Feb  6 22:07:13 2006
@@ -255,7 +255,8 @@
             }
         }
 
-        if ( !successful )
+        // if it already exists locally we were just trying to force it - 
ignore the update
+        if ( !successful && !artifact.getFile().exists() )
         {
             throw new ResourceDoesNotExistException( "Unable to download the 
artifact from any repository" );
         }

Modified: 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
URL: 
http://svn.apache.org/viewcvs/maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java?rev=375504&r1=375503&r2=375504&view=diff
==============================================================================
--- 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
 (original)
+++ 
maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
 Mon Feb  6 22:07:13 2006
@@ -78,10 +78,11 @@
                     File file = new File( localRepository.getBasedir(),
                                           
localRepository.pathOfLocalRepositoryMetadata( metadata, repository ) );
 
-                    boolean checkForUpdates = policy.checkOutOfDate( new Date( 
file.lastModified() ) ) || !file.exists();
+                    boolean checkForUpdates =
+                        policy.checkOutOfDate( new Date( file.lastModified() ) 
) || !file.exists();
 
                     boolean metadataIsEmpty = true;
-                    
+
                     if ( checkForUpdates )
                     {
                         getLogger().info( metadata.getKey() + ": checking for 
updates from " + repository.getId() );
@@ -201,6 +202,8 @@
 
                     if ( !m.getVersioning().getSnapshot().isLocalCopy() )
                     {
+                        // TODO: I think this is incorrect (it results in 
localCopy set in a remote profile). Probably
+                        //   harmless so not removing at this point until full 
tests in place.
                         m.getVersioning().getSnapshot().setLocalCopy( true );
                         metadata.setMetadata( m );
                         metadata.storeInLocalRepository( localRepository, 
repository );
@@ -372,7 +375,7 @@
             getLogger().info( "Repository '" + repository.getId() + "' will be 
blacklisted" );
             getLogger().debug( "Exception", e );
             repository.setBlacklisted( allowBlacklisting );
-            
+
             throw e;
         }
     }

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/viewcvs/maven/components/branches/maven-2.0.x/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=375504&r1=375503&r2=375504&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
 Mon Feb  6 22:07:13 2006
@@ -152,7 +152,7 @@
                             wagonManager.getArtifact( artifact, repositories );
                         }
 
-                        if ( !artifact.isResolved() )
+                        if ( !artifact.isResolved() && !destination.exists() )
                         {
                             throw new ArtifactResolutionException(
                                 "Failed to resolve artifact, possibly due to a 
repository list that is not appropriately equipped for this artifact's 
metadata.",


Reply via email to