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

Ban com.sun.xml.bind:jaxb-core and jaxb-impl to avoid clash with org.glassfish.jaxb:jaxb-core and jaxb-runtime #6382

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Aug 23, 2024

Those two groups of artifacts have lots of overlapping classes. The Glassfish ones should be enough. Let's see if all tests are going to pass and then I'd do the same in Camel.

@ppalaga ppalaga force-pushed the 240823-ban-com.com.sun.xml.bind-main branch from 07684c7 to 6d1f167 Compare August 23, 2024 19:14
@ppalaga
Copy link
Contributor Author

ppalaga commented Aug 23, 2024

All test have passed 🎉, so I'd be in favor of merging this PR for main, and rather not merging the one for 3.8.x

Here is the proper fix for Camel: apache/camel#15298

@zhfeng zhfeng merged commit c47698c into apache:main Aug 26, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants