ppalaga commented on issue #8188: URL: https://github.com/apache/camel-quarkus/issues/8188#issuecomment-3789289588
> @ppalaga not sure if the build failure should be considered a bug in the `cq-maven-plugin` `bom-overlaps` feature? > > ``` > [ERROR] The following artifacts are managed in both org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT and org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT: > [ERROR] - at.yawk.lz4:lz4-java 1.10.1 ✅ 1.10.1 > [ERROR] > [ERROR] You may want to either remove the listed artifacts from org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT or add them to <ignoredOverlaps> > ``` > > `at.yawk.lz4:lz4-java` is not defined directly in `camel-quarkus-bom`. It is now being pulled in via `quarkus-cassandra-bom`: > > https://github.com/datastax/cassandra-quarkus/blob/736a91524ce86496bf73992a3b21382fe59bb85f/bom/pom.xml#L63-L68 I would say `bom-overlaps` did the right thing not allowing `at.yawk.lz4:lz4-java` to end up in our flattened BOM. > For now, I can fix these issues in CQ. I can see you did two things to workaround the problem on our side: 1. `<resolutionExclude>at.yawk.lz4:lz4-java:*:*:*:*</resolutionExclude>` and 2. `<ignoredOverlap>at.yawk.lz4:lz4-java</ignoredOverlap>` Ideally we need `cassandra-quarkus` to upgrade to Quarkus 3.31 and remove `lz4-java` from `cassandra-quarkus-bom`. I had a quick attempt at doing this and it did not seem trivial at first glance. Out of those, only the first one really does what we want: not to take over the `at.yawk.lz4:lz4-java` entry from Cassandra BOM to our flattened and minimized BOM. `<ignoredOverlap>at.yawk.lz4:lz4-java</ignoredOverlap>` can be removed. > Ideally we need `cassandra-quarkus` to upgrade to Quarkus 3.31 and remove `lz4-java` from `cassandra-quarkus-bom`. I had a quick attempt at doing this and it did not seem trivial at first glance. Yeah, fixing it in `cassandra-quarkus-bom` would be ideal. -- 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]
