gnodet commented on PR #1161: URL: https://github.com/apache/maven-compiler-plugin/pull/1161#issuecomment-6077540420
Closing this PR — the simplification removes the safety net in the timestamp strategy's `amendincrementalCompilation()` without adding a compensating mechanism. The `ProcessorClassification` added in #1165 only protects the graph strategy path (`computeProcessorCascade()`), not the timestamp path. Relaxing the timestamp strategy's conservative behavior for `proc=only`/`proc=full` could produce invalid compiled output when annotation processors generate or modify classes that the timestamp strategy can't track. See also #1143 for context on why the conservative behavior is correct. _Claude Code on behalf of Guillaume Nodet_ -- 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]
