You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2018/07/01 11:31:06 UTC
[maven-artifact-transfer] branch CODE-IMPROVEMENTS updated (9938700
-> fdf72a7)
This is an automated email from the ASF dual-hosted git repository.
khmarbaise pushed a change to branch CODE-IMPROVEMENTS
in repository https://gitbox.apache.org/repos/asf/maven-artifact-transfer.git.
discard 9938700 Removed outcommented code.
add 34ff96d Added .checkstyle to .gitignore based on Eclipse configuration.
add b900eea [MSHARED-724] - Upgrade mave-surefire/failsafe-plugin 2.21.0
add 5da6ce6 [INFRA-16467] move components documentation out of CMS space
add 4dc2562 [MSHARED-737] - Upgrade mockito-core to 2.18.3 JDK 10 support o Updated Jenkinsfile accordingly to build also JDK 10
add 7b1771c [MSHARED-695] - Move checksum generation from install to deploy o Moved checksum generation into deploy part. Removed checksum from install
add 2155fff Bumped version to 0.10.0-SNAPSHOT
add 305fdad [MSHARED-741] - Upgrade commons-codec to 1.11
add 3a01c3d [MSHARED-698] - Change the package to org.apache.maven.shared.transfer.*
new fdf72a7 Removed outcommented code.
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (9938700)
\
N -- N -- N refs/heads/CODE-IMPROVEMENTS (fdf72a7)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.gitignore | 1 +
Jenkinsfile | 5 +-
pom.xml | 11 +-
src/it/maven-artifact-deployer-plugin/pom.xml | 3 +-
.../artifact/deployer/ArtifactDeployerMojo.java | 6 +-
.../artifact/deployer/ArtifactDeployerTest.java | 2 +-
src/it/maven-artifact-installer-plugin/pom.xml | 10 +-
.../artifact/installer/ArtifactInstallerMojo.java | 6 +-
.../artifact/installer/ArtifactInstallerTest.java | 2 +-
src/it/maven-project-deployer-plugin/pom.xml | 3 +-
.../plugin/project/deploy/ProjectDeployerMojo.java | 12 +-
.../plugin/project/deploy/ProjectDeployerTest.java | 2 +-
src/it/maven-project-installer-plugin/pom.xml | 3 +-
.../project/install/ProjectInstallerMojo.java | 11 +-
.../project/install/ProjectInstallerTest.java | 33 ++--
.../deploy/internal/DefaultProjectDeployer.java | 189 ------------------
.../project/install/ProjectInstallerRequest.java | 93 ---------
.../artifact/ArtifactCoordinate.java | 2 +-
.../artifact/DefaultArtifactCoordinate.java | 2 +-
.../{ => transfer}/artifact/TransferUtils.java | 2 +-
.../artifact/deploy/ArtifactDeployer.java | 2 +-
.../artifact/deploy/ArtifactDeployerException.java | 2 +-
.../deploy/internal/DefaultArtifactDeployer.java | 6 +-
.../artifact/deploy/internal/Invoker.java | 4 +-
.../deploy/internal/Maven30ArtifactDeployer.java | 6 +-
.../deploy/internal/Maven31ArtifactDeployer.java | 6 +-
.../artifact/install/ArtifactInstaller.java | 2 +-
.../install/ArtifactInstallerException.java | 2 +-
.../install/internal/DefaultArtifactInstaller.java | 6 +-
.../artifact/install/internal/Invoker.java | 4 +-
.../install/internal/Maven30ArtifactInstaller.java | 8 +-
.../install/internal/Maven31ArtifactInstaller.java | 8 +-
.../artifact/resolve/ArtifactResolver.java | 4 +-
.../resolve/ArtifactResolverException.java | 2 +-
.../artifact/resolve/ArtifactResult.java | 2 +-
.../resolve/internal/DefaultArtifactResolver.java | 10 +-
.../artifact/resolve/internal/Invoker.java | 4 +-
.../resolve/internal/Maven30ArtifactResolver.java | 14 +-
.../resolve/internal/Maven30ArtifactResult.java | 8 +-
.../resolve/internal/Maven31ArtifactResolver.java | 14 +-
.../resolve/internal/Maven31ArtifactResult.java | 8 +-
.../dependencies/DefaultDependableCoordinate.java | 2 +-
.../dependencies/DependableCoordinate.java | 2 +-
.../{ => transfer}/dependencies/TransferUtils.java | 2 +-
.../dependencies/collect/CollectorResult.java | 2 +-
.../dependencies/collect/DependencyCollector.java | 4 +-
.../collect/DependencyCollectorException.java | 2 +-
.../internal/DefaultDependencyCollector.java | 10 +-
.../dependencies/collect/internal/Invoker.java | 4 +-
.../internal/Maven30ArtifactRepositoryAdapter.java | 2 +-
.../collect/internal/Maven30CollectorResult.java | 4 +-
.../internal/Maven30DependencyCollector.java | 10 +-
.../internal/Maven31ArtifactRepositoryAdapter.java | 2 +-
.../collect/internal/Maven31CollectorResult.java | 4 +-
.../internal/Maven31DependencyCollector.java | 10 +-
.../dependencies/resolve/DependencyResolver.java | 6 +-
.../resolve/DependencyResolverException.java | 2 +-
.../dependencies/resolve/DependencyResult.java | 2 +-
.../internal/DefaultDependencyResolver.java | 10 +-
.../dependencies/resolve/internal/Invoker.java | 4 +-
.../resolve/internal/Maven30ArtifactResult.java | 8 +-
.../internal/Maven30DependencyResolver.java | 24 +--
.../Maven30DependencyResolverException.java | 6 +-
.../resolve/internal/Maven31ArtifactResult.java | 8 +-
.../internal/Maven31DependencyResolver.java | 24 +--
.../Maven31DependencyResolverException.java | 6 +-
.../project/NoFileAssignedException.java | 2 +-
.../project/deploy/ProjectDeployer.java | 6 +-
.../project/deploy/ProjectDeployerRequest.java | 24 +--
.../deploy/internal/DefaultProjectDeployer.java} | 217 +++++++++++----------
.../project/deploy}/internal/DualDigester.java | 2 +-
.../project/install/ProjectInstaller.java | 10 +-
.../project/install/ProjectInstallerRequest.java} | 26 ++-
.../install/internal/DefaultProjectInstaller.java | 145 ++------------
.../repository/RepositoryManager.java | 4 +-
.../repository/RepositoryManagerException.java | 2 +-
.../internal/DefaultRepositoryManager.java | 8 +-
.../repository/internal/Invoker.java | 4 +-
.../internal/Maven302RepositoryManager.java | 4 +-
.../internal/Maven30RepositoryManager.java | 8 +-
.../internal/Maven31RepositoryManager.java | 8 +-
.../artifact/DefaultArtifactCoordinateTest.java | 3 +-
.../internal/DefaultArtifactDeployerTest.java | 8 +-
.../internal/DefaultArtifactInstallerTest.java | 7 +-
.../internal/Maven30ArtifactInstallerTest.java | 5 +-
.../internal/DefaultArtifactResolverTest.java | 11 +-
.../DefaultDependableCoordinateTest.java | 4 +-
.../internal/DefaultDependencyCollectorTest.java | 14 +-
.../internal/DefaultDependencyResolverTest.java | 9 +-
.../internal/DefaultProjectDeployerTest.java | 11 +-
.../internal/DefaultProjectInstallerTest.java | 9 +-
.../internal/Maven30RepositoryManagerTest.java | 6 +-
src/test/resources/test.jar | 1 +
93 files changed, 435 insertions(+), 803 deletions(-)
delete mode 100644 src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java
delete mode 100644 src/main/java/org/apache/maven/shared/project/install/ProjectInstallerRequest.java
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/ArtifactCoordinate.java (93%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/DefaultArtifactCoordinate.java (94%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/TransferUtils.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/ArtifactDeployer.java (97%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/ArtifactDeployerException.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/internal/DefaultArtifactDeployer.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/internal/Invoker.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/internal/Maven30ArtifactDeployer.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/internal/Maven31ArtifactDeployer.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/ArtifactInstaller.java (97%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/ArtifactInstallerException.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/DefaultArtifactInstaller.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/Invoker.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/Maven30ArtifactInstaller.java (93%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/Maven31ArtifactInstaller.java (93%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/ArtifactResolver.java (94%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/ArtifactResolverException.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/ArtifactResult.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/DefaultArtifactResolver.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/Invoker.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/Maven30ArtifactResolver.java (87%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/Maven30ArtifactResult.java (81%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/Maven31ArtifactResolver.java (87%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/Maven31ArtifactResult.java (81%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/DefaultDependableCoordinate.java (94%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/DependableCoordinate.java (94%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/TransferUtils.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/CollectorResult.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/DependencyCollector.java (93%)
copy src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/DependencyCollectorException.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/DefaultDependencyCollector.java (90%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Invoker.java (94%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven30ArtifactRepositoryAdapter.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven30CollectorResult.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven30DependencyCollector.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven31ArtifactRepositoryAdapter.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven31CollectorResult.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/Maven31DependencyCollector.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/DependencyResolver.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/DependencyResolverException.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/DependencyResult.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/DefaultDependencyResolver.java (91%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Invoker.java (93%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven30ArtifactResult.java (80%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven30DependencyResolver.java (88%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven30DependencyResolverException.java (85%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven31ArtifactResult.java (80%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven31DependencyResolver.java (88%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/Maven31DependencyResolverException.java (83%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/project/NoFileAssignedException.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/project/deploy/ProjectDeployer.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/project/deploy/ProjectDeployerRequest.java (83%)
copy src/main/java/org/apache/maven/shared/{project/install/internal/DefaultProjectInstaller.java => transfer/project/deploy/internal/DefaultProjectDeployer.java} (58%)
rename src/main/java/org/apache/maven/shared/{project/install => transfer/project/deploy}/internal/DualDigester.java (97%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/project/install/ProjectInstaller.java (90%)
rename src/main/java/org/apache/maven/shared/{dependencies/collect/DependencyCollectorException.java => transfer/project/install/ProjectInstallerRequest.java} (59%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/project/install/internal/DefaultProjectInstaller.java (52%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/RepositoryManager.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/RepositoryManagerException.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/internal/DefaultRepositoryManager.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/internal/Invoker.java (95%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/internal/Maven302RepositoryManager.java (92%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/internal/Maven30RepositoryManager.java (96%)
rename src/main/java/org/apache/maven/shared/{ => transfer}/repository/internal/Maven31RepositoryManager.java (96%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/artifact/DefaultArtifactCoordinateTest.java (90%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/artifact/deploy/internal/DefaultArtifactDeployerTest.java (92%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/DefaultArtifactInstallerTest.java (94%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/artifact/install/internal/Maven30ArtifactInstallerTest.java (93%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/artifact/resolve/internal/DefaultArtifactResolverTest.java (87%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/dependencies/DefaultDependableCoordinateTest.java (90%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/dependencies/collect/internal/DefaultDependencyCollectorTest.java (87%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/dependencies/resolve/internal/DefaultDependencyResolverTest.java (90%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/project/deploy/internal/DefaultProjectDeployerTest.java (86%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/project/install/internal/DefaultProjectInstallerTest.java (85%)
rename src/test/java/org/apache/maven/shared/{ => transfer}/repository/internal/Maven30RepositoryManagerTest.java (93%)
create mode 100644 src/test/resources/test.jar
[maven-artifact-transfer] 01/01: Removed outcommented code.
Posted by kh...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
khmarbaise pushed a commit to branch CODE-IMPROVEMENTS
in repository https://gitbox.apache.org/repos/asf/maven-artifact-transfer.git
commit fdf72a79b6cbf1ffe7ea02fad92c883222336648
Author: Karl Heinz Marbaise <kh...@apache.org>
AuthorDate: Sun Mar 4 12:36:23 2018 +0100
Removed outcommented code.
---
.../artifact/resolve/ArtifactResolver.java | 30 ----
.../resolve/internal/DefaultArtifactResolver.java | 40 ------
.../resolve/internal/Maven30ArtifactResolver.java | 157 --------------------
.../resolve/internal/Maven31ArtifactResolver.java | 158 ---------------------
4 files changed, 385 deletions(-)
diff --git a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/ArtifactResolver.java b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/ArtifactResolver.java
index 0491023..86b8330 100644
--- a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/ArtifactResolver.java
+++ b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/ArtifactResolver.java
@@ -51,34 +51,4 @@ public interface ArtifactResolver
ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate )
throws ArtifactResolverException, IllegalArgumentException;
- // /**
- // * This will resolve the dependencies of the coordinate, not resolving the the artifact of the coordinate itself.
- // * If the coordinate needs to be resolved too, use
- // * {@link #resolveDependencies(ProjectBuildingRequest, Collection, Collection, TransformableFilter)} passing
- // * {@code Collections.singletonList(coordinate)}
- // *
- // * @param buildingRequest {@link ProjectBuildingRequest}
- // * @param coordinate {@link ArtifactCoordinate}
- // * @param filter {@link TransformableFilter}
- // * @return the resolved dependencies.
- // * @throws ArtifactResolverException in case of an error.
- // */
- // Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest, ArtifactCoordinate
- // coordinate,
- // TransformableFilter filter )
- // throws ArtifactResolverException;
- //
- // /**
- // * @param buildingRequest the project building request, never {@code null}
- // * @param dependencies the dependencies to resolve, never {@code null}
- // * @param managedDependencies managed dependencies, can be {@code null}
- // * @param filter a filter, can be {@code null}
- // * @return the resolved dependencies.
- // * @throws ArtifactResolverException in case of an error.
- // */
- // Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
- // Collection<Dependency> dependencies,
- // Collection<Dependency> managedDependencies,
- // TransformableFilter filter )
- // throws ArtifactResolverException;
}
diff --git a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolver.java b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolver.java
index e778630..7e25f1f 100644
--- a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolver.java
+++ b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolver.java
@@ -104,46 +104,6 @@ class DefaultArtifactResolver
}
}
- // @Override
- // public Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
- // Collection<Dependency> coordinates,
- // Collection<Dependency> managedDependencies,
- // TransformableFilter filter )
- // throws ArtifactResolverException
- // {
- // try
- // {
- // String hint = isMaven31() ? "maven31" : "maven3";
- //
- // ArtifactResolver effectiveArtifactResolver = container.lookup( ArtifactResolver.class, hint );
- //
- // return effectiveArtifactResolver.resolveDependencies( buildingRequest, coordinates, null, filter );
- // }
- // catch ( ComponentLookupException e )
- // {
- // throw new ArtifactResolverException( e.getMessage(), e );
- // }
- // }
- //
- // @Override
- // public Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
- // ArtifactCoordinate coordinate, TransformableFilter filter )
- // throws ArtifactResolverException
- // {
- // try
- // {
- // String hint = isMaven31() ? "maven31" : "maven3";
- //
- // ArtifactResolver effectiveArtifactResolver = container.lookup( ArtifactResolver.class, hint );
- //
- // return effectiveArtifactResolver.resolveDependencies( buildingRequest, coordinate, filter );
- // }
- // catch ( ComponentLookupException e )
- // {
- // throw new ArtifactResolverException( e.getMessage(), e );
- // }
- // }
-
/**
* @return true if the current Maven version is Maven 3.1.
*/
diff --git a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven30ArtifactResolver.java b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven30ArtifactResolver.java
index fa9f7b3..f81b243 100644
--- a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven30ArtifactResolver.java
+++ b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven30ArtifactResolver.java
@@ -117,161 +117,4 @@ class Maven30ArtifactResolver
}
}
-// @Override
-// // CHECKSTYLE_OFF: LineLength
-// public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// ArtifactCoordinate coordinate,
-// TransformableFilter dependencyFilter )
-// // CHECKSTYLE_ON:
-// // LineLength
-// throws ArtifactResolverException
-// {
-// ArtifactTypeRegistry typeRegistry =
-// (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-// ArtifactHandlerManager.class, artifactHandlerManager );
-//
-// Dependency aetherRoot = toDependency( coordinate, typeRegistry );
-//
-// @SuppressWarnings( "unchecked" )
-// List<RemoteRepository> aetherRepositories =
-// (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
-// buildingRequest.getRemoteRepositories() );
-//
-// CollectRequest request = new CollectRequest( aetherRoot, aetherRepositories );
-//
-// return resolveDependencies( buildingRequest, aetherRepositories, dependencyFilter, request );
-// }
-//
-// @Override
-// // CHECKSTYLE_OFF: LineLength
-// public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// Collection<org.apache.maven.model.Dependency> mavenDependencies,
-// Collection<org.apache.maven.model.Dependency> managedMavenDependencies,
-// TransformableFilter filter )
-// // CHECKSTYLE_ON:
-// // LineLength
-// throws ArtifactResolverException
-// {
-// ArtifactTypeRegistry typeRegistry =
-// (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-// ArtifactHandlerManager.class, artifactHandlerManager );
-//
-// List<Dependency> aetherDependencies = new ArrayList<Dependency>( mavenDependencies.size() );
-//
-// final Class<?>[] argClasses =
-// new Class<?>[] { org.apache.maven.model.Dependency.class, ArtifactTypeRegistry.class };
-//
-// for ( org.apache.maven.model.Dependency mavenDependency : mavenDependencies )
-// {
-// Object[] args = new Object[] { mavenDependency, typeRegistry };
-//
-// Dependency aetherDependency =
-// (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args );
-//
-// aetherDependencies.add( aetherDependency );
-// }
-//
-// List<Dependency> aetherManagedDependencies = new ArrayList<Dependency>( managedMavenDependencies.size() );
-//
-// for ( org.apache.maven.model.Dependency mavenDependency : managedMavenDependencies )
-// {
-// Object[] args = new Object[] { mavenDependency, typeRegistry };
-//
-// Dependency aetherDependency =
-// (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args );
-//
-// aetherManagedDependencies.add( aetherDependency );
-// }
-//
-// @SuppressWarnings( "unchecked" )
-// List<RemoteRepository> aetherRepositories =
-// (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
-// buildingRequest.getRemoteRepositories() );
-//
-// CollectRequest request =
-// new CollectRequest( aetherDependencies, aetherManagedDependencies, aetherRepositories );
-//
-// return resolveDependencies( buildingRequest, aetherRepositories, filter, request );
-// }
-
-// // CHECKSTYLE_OFF: LineLength
-// private Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// List<RemoteRepository> aetherRepositories,
-// TransformableFilter dependencyFilter,
-// CollectRequest request )
-// throws ArtifactResolverException
-// // CHECKSTYLE_ON :LineLength
-// {
-// try
-// {
-// DependencyFilter depFilter = null;
-// if ( dependencyFilter != null )
-// {
-// depFilter = dependencyFilter.transform( new SonatypeAetherFilterTransformer() );
-// }
-//
-// RepositorySystemSession session =
-// (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
-//
-// List<ArtifactResult> dependencyResults =
-// repositorySystem.resolveDependencies( session, request, depFilter );
-//
-// Collection<ArtifactRequest> artifactRequests = new ArrayList<ArtifactRequest>( dependencyResults.size() );
-//
-// for ( ArtifactResult artifactResult : dependencyResults )
-// {
-// artifactRequests.add( new ArtifactRequest( artifactResult.getArtifact(), aetherRepositories, null ) );
-// }
-//
-// final List<ArtifactResult> artifactResults = repositorySystem.resolveArtifacts( session, artifactRequests );
-//
-// // Keep it lazy! Often artifactsResults aren't used, so transforming up front is too expensive
-// return new Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult>()
-// {
-// @Override
-// public Iterator<org.apache.maven.shared.artifact.resolve.ArtifactResult> iterator()
-// {
-// Collection<org.apache.maven.shared.artifact.resolve.ArtifactResult> artResults =
-// new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
-//
-// for ( ArtifactResult artifactResult : artifactResults )
-// {
-// artResults.add( new Maven30ArtifactResult( artifactResult ) );
-// }
-//
-// return artResults.iterator();
-// }
-// };
-// }
-// catch ( ArtifactResolutionException e )
-// {
-// throw new ArtifactResolverException( e.getMessage(), e );
-// }
-// catch ( DependencyCollectionException e )
-// {
-// throw new ArtifactResolverException( e.getMessage(), e );
-// }
-// }
-
-// /**
-// * Based on RepositoryUtils#toDependency(org.apache.maven.model.Dependency, ArtifactTypeRegistry)
-// *
-// * @param coordinate
-// * @param stereotypes
-// * @return as Aether Dependency
-// */
-// private static Dependency toDependency( ArtifactCoordinate coordinate, ArtifactTypeRegistry stereotypes )
-// {
-// ArtifactType stereotype = stereotypes.get( coordinate.getExtension() );
-// if ( stereotype == null )
-// {
-// stereotype = new DefaultArtifactType( coordinate.getExtension() );
-// }
-//
-// Artifact artifact =
-// new DefaultArtifact( coordinate.getGroupId(), coordinate.getArtifactId(), coordinate.getClassifier(), null,
-// coordinate.getVersion(), null, stereotype );
-//
-// return new Dependency( artifact, null );
-// }
}
diff --git a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven31ArtifactResolver.java b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven31ArtifactResolver.java
index b326e53..56e4519 100644
--- a/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven31ArtifactResolver.java
+++ b/src/main/java/org/apache/maven/shared/transfer/artifact/resolve/internal/Maven31ArtifactResolver.java
@@ -117,162 +117,4 @@ class Maven31ArtifactResolver
}
}
-// @Override
-// // CHECKSTYLE_OFF: LineLength
-// public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// ArtifactCoordinate coordinate,
-// TransformableFilter dependencyFilter )
-// throws ArtifactResolverException
-// // CHECKSTYLE_ON: LineLength
-// {
-// ArtifactTypeRegistry typeRegistry =
-// (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-// ArtifactHandlerManager.class, artifactHandlerManager );
-//
-// Dependency aetherRoot = toDependency( coordinate, typeRegistry );
-//
-// @SuppressWarnings( "unchecked" )
-// List<RemoteRepository> aetherRepositories =
-// (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
-// buildingRequest.getRemoteRepositories() );
-//
-// CollectRequest request = new CollectRequest( aetherRoot, aetherRepositories );
-//
-// return resolveDependencies( buildingRequest, aetherRepositories, dependencyFilter, request );
-// }
-//
-// @Override
-// // CHECKSTYLE_OFF: LineLength
-// public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// Collection<org.apache.maven.model.Dependency> mavenDependencies,
-// Collection<org.apache.maven.model.Dependency> managedMavenDependencies,
-// TransformableFilter filter )
-// throws ArtifactResolverException
-// // CHECKSTYLE_ON: LineLength
-// {
-// ArtifactTypeRegistry typeRegistry =
-// (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-// ArtifactHandlerManager.class, artifactHandlerManager );
-//
-// List<Dependency> aetherDeps = new ArrayList<Dependency>( mavenDependencies.size() );
-//
-// final Class<?>[] argClasses =
-// new Class<?>[] { org.apache.maven.model.Dependency.class, ArtifactTypeRegistry.class };
-//
-// for ( org.apache.maven.model.Dependency mavenDependency : mavenDependencies )
-// {
-// Object[] args = new Object[] { mavenDependency, typeRegistry };
-//
-// Dependency aetherDependency =
-// (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args );
-//
-// aetherDeps.add( aetherDependency );
-// }
-//
-// List<Dependency> aetherManagedDeps = new ArrayList<Dependency>( managedMavenDependencies.size() );
-//
-// for ( org.apache.maven.model.Dependency mavenDependency : managedMavenDependencies )
-// {
-// Object[] args = new Object[] { mavenDependency, typeRegistry };
-//
-// Dependency aetherDependency =
-// (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args );
-//
-// aetherManagedDeps.add( aetherDependency );
-// }
-//
-// @SuppressWarnings( "unchecked" )
-// List<RemoteRepository> aetherRepos =
-// (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
-// buildingRequest.getRemoteRepositories() );
-//
-// CollectRequest request = new CollectRequest( aetherDeps, aetherManagedDeps, aetherRepos );
-//
-// return resolveDependencies( buildingRequest, aetherRepos, filter, request );
-// }
-
-// // CHECKSTYLE_OFF: LineLength
-// private Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-// List<RemoteRepository> aetherRepositories,
-// TransformableFilter dependencyFilter,
-// CollectRequest request )
-// throws ArtifactResolverException
-// // CHECKSTYLE_ON: LineLength
-// {
-// try
-// {
-// DependencyFilter depFilter = null;
-// if ( dependencyFilter != null )
-// {
-// depFilter = dependencyFilter.transform( new EclipseAetherFilterTransformer() );
-// }
-//
-// DependencyRequest depRequest = new DependencyRequest( request, depFilter );
-//
-// RepositorySystemSession session =
-// (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
-//
-// DependencyResult dependencyResults = repositorySystem.resolveDependencies( session, depRequest );
-//
-// Collection<ArtifactRequest> artifactRequests =
-// new ArrayList<ArtifactRequest>( dependencyResults.getArtifactResults().size() );
-//
-// for ( ArtifactResult artifactResult : dependencyResults.getArtifactResults() )
-// {
-// artifactRequests.add( new ArtifactRequest( artifactResult.getArtifact(), aetherRepositories, null ) );
-// }
-//
-// final List<ArtifactResult> artifactResults = repositorySystem.resolveArtifacts( session, artifactRequests );
-//
-// // Keep it lazy! Often artifactsResults aren't used, so transforming up front is too expensive
-// return new Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult>()
-// {
-// @Override
-// public Iterator<org.apache.maven.shared.artifact.resolve.ArtifactResult> iterator()
-// {
-// // CHECKSTYLE_OFF: LineLength
-// Collection<org.apache.maven.shared.artifact.resolve.ArtifactResult> artResults =
-// new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
-// // CHECKSTYLE_ON: LineLength
-//
-// for ( ArtifactResult artifactResult : artifactResults )
-// {
-// artResults.add( new Maven31ArtifactResult( artifactResult ) );
-// }
-//
-// return artResults.iterator();
-// }
-// };
-// }
-// catch ( ArtifactResolutionException e )
-// {
-// throw new ArtifactResolverException( e.getMessage(), e );
-// }
-// catch ( DependencyResolutionException e )
-// {
-// throw new ArtifactResolverException( e.getMessage(), e );
-// }
-// }
-
-// /**
-// * Based on RepositoryUtils#toDependency(org.apache.maven.model.Dependency, ArtifactTypeRegistry)
-// *
-// * @param coordinate {@link ArtifactCoordinate}
-// * @param stereotypes {@link ArtifactTypeRegistry
-// * @return as Aether Dependency
-// */
-// private static Dependency toDependency( ArtifactCoordinate coordinate, ArtifactTypeRegistry stereotypes )
-// {
-// ArtifactType stereotype = stereotypes.get( coordinate.getExtension() );
-// if ( stereotype == null )
-// {
-// stereotype = new DefaultArtifactType( coordinate.getExtension() );
-// }
-//
-// Artifact artifact =
-// new DefaultArtifact( coordinate.getGroupId(), coordinate.getArtifactId(), coordinate.getClassifier(), null,
-// coordinate.getVersion(), null, stereotype );
-//
-// return new Dependency( artifact, null );
-// }
}