Author: gboue
Date: Sat Oct 8 18:43:58 2016
New Revision: 1763929
URL: http://svn.apache.org/viewvc?rev=1763929&view=rev
Log:
[MSHARED-595] In DefaultProjectInstaller, the path to the local
repository should be retrieved from the RepositoryManager
We need to rely on the RepositoryManager to get a hold of the local
repository base directory.
Modified:
maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/install/internal/DefaultProjectInstaller.java
Modified:
maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/install/internal/DefaultProjectInstaller.java
URL:
http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/install/internal/DefaultProjectInstaller.java?rev=1763929&r1=1763928&r2=1763929&view=diff
==============================================================================
---
maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/install/internal/DefaultProjectInstaller.java
(original)
+++
maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/install/internal/DefaultProjectInstaller.java
Sat Oct 8 18:43:58 2016
@@ -101,7 +101,7 @@ public class DefaultProjectInstaller
{
installer.install( buildingRequest,
Collections.<Artifact>singletonList( new ProjectArtifact( project )
) );
- installChecksums( buildingRequest,
artifactRepository, artifact, createChecksum );
+ installChecksums( buildingRequest, artifact,
createChecksum );
addMetaDataFilesForArtifact( artifactRepository,
artifact, metadataFiles, createChecksum );
}
}
@@ -120,7 +120,7 @@ public class DefaultProjectInstaller
if ( file != null && file.isFile() )
{
installer.install( buildingRequest,
Collections.<Artifact>singletonList( artifact ) );
- installChecksums( buildingRequest,
artifactRepository, artifact, createChecksum );
+ installChecksums( buildingRequest, artifact,
createChecksum );
addMetaDataFilesForArtifact( artifactRepository,
artifact, metadataFiles, createChecksum );
}
else if ( !attachedArtifacts.isEmpty() )
@@ -139,7 +139,7 @@ public class DefaultProjectInstaller
for ( Artifact attached : attachedArtifacts )
{
installer.install( buildingRequest,
Collections.singletonList( attached ) );
- installChecksums( buildingRequest, artifactRepository,
attached, createChecksum );
+ installChecksums( buildingRequest, attached,
createChecksum );
addMetaDataFilesForArtifact( artifactRepository,
attached, metadataFiles, createChecksum );
}
@@ -153,12 +153,12 @@ public class DefaultProjectInstaller
* the original POM file (cf. MNG-2820). While the plugin
currently requires Maven 2.0.6, we continue to hash the
* installed POM for robustness with regard to future changes
like re-introducing some kind of POM filtering.
*
+ * @param buildingRequest The project building request, must
not be <code>null</code>.
* @param artifact The artifact for which to create checksums,
must not be <code>null</code>.
* @param createChecksum {@code true} if checksum should be
created, otherwise {@code false}.
* @throws IOException If the checksums could not be installed.
*/
- private void installChecksums( ProjectBuildingRequest
buildingRequest, ArtifactRepository artifactRepository,
- Artifact artifact, boolean
createChecksum )
+ private void installChecksums( ProjectBuildingRequest
buildingRequest, Artifact artifact, boolean createChecksum )
throws IOException
{
if ( !createChecksum )
@@ -166,7 +166,7 @@ public class DefaultProjectInstaller
return;
}
- File artifactFile = getLocalRepoFile( buildingRequest,
artifactRepository, artifact );
+ File artifactFile = getLocalRepoFile( buildingRequest,
artifact );
installChecksums( artifactFile );
}
@@ -257,14 +257,14 @@ public class DefaultProjectInstaller
* Gets the path of the specified artifact within the local
repository. Note that the returned path need not exist
* (yet).
*
+ * @param buildingRequest The project building request, must
not be <code>null</code>.
* @param artifact The artifact whose local repo path should be
determined, must not be <code>null</code>.
* @return The absolute path to the artifact when installed,
never <code>null</code>.
*/
- private File getLocalRepoFile( ProjectBuildingRequest
buildingRequest, ArtifactRepository artifactRepository,
- Artifact artifact )
+ private File getLocalRepoFile( ProjectBuildingRequest
buildingRequest, Artifact artifact )
{
String path = repositoryManager.getPathForLocalArtifact(
buildingRequest, artifact );
- return new File( artifactRepository.getBasedir(), path );
+ return new File(
repositoryManager.getLocalRepositoryBasedir( buildingRequest ),
path );
}
/**