mirror of https://github.com/apache/maven.git
o Restored original behavior of legacy resolution methods (every client and not just Surefire that calls the old methods expects the root artifact to not be resolved as in 2.x now)
git-svn-id: https://svn.apache.org/repos/asf/maven/components/branches/MNG-2766@778161 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
22b4d0ee5f
commit
8b22b2dffa
|
@ -329,6 +329,7 @@ public ArtifactResolutionResult resolveTransitively( Set<Artifact> artifacts, Ar
|
|||
{
|
||||
ArtifactResolutionRequest request = new ArtifactResolutionRequest()
|
||||
.setArtifact( originatingArtifact )
|
||||
.setResolveRoot( false )
|
||||
// This is required by the surefire plugin
|
||||
.setArtifactDependencies( artifacts )
|
||||
.setManagedVersionMap( managedVersions )
|
||||
|
@ -358,11 +359,6 @@ public ArtifactResolutionResult resolveWithExceptions( ArtifactResolutionRequest
|
|||
//
|
||||
// ------------------------------------------------------------------------
|
||||
|
||||
private boolean isDummy( ArtifactResolutionRequest request )
|
||||
{
|
||||
return request.getArtifact().getArtifactId().equals( "dummy" ) && request.getArtifact().getGroupId().equals( "dummy" );
|
||||
}
|
||||
|
||||
public ArtifactResolutionResult resolve( ArtifactResolutionRequest request )
|
||||
{
|
||||
Artifact rootArtifact = request.getArtifact();
|
||||
|
@ -387,14 +383,6 @@ public ArtifactResolutionResult resolve( ArtifactResolutionRequest request )
|
|||
}
|
||||
}
|
||||
|
||||
// This is an extreme hack because of the ridiculous APIs we have a root that is disconnected and
|
||||
// useless. The SureFire plugin passes in a dummy root artifact that is actually used in the production
|
||||
// plugin ... We have no choice but to put this hack in the core.
|
||||
if ( isDummy( request ) )
|
||||
{
|
||||
request.setResolveRoot( false );
|
||||
}
|
||||
|
||||
if ( listeners == null )
|
||||
{
|
||||
listeners = new ArrayList<ResolutionListener>();
|
||||
|
@ -513,7 +501,7 @@ public ArtifactResolutionResult resolve( ArtifactResolutionRequest request )
|
|||
|
||||
// We want to send the root artifact back in the result but we need to do this after the other dependencies
|
||||
// have been resolved.
|
||||
if ( request.isResolveRoot() && !isDummy( request ) )
|
||||
if ( request.isResolveRoot() )
|
||||
{
|
||||
// Add the root artifact (as the first artifact to retain logical order of class path!)
|
||||
Set<Artifact> allArtifacts = new LinkedHashSet<Artifact>();
|
||||
|
|
|
@ -113,7 +113,7 @@ public void testTransitiveResolutionWhereAllArtifactsArePresentInTheLocalReposit
|
|||
|
||||
ArtifactResolutionResult result = artifactResolver.resolveTransitively( Collections.singleton( g ), projectArtifact, remoteRepositories(), localRepository(), null );
|
||||
|
||||
assertEquals( 3, result.getArtifacts().size() );
|
||||
assertEquals( 2, result.getArtifacts().size() );
|
||||
|
||||
assertTrue( result.getArtifacts().contains( g ) );
|
||||
|
||||
|
@ -135,7 +135,7 @@ public void testTransitiveResolutionWhereAllArtifactsAreNotPresentInTheLocalRepo
|
|||
|
||||
ArtifactResolutionResult result = artifactResolver.resolveTransitively( Collections.singleton( i ), projectArtifact, remoteRepositories(), localRepository(), null );
|
||||
|
||||
assertEquals( 3, result.getArtifacts().size() );
|
||||
assertEquals( 2, result.getArtifacts().size() );
|
||||
|
||||
assertTrue( result.getArtifacts().contains( i ) );
|
||||
|
||||
|
@ -231,7 +231,6 @@ public Artifact retrieveRelocatedArtifact( Artifact artifact,
|
|||
artifactResolver.resolveTransitively( set, projectArtifact, remoteRepositories(), localRepository(), mds );
|
||||
|
||||
Iterator i = result.getArtifacts().iterator();
|
||||
i.next();
|
||||
assertEquals( "n should be first", n, i.next() );
|
||||
assertEquals( "m should be second", m, i.next() );
|
||||
|
||||
|
@ -244,7 +243,6 @@ public Artifact retrieveRelocatedArtifact( Artifact artifact,
|
|||
artifactResolver.resolveTransitively( set, projectArtifact, remoteRepositories(), localRepository(), mds );
|
||||
|
||||
i = result.getArtifacts().iterator();
|
||||
i.next();
|
||||
assertEquals( "m should be first", m, i.next() );
|
||||
assertEquals( "n should be second", n, i.next() );
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue