Skip to content

Fix version of fasterxml.woodstox - #4581

Merged
wborn merged 1 commit into
openhab:mainfrom
holgerfriedrich:pr-fix-deps
Jan 28, 2025
Merged

Fix version of fasterxml.woodstox#4581
wborn merged 1 commit into
openhab:mainfrom
holgerfriedrich:pr-fix-deps

Conversation

@holgerfriedrich

@holgerfriedrich holgerfriedrich commented Jan 28, 2025

Copy link
Copy Markdown
Member
  • Add wstx version matching cxf 3.x

This seems to be required for the demo app to resolve properly.
Transitive dependencies from karaf libs brings fasterxml.woodstox 7.x, whereas cxf 3.6.4 still requires fasterxml.woodstox 6.x.
Adding this resolves the issue with the demo app.

Relates to latest karaf upgrade, #4406.
Relates to openhab/openhab-distro#1721.

* Add wstx version matching cxf 3.x

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@holgerfriedrich holgerfriedrich added bug An unexpected problem or unintended behavior of the Core regression labels Jan 28, 2025
@holgerfriedrich
holgerfriedrich marked this pull request as ready for review January 28, 2025 20:57
@holgerfriedrich
holgerfriedrich requested a review from a team as a code owner January 28, 2025 20:57
@openhab-bot

Copy link
Copy Markdown
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/potential-problem-with-eclipse-and-latest-core/162024/10

@wborn wborn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@wborn
wborn merged commit 7b85d6d into openhab:main Jan 28, 2025
@wborn wborn added this to the 5.0 milestone Jan 28, 2025
@holgerfriedrich
holgerfriedrich deleted the pr-fix-deps branch January 28, 2025 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An unexpected problem or unintended behavior of the Core regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants