[GitHub] [maven] maximilian-novikov-db commented on pull request #597: [MNG-7129] Incremental build and shared cache feature

[GitBox <gi...@apache.org>]

maximilian-novikov-db commented on pull request #597:
URL: https://github.com/apache/maven/pull/597#issuecomment-962897611


   > > This diff looks huge to me
   > 
   > yes surely there is some rework to make this diff easier to evaluate and choose to merge into a main branch for a "normal" Maven release I tried to move the whole "caching" package into a separate module a try to have a more visible separation of concerns, but 3 classes had dependencies on maven-core: MojoExecutionManager, ProjectUtils, CacheController. Then it's not so easy...
   > 
   > I chose to start by working on Modello migration #598, that will ease some parts: help welcome any other idea on finding a concrete step to go forward are welcome
   
   @hboutemy @michael-o we've prepared a change to the core which will allow to move the caching module to an extension, it's in testing now. We're going to raise a pr in a week.


-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org