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

Fix for dependency conflicts introduced with ehcache 3.x upgrade (PR #1441) #1459

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

tfr42
Copy link
Member

@tfr42 tfr42 commented Jan 23, 2023

This PR fixes the broken JAXB dependencies introduced with PR #1441.

The deegree project uses the jakarta.xml.bind:jakarta.xml.bind-api(API) with com.sun.xml.bind:jaxb-impl (JAXB implementation).
The ehcache project uses org.glassfish.jaxb:jaxb-runtime in the version range of [2.2,3). This brings duplications of types in different versions from the com.sun.xml.bind package into the classpath and results in build errors.

To fix this the org.glassfish.jaxb:jaxb-runtime dependencies introduced with ehcache are excluded from the dependencies. This shall not affect ehcache at runtime since the difference between jaxb-impl and the jaxb-runtime JAR is that the jaxb-impl JAR also bundles types from com.sun.istack and com.sun.xml.txw2 whereas the jaxb-runtime provides them as separate dependencies.

More info about JAXB https://github.com/eclipse-ee4j/jaxb-ri/.

@tfr42 tfr42 added dependencies Pull requests that update a dependency (library) CI and build CI and build system labels Jan 23, 2023
@tfr42 tfr42 added this to the 3.5 milestone Jan 23, 2023
This was referenced Jan 23, 2023
@lgoltz lgoltz removed their assignment Jan 24, 2023
@copierrj copierrj merged commit 182e4cc into deegree:main Jan 25, 2023
@tfr42 tfr42 deleted the fixJaxbDependencies branch January 25, 2023 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI and build CI and build system dependencies Pull requests that update a dependency (library)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants