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

JAX-WS: JAXB: backup with parent namespace test #30566

Conversation

BerksanAtes
Copy link
Member

JAXB provides com.sun.xml.bind.backupWithParentNamespace property to support old JAXB behavior. Tests are added to check the behavior.

@BerksanAtes BerksanAtes added in:Web Services team:Bobcats JAX-WS Web Services Team labels Jan 15, 2025
@BerksanAtes BerksanAtes self-assigned this Jan 15, 2025
@BerksanAtes
Copy link
Member Author

BerksanAtes commented Jan 15, 2025

#build (view Open Liberty Personal Build - ✅ completed successfully!)

Note: Target locations of links might be accessible only to IBM employees.

@BerksanAtes BerksanAtes force-pushed the JAXB_BackupWithParentNamespace_Test branch from d3274f0 to 85b860f Compare January 15, 2025 22:23
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 11 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • Please describe in a separate comment how you tested your changes.

@BerksanAtes BerksanAtes force-pushed the JAXB_BackupWithParentNamespace_Test branch from 85b860f to 0df5832 Compare January 15, 2025 22:25
@BerksanAtes BerksanAtes force-pushed the JAXB_BackupWithParentNamespace_Test branch from 0df5832 to 33ed703 Compare January 15, 2025 22:27
@BerksanAtes
Copy link
Member Author

BerksanAtes commented Jan 16, 2025

#build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 11 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • Please describe in a separate comment how you tested your changes.

Copy link
Member

@neuwerk neuwerk left a comment

Choose a reason for hiding this comment

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

Changes look ok other then Copyrights. Also, do you need this empty file?

dev/com.ibm.ws.jaxb_fat/publish/servers/jaxbspec_fat/jvm.options

@@ -0,0 +1,134 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -0,0 +1,50 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -0,0 +1,15 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -0,0 +1,83 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -0,0 +1,50 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -0,0 +1,15 @@
/*******************************************************************************
* Copyright (c) 2024 IBM Corporation and others.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2024 IBM Corporation and others.
* Copyright (c) 2025 IBM Corporation and others.

@@ -24,6 +24,7 @@
@RunWith(Suite.class)
@SuiteClasses({
LibertyJAXBTest.class,
LibertyJAXBSpecTest.class,
Copy link
Member

Choose a reason for hiding this comment

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

Copyright needs to be

2019, 2025

@BerksanAtes BerksanAtes force-pushed the JAXB_BackupWithParentNamespace_Test branch from 30d3dc5 to f02402f Compare January 27, 2025 16:23
@neuwerk
Copy link
Member

neuwerk commented Jan 27, 2025

#build (view Open Liberty Personal Build - ✅ completed successfully!)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 10 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • Please describe in a separate comment how you tested your changes.

@LibbyBot
Copy link

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_syK9YNzjEe-M5OZShUU9WA

The build will run your changed FAT bucket(s) 10 times.

Target locations of links might be accessible only to IBM employees.

@BerksanAtes BerksanAtes merged commit 4f3fefe into OpenLiberty:integration Jan 28, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants