Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate JAXB artifacts #1394

Closed
Sanne opened this issue Mar 11, 2019 · 10 comments
Closed

Duplicate JAXB artifacts #1394

Sanne opened this issue Mar 11, 2019 · 10 comments
Labels
triage/out-of-date This issue/PR is no longer valid or relevant

Comments

@Sanne
Copy link
Member

Sanne commented Mar 11, 2019

When building Quarkus, specifically while wathing module quarkus-integration-test-main one can see this warning being logged:

Duplicate entry javax/xml/bind/util/Messages.properties entry from javax.xml.bind:jaxb-api::jar:2.3.1(provided) will be ignored. Existing file was provided by org.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.3_spec::jar:1.0.1.Final(compile)

So looks like we're bringing in both javax.xml.bind:jaxb-api and org.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.3_spec.

Incidentally, 2.3.2 was released but the GAV of JAXB changed once again :(

@Sanne Sanne added the kind/bug Something isn't working label Mar 11, 2019
@Dufgui
Copy link
Contributor

Dufgui commented Aug 21, 2019

I take a look at old issue, as I can see it's still the case:

[INFO] org.acme:getting-started:jar:1.0-SNAPSHOT
[INFO] +- io.quarkus:quarkus-resteasy:jar:999-SNAPSHOT:compile
[INFO] |  +- io.quarkus:quarkus-undertow:jar:999-SNAPSHOT:compile
[INFO] |  \- io.quarkus:quarkus-resteasy-server-common:jar:999-SNAPSHOT:compile
[INFO] |     +- io.quarkus:quarkus-resteasy-common:jar:999-SNAPSHOT:compile
[INFO] |     |  +- org.jboss.resteasy:resteasy-core:jar:4.2.0.Final:compile
[INFO] |     |  |  +- org.jboss.spec.javax.ws.rs:jboss-jaxrs-api_2.1_spec:jar:1.0.2.Final:compile
[INFO] |     |  |  +- org.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.3_spec:jar:1.0.1.Final:compile
[INFO] |     |  |  +- org.jboss.resteasy:resteasy-core-spi:jar:4.2.0.Final:compile
[INFO] |     |  |  \- org.reactivestreams:reactive-streams:jar:1.0.2:compile
[INFO] |     |  +- org.jboss.resteasy:resteasy-context-propagation:jar:4.2.0.Final:compile
[INFO] |     |  |  \- io.smallrye:smallrye-context-propagation:jar:1.0.7:compile
[INFO] |     |  |     \- io.smallrye:smallrye-context-propagation-api:jar:1.0.7:compile
[INFO] |     |  \- com.sun.activation:jakarta.activation:jar:1.2.1:compile
[INFO] |     +- javax.validation:validation-api:jar:2.0.1.Final:compile
[INFO] |     \- io.quarkus:quarkus-jaxb:jar:999-SNAPSHOT:compile
[INFO] \- io.rest-assured:rest-assured:jar:3.3.0:test
[INFO]    +- io.rest-assured:json-path:jar:3.3.0:test
[INFO]    \- io.rest-assured:xml-path:jar:3.3.0:test
[INFO]       +- org.apache.commons:commons-lang3:jar:3.8.1:test
[INFO]       +- javax.xml.bind:jaxb-api:jar:2.3.1:compile
[INFO]       \- org.apache.sling:org.apache.sling.javax.activation:jar:0.1.0:test

Which one do you want to keep . jakarta.xml.bind-api (jaxb-api) or jboss-jaxb-api ?

@stale
Copy link

stale bot commented Nov 13, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you!
We are doing this automatically to ensure out-of-date issues does not stay around indefinitely.
If you believe this issue is still relevant please put a comment on it on why and if it truly needs to stay request or add 'pinned' label.

@stale stale bot added the stale label Nov 13, 2019
@maxandersen maxandersen removed the stale label Nov 13, 2019
@gsmet
Copy link
Member

gsmet commented Nov 13, 2019

This has been fixed by the big Jakarta move.

@gsmet gsmet closed this as completed Nov 13, 2019
@gsmet gsmet added the triage/out-of-date This issue/PR is no longer valid or relevant label Nov 13, 2019
@Sanne Sanne reopened this Nov 14, 2019
@Sanne
Copy link
Member Author

Sanne commented Nov 14, 2019

Re-opening: It wasn't fixed, like you just explained to me on the mailing list you decided to not go for the Jakarta artifacts.

We're having:

  • javax.xml.bind:jaxb-api - brought by rest-assured
  • org.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.3_spec - most Quarkus components - especially Resteasy

The good news is that rest-assured is mostly a test component - but it's expected to be used for testing by end users too so it's not a non-issue.

@Sanne Sanne removed the triage/out-of-date This issue/PR is no longer valid or relevant label Nov 14, 2019
@gsmet
Copy link
Member

gsmet commented Nov 14, 2019

Well, the only thing we can do here is wait for REST Assured to move to Jakarta.

We can keep it open if you prefer but there's nothing we can do on our side.

@Sanne
Copy link
Member Author

Sanne commented Nov 14, 2019

rest-assured/rest-assured#1228 ;-)

@rsvoboda
Copy link
Member

I tried to update rest-assured to 4.2.0 with @Sanne's fix but I receive Found Banned Dependency: jakarta.xml.bind:jakarta.xml.bind-api:jar:2.3.2

https://github.com/quarkusio/quarkus/blob/master/build-parent/pom.xml#L222

Jakarta APIs still on hold ?

@gsmet
Copy link
Member

gsmet commented Jan 24, 2020

I already merged the upgrade here: #6704 .

The situation is a bit more complex than that because RESTEasy is using repackaged Jakarta artifacts for JAXB and JAX-RS APIs (the usual JBoss ones). For now, we decided to use that ones to avoid complexity for the users, at least until RESTEasy migrates to vanilla Jakarta artifacts.

@gastaldi
Copy link
Contributor

Is this still an issue or can be closed?

@gastaldi
Copy link
Contributor

gastaldi commented Oct 9, 2020

I don't see the warning anymore, so I assume this issue is now fixed.

@gastaldi gastaldi closed this as completed Oct 9, 2020
@gastaldi gastaldi added triage/out-of-date This issue/PR is no longer valid or relevant and removed kind/bug Something isn't working labels Oct 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/out-of-date This issue/PR is no longer valid or relevant
Projects
None yet
Development

No branches or pull requests

6 participants