[GitHub] [maven-build-cache-extension] AlexanderAshitkin commented on pull request #30: [MBUILDCACHE-24] - Fixes IllegalArgumentException in forked executions

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

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

   > I'd like to start a release of `maven-build-cache-extension` this week after merging #29. @AlexanderAshitkin Is this PR something you want in this first release ?
   
   HI. So, current situation is that plugin falls with error for a number of popular plugins (pod and spot bugs among them). I think we need fix this before releasing first version. The fix looks pretty safe for me, but it requires to get execution events which are only available in "core extension mode". 
   In "build extension" mode `EventSpyDispatcher` is not available to extension and it is not possible to subscribe for events easily. If I add EventSpy into extension, it disables cache because `BuildCacheMojosExecutionStrategy` is not discovered by guide. So I'm trying to find solution which will allow to run extension in both modes
   


-- 
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