efegokdemir commented on PR #1136: URL: https://github.com/apache/maven-compiler-plugin/pull/1136#issuecomment-5968883854
Addressed the implicit pre-Java-23 `proc` regression in 55c4389. When `proc` is unset on Java <23, the compiler default only means processing is enabled; it does not establish that a processor is present. That path now retains the previous `REBUILD_ON_ADD` / `REBUILD_ON_CHANGE` behavior instead of clearing all aspects to `NONE`. Explicit `proc=only` and `proc=full` still use `NONE`, and the explicit `incrementalCompilation` guard remains unchanged. Validation: focused `proc=only` and `proc=full` unit tests passed (2/2) on local JDK 27; `git diff --check` passed. The full `CompilerMojoTestCase` run had 1 local JDK-27-only failure in the explicit-incremental expectation (17 passed); the remote JDK-17 CI run before this commit showed that test passing and failed in `MCOMPILER-500-package-info-incr`. I could not execute the reported JDK-17 integration cases locally. Spotless is blocked by the repository formatter’s `JCCompilationUnit.endPositions` incompatibility with JDK 27. No checks are reported for the new remote head yet. -- 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]
