search

eclipse-sisu / sisu-project

Sisu Inject
https://www.eclipse.org/sisu
Eclipse Public License 2.0
17 stars 15 forks source link

Don’t register annotation processor #104

Closed kwin closed 5 months ago

kwin commented 6 months ago

Javac (even in v21) still executes all annotation processors found on the classpath by default. That is rarely necessary for https://github.com/eclipse/sisu.inject/blob/ba4cd7f6110d8705a82c4741c34768b4306950a9/org.eclipse.sisu.inject/META-INF/services/javax.annotation.processing.Processor#L1. Instead that annotation processor should always be explicitly registered if necessary (most of Sisu consumers leverage a dedicated maven goal for generating the index)

kwin commented 6 months ago

Compare with https://bugs.openjdk.org/browse/JDK-8306819