Skip to content

Update cxf #165

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
Draft
StyopinN wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Update cxf #165

StyopinN wants to merge 2 commits into from

Conversation

@StyopinN
Copy link

@StyopinN StyopinN commented Jan 12, 2023

Trying to solve #164

Fabio1988
Fabio1988 reviewed Jan 25, 2023
View reviewed changes
service/pom.xml
<dependency>
<groupId>jakarta.xml.ws</groupId>
<artifactId>jakarta.xml.ws-api</artifactId>
<version>3.0.1</version>
Copy link

@Fabio1988 Fabio1988 Jan 25, 2023

Choose a reason for hiding this comment

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

did you also try with newer version?
https://mvnrepository.com/artifact/jakarta.xml.ws/jakarta.xml.ws-api

Copy link
Author

@StyopinN StyopinN Feb 1, 2023

Choose a reason for hiding this comment

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

No. My goal was in updating cxf to 4.+, and it has jakarta.xml.ws-api v.3.0.1 as dependency

Copy link
Author

@StyopinN StyopinN Feb 1, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

And I have a problem with generating code for client: jaxws/bindings.xml rules should change package name, but they don't. If anyone know how to solve this problem - just push changes to this pull request.

@knalli
Copy link

knalli commented Mar 21, 2023

fyi: I have completed the migration to CXF v4, JDK 17 and Jakarta. Available atm in a dedicated branch in our fork. Please feel free to check this out and let me know if you have issues. I'm running this custom build within a JDK17/SB3/Jakarta environment successfully.

Note: The branch in question is based on my proposed PR #158.

@knalli knalli mentioned this pull request Mar 29, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Fabio1988 Fabio1988 Fabio1988 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@StyopinN @knalli @Fabio1988