cowwoc commented on PR #389:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/389#issuecomment-6586185220

   @olamy Addressed your 
[observation](https://github.com/apache/maven-build-cache-extension/pull/389#issuecomment-6575256179)
 in f8dc720. `beforeMojoExecution` now captures normalized parameter values as 
immutable strings, so compiler changes made during execution 
(`--module-version`, JPMS `--patch-module`/`--add-reads`, and toolchain-related 
field changes) do not affect the saved values. I removed the `--module-version` 
special case. The JPMS `testCompile` and issue #375 integration tests assert 
that the second build actually skips compiler execution; both pass.
   


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

Reply via email to