This is an automated email from the ASF dual-hosted git repository. michaelo pushed a commit to branch maven-resolver-1.4.0 in repository https://gitbox.apache.org/repos/asf/maven.git
commit 69d5a55a8ac1744e8b7f8d8bec27fc36c72d6105 Author: Michael Osipov <micha...@apache.org> AuthorDate: Sat Jun 8 21:31:01 2019 +0200 Upgrade to Maven Resolver 1.4.0 --- .../internal/DefaultVersionResolver.java | 41 ++++++++++++++++++---- pom.xml | 2 +- 2 files changed, 35 insertions(+), 8 deletions(-) diff --git a/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java b/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java index 502aba1..d307c55 100644 --- a/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java +++ b/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java @@ -35,7 +35,6 @@ import org.eclipse.aether.impl.MetadataResolver; import org.eclipse.aether.impl.RepositoryEventDispatcher; import org.eclipse.aether.impl.SyncContextFactory; import org.eclipse.aether.impl.VersionResolver; -import org.eclipse.aether.internal.impl.CacheUtils; import org.eclipse.aether.metadata.DefaultMetadata; import org.eclipse.aether.metadata.Metadata; import org.eclipse.aether.repository.ArtifactRepository; @@ -154,7 +153,7 @@ public class DefaultVersionResolver Record record = (Record) obj; result.setVersion( record.version ); result.setRepository( - CacheUtils.getRepository( session, request.getRepositories(), record.repoClass, record.repoId ) ); + getRepository( session, request.getRepositories(), record.repoClass, record.repoId ) ); return result; } } @@ -429,6 +428,34 @@ public class DefaultVersionResolver return StringUtils.clean( classifier ) + ':' + StringUtils.clean( extension ); } + private ArtifactRepository getRepository( RepositorySystemSession session, + List<RemoteRepository> repositories, Class<?> repoClass, + String repoId ) + { + if ( repoClass != null ) + { + if ( WorkspaceRepository.class.isAssignableFrom( repoClass ) ) + { + return session.getWorkspaceReader().getRepository(); + } + else if ( LocalRepository.class.isAssignableFrom( repoClass ) ) + { + return session.getLocalRepository(); + } + else + { + for ( RemoteRepository repository : repositories ) + { + if ( repoId.equals( repository.getId() ) ) + { + return repository; + } + } + } + } + return null; + } + private boolean isSafelyCacheable( RepositorySystemSession session, Artifact artifact ) { /* @@ -502,7 +529,8 @@ public class DefaultVersionResolver extension = artifact.getExtension(); version = artifact.getVersion(); localRepo = session.getLocalRepository().getBasedir(); - workspace = CacheUtils.getWorkspace( session ); + WorkspaceReader reader = session.getWorkspaceReader(); + workspace = ( reader != null ) ? reader.getRepository() : null; repositories = new ArrayList<>( request.getRepositories().size() ); boolean repoMan = false; for ( RemoteRepository repository : request.getRepositories() ) @@ -526,7 +554,7 @@ public class DefaultVersionResolver hash = hash * 31 + extension.hashCode(); hash = hash * 31 + version.hashCode(); hash = hash * 31 + localRepo.hashCode(); - hash = hash * 31 + CacheUtils.repositoriesHashCode( repositories ); + hash = hash * 31 + repositories.hashCode(); hashCode = hash; } @@ -546,8 +574,7 @@ public class DefaultVersionResolver return artifactId.equals( that.artifactId ) && groupId.equals( that.groupId ) && classifier.equals( that.classifier ) && extension.equals( that.extension ) && version.equals( that.version ) && context.equals( that.context ) && localRepo.equals( that.localRepo ) - && CacheUtils.eq( workspace, that.workspace ) - && CacheUtils.repositoriesEquals( repositories, that.repositories ); + && Objects.equals( workspace, that.workspace ) && repositories.equals( that.repositories ); } @Override @@ -582,4 +609,4 @@ public class DefaultVersionResolver } } -} \ No newline at end of file +} diff --git a/pom.xml b/pom.xml index cb9dae5..5d016ef 100644 --- a/pom.xml +++ b/pom.xml @@ -64,7 +64,7 @@ under the License. <cipherVersion>1.7</cipherVersion> <modelloVersion>1.10.0</modelloVersion> <jxpathVersion>1.3</jxpathVersion> - <resolverVersion>1.3.3</resolverVersion> + <resolverVersion>1.4.0-SNAPSHOT</resolverVersion> <slf4jVersion>1.7.25</slf4jVersion> <xmlunitVersion>2.2.1</xmlunitVersion> <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>