gnodet commented on PR #26330: URL: https://github.com/apache/camel/pull/26330#issuecomment-5758882666
Fixed `Dependency Analysis (non-blocking)` CI failure in 5e3b40ae89f657b793d45adbbdb55c1f66df6dbd. **Root cause:** `camel-xml-io`'s `generate-xml-sources` execution (generate-xml-parser + generate-xml-writer) runs at `generate-sources` phase and reads `camel-core-model/target/classes/META-INF/jandex.idx`, which is produced by `PackageJandexMojo` at `process-classes` phase. In the dep-check workflow's clean checkout, only `compile` phase runs — so `process-classes` never executes and `jandex.idx` is missing. **Why the previous fix didn't work:** The `7d68953` commit tried to bind `generate-xml-sources` to `phase=none` via `pluginManagement` in the root dep-check profile. This doesn't work: Maven's pluginManagement phase binding in a **parent** profile is overridden by an explicit `<phase>` in the **child pom's** own execution block. So `camel-xml-io/pom.xml`'s own `<phase>generate-sources</phase>` always won. **Fix:** Add `!core/camel-xml-io` to `EXCLUDED_MODULES` in the workflow. The generated sources are pre-committed in `src/generated/java`, so skipping this module from dep-check is safe. Also removed the now-dead `pluginManagement` entry from `pom.xml`. -- 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]
