Author: brett
Date: Mon Feb 6 21:51:14 2006
New Revision: 375497
URL: http://svn.apache.org/viewcvs?rev=375497&view=rev
Log:
[MNG-1908] correct error introduced in last rev with locally installed snapshots
Modified:
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
Modified:
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
URL:
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java?rev=375497&r1=375496&r2=375497&view=diff
==============================================================================
---
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
(original)
+++
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java
Mon Feb 6 21:51:14 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/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
URL:
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java?rev=375497&r1=375496&r2=375497&view=diff
==============================================================================
---
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
(original)
+++
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
Mon Feb 6 21:51:14 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/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
URL:
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=375497&r1=375496&r2=375497&view=diff
==============================================================================
---
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
(original)
+++
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
Mon Feb 6 21:51:14 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.",
