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

Upgraded jaxb libraries to the newest 2.3.x versions #2793

Merged
merged 1 commit into from
May 9, 2022

Conversation

nvoxland
Copy link
Contributor

@nvoxland nvoxland commented Apr 25, 2022

Description

Updated shipped jaxb libraries. To continue to support libraries we use which expect javax.xml vs. jakarta.xml packages we had to stay on 2.3.x vs. upgrading to 3.x

Things to be aware of

The new jaxb-runtime-2.3.6.jar file replaces the old jaxb-impl-2.3.0.jar file.

The jaxb-api and jaxb-core jar files have the same base names, just different versions now.

@nvoxland nvoxland added this to the NEXT milestone Apr 25, 2022
@github-actions
Copy link

Unit Test Results

  4 512 files  ±0    4 512 suites  ±0   41m 7s ⏱️ + 3m 50s
  4 374 tests ±0    4 156 ✔️ ±0     218 💤 ±0  0 ±0 
51 768 runs  ±0  46 756 ✔️ ±0  5 012 💤 ±0  0 ±0 

Results for commit ef86eb2. ± Comparison against base commit 6793c86.

@kataggart kataggart removed this from the NEXT milestone Apr 27, 2022
@XDelphiGrl XDelphiGrl self-assigned this May 5, 2022
@XDelphiGrl
Copy link
Contributor

There are no open CVEs reported by Maven Central for the upgraded dependencies.
The test harness execution failed due to a known issue unrelated to this PR.

Failing Test Harness Execution
Passing Functional Tests

Copy link
Contributor

@XDelphiGrl XDelphiGrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@kataggart kataggart added this to the NEXT milestone May 9, 2022
@nvoxland nvoxland merged commit 4946f4b into master May 9, 2022
@nvoxland nvoxland deleted the update-jaxb-libraries branch May 9, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants