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.

Sign up for GitHub

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

Jump to bottom

[NETBEANS-6410] - Upgrade JAXB from 2.3.3 to 2.3.5 #3545

Merged
merged 1 commit into from
Feb 3, 2022
Merged

[NETBEANS-6410] - Upgrade JAXB from 2.3.3 to 2.3.5 #3545

merged 1 commit into from
Feb 3, 2022

Conversation

pepness
Copy link
Member

@pepness pepness commented Jan 31, 2022

Library notes:

  • Bug fixes and improvements
  • Tests fixes
  • Updated dependencies
  • Code cleanup
  • Remove use of soon to be deleted Java APIs. See PR-1463

Testing:

  • Full build done
  • Verify successfull execution of unit tests for modules /ide/libs.jaxb & /ide/xml.jaxb.api
  • Verify successfull execution of ant verify-libs-and-licenses
  • Successfully create project that bind WSDL to Java with JAXB as per NetBeans Documentation(Used web.archive.org)
  • Verify that the files generated used the Eclipse implementation of JAXB
    NETBEANS-6410

JAXB Web Page
Release Notes

@pepness pepness self-assigned this Jan 31, 2022
@pepness pepness added the Upgrade Library Library (Dependency) Upgrade label Jan 31, 2022
@pepness pepness added this to the NB14 milestone Jan 31, 2022
matthiasblaesing
matthiasblaesing approved these changes Feb 2, 2022
View reviewed changes
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Looks sane to me. When working with the jaxb modules, it seemed, that 2.3 was stabelized so I agree this should be a save update. If I remember correctly 2.4 (or did they jump to 3?) will break packages, so will not be binary compatible.

Lets get this in.

@pepness pepness merged commit 44ea743 into apache:master Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthiasblaesing matthiasblaesing matthiasblaesing approved these changes

Assignees

@pepness pepness

Labels
Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Milestone
NB14
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pepness @matthiasblaesing