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

[6.2]feat: bump jaxb to 4 and dynamic generate code #1128

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

miurahr
Copy link
Member

@miurahr miurahr commented Aug 29, 2024

Java 9 and later drops JAXB from Java runtime.
JAXB standard is now version 4 but OmegaT depends version 2 which is bundled version of Java 8.

OmegaT 6.1.0_beta and later moves to Java 11, so we can update JAXB 4.

There is a package jakarta.xml.bind in JAXB 4 instead of javax.xml.bind in JAXB 2, so we need to modify source code.

Pull request type

  • Feature enhancement -> [enhancement]
  • Build and release changes -> [build/release]

Which ticket is resolved?

What does this PR change?

  • Drop jaxb generated files from repository

  • Update task dependency to use generated code

  • Tweak JAXB generation code difference; filters.getFilters() -> getFilter()

  • Tweak JAXB api package from javax.xml.* to jakarta.xml.*

  • Bump [email protected] which based on JAXB 4

  • Update javax.xml.bind:jaxb.xml.bind-api:2.0.5 to jakarta.xml.bind:jakarta.xml.bind-api:4.0.2

  • Update org.glassfish.jaxb:jaxb-xjc:2.3.1 to org.glassfish.jaxb:jaxb-xjc:4.0.5

  • Update org.glassfish.jaxb:jaxb-runtime:2.3.0 to org.glassfish.jaxb:jaxb-runtime:4.0.5

  • Update com.fasterxml.jackson.module:jackson-module-jaxb-annotations:2.16.1 to com.fasterxml.jackson.module:jackson-module-jakarta-xmlbind-annotations:2.16.1

Other information

@miurahr miurahr added enhancement build/release refactoring dependencies Pull requests that update a dependency file labels Aug 29, 2024
Copy link

❌ Unit Tests, Quality checks, and Acceptance Tests failed.

Please look a Gradle Scan page for details:
https://gradle.com/s/vwbnmw32n7rhq

- Drop jaxb generated files from repository
- Update task dependency to use generated code
- Tweak JAXB generation code difference; filters.getFilters() -> getFilter()
- Tweak JAXB api package from javax.xml.* to jakarta.xml.*
- Bump [email protected] which based on JAXB 4
- Update javax.xml.bind:jaxb.xml.bind-api:2.0.5 to
  jakarta.xml.bind:jakarta.xml.bind-api:4.0.2
- Update org.glassfish.jaxb:jaxb-xjc:2.3.1 to
  org.glassfish.jaxb:jaxb-xjc:4.0.5
- Update org.glassfish.jaxb:jaxb-runtime:2.3.0 to
  org.glassfish.jaxb:jaxb-runtime:4.0.5
- Update
  com.fasterxml.jackson.module:jackson-module-jaxb-annotations:2.16.1 to
com.fasterxml.jackson.module:jackson-module-jakarta-xmlbind-annotations:2.16.1

Signed-off-by: Hiroshi Miura <[email protected]>
@miurahr miurahr force-pushed the topic/miurahr/gen/bump-jaxb-4 branch from a31ccf1 to 8063169 Compare September 1, 2024 01:57
@miurahr miurahr added this to the 6.2.0 - UI/Process change milestone Sep 7, 2024
@miurahr miurahr changed the title feat: bump jaxb to 4 and dynamic generate code [6.2]feat: bump jaxb to 4 and dynamic generate code Oct 6, 2024
Copy link

github-actions bot commented Oct 6, 2024

❌ Unit Tests, Quality checks, and Acceptance Tests failed.

Please look a Gradle Scan page for details:
https://gradle.com/s/3lbw5egza5kjo

@miurahr miurahr requested review from briacp and t-cordonnier October 6, 2024 01:06
build.gradle Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build/release dependencies Pull requests that update a dependency file enhancement refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant