laeubi commented on a change in pull request #667:
URL: https://github.com/apache/maven/pull/667#discussion_r795526006
##########
File path: maven-core/src/main/java/org/apache/maven/DefaultMaven.java
##########
@@ -367,11 +354,37 @@ private MavenExecutionResult doExecute(
MavenExecutionRequest request, MavenSess
return result;
}
- private void afterSessionStart( MavenSession session )
+ private void finishRepositorySystemSession( MavenSession session,
DefaultRepositorySystemSession repoSession )
+ throws ComponentLookupException
+ {
+ WorkspaceReader reactorWorkspace = container.lookup(
WorkspaceReader.class, ReactorReader.HINT );
+ //
+ // Desired order of precedence for local artifact repositories
+ //
+ // Reactor
+ // Workspace
+ // User Local Repository
+ //
+
+ // we only lookup the project scoped ones here, lookup at system level
will also find the system components
+ // (like ReactorReader) but that does not work well as these are
special handled!
+ for ( WorkspaceReader workspaceReader :
getProjectScopedExtensionComponents( session.getProjects(),
+
WorkspaceReader.class ) )
+ {
+ repoSession.setWorkspaceReader(
ChainedWorkspaceReader.newInstance( repoSession.getWorkspaceReader(),
Review comment:
Okay, ChainedWorkspaceReader is not part of this repository should I
suggest a change in ChainedWorkspaceReader or should I handle it in
MavenDefault?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
