search

FasterXML / jackson-dataformat-xml

Extension for Jackson JSON processor that adds support for serializing POJOs as XML (and deserializing from XML) as an alternative to JSON
Apache License 2.0
567 stars 221 forks source link

Have you eliminated the dependency on jakarta.xml.bind? #519

Closed ShunichirouKamino closed 2 years ago

ShunichirouKamino commented 2 years ago

Thanks for the great framework. I upgraded the following three from 2.12.4 to 2.13.2.

Previously there was a transitional dependency on jakarta.xml.bind-api, which appears to have been lost with this update.

Should I make my own dependencies on jakarta? What is the reason for abolishing dependencies?

cowtowncoder commented 2 years ago

Yet, correct, XML module should not depend on JAXB since it does not use it. So you do need to include it either explicitly, or have other component you depend on that depends on it.

Part of the reason for removal was to solve the tricky issue of "Jakarta vs JAXB" problem, where post-Oracle java extension packages moved to new Java packages (due to trademark issues, I think).

Originally this dependency was included for some interoperability reasons with JAXB module but it was always something I wanted not to have. Minimizing dependencies is a worthy goal in general. It is only unfortunate that the dependency existed in the first place.

ShunichirouKamino commented 2 years ago

Thank you for your thoughtful response.

Part of the reason for removal was to solve the tricky issue of "Jakarta vs JAXB" problem.

As you say, if you are not using the JAXB module in the first place, I think it is appropriate to delete it. We will close it. Thank you very much.