AlexanderAshitkin commented on PR #30:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/30#issuecomment-1289185657

   > How are `EventSpy` and `BuildCacheMojosExecutionStrategy` related ?
   I haven't traced it down and so far do not understand how exactly it 
interferes, but everything points to Guice initialization lifecycle. As I 
understand, adding `EventSpy` forces earlier forces earlier resolution in 
extension realm. After that for some reason `BuildCacheMojosExecutionStrategy` 
strategy is not picked up and build continues with default non-caching strategy.
   
   It could be reproduced in master branch with 2 steps:
   * delete `CacheLifecycleParticipant` (to avoid circular dependency conflict)
   * make `LifecyclePhasesHelper` extend `AbstractEventSpy`
   Expected: `BuildExtensionTest` starts failing failing because cache is not 
intialized
   
   If there is a proper way to resolve this - I'm happily ready to pick it up


-- 
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-unsubscr...@maven.apache.org

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

Reply via email to