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

Upgrade to Karaf 4.4.4 #3814

Merged
merged 1 commit into from
Sep 29, 2023
Merged

Upgrade to Karaf 4.4.4 #3814

merged 1 commit into from
Sep 29, 2023

Conversation

wborn
Copy link
Member

@wborn wborn commented Sep 27, 2023

  • Sync runtime dependencies with Karaf 4.4.4, most notably:
    • Jetty 9.4.52.v20230823
    • JNA 5.13.0
    • SLF4J 2.0.6
    • Pax Logging 2.2.3
    • Pax Web 8.0.22
  • Resolve itest runbundles
  • Use new Pax Web features to simplify dependency management
  • Add specs features because Pax Web now depends on "asm"

* Sync runtime dependencies with Karaf 4.4.4, most notably:
  * Jetty 9.4.52.v20230823
  * JNA 5.13.0
  * SLF4J 2.0.6
  * Pax Logging 2.2.3
  * Pax Web 8.0.22
* Resolve itest runbundles
* Use new Pax Web features to simplify dependency management
* Add specs features because Pax Web now depends on "asm"

Signed-off-by: Wouter Born <github@maindrain.net>
Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

Thanks!

@J-N-K J-N-K added this to the 4.1 milestone Sep 29, 2023
@J-N-K J-N-K merged commit d4a433a into openhab:main Sep 29, 2023
2 checks passed
@J-N-K
Copy link
Member

J-N-K commented Sep 29, 2023

I have triggered a Jenkins build.

@wborn wborn deleted the karaf-4.4.4 branch September 29, 2023 20:11
florian-h05 pushed a commit to openhab/openhab-webui that referenced this pull request Sep 30, 2023
* Add specs features because Pax Web now depends on "asm".

Depends on openhab/openhab-core#3814.

Signed-off-by: Wouter Born <github@maindrain.net>
@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/unable-to-resolve-dependencies-for-app-bndrun-on-eclipse-ide/150025/2

@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/philips-hue-clip-2-api-v2-discussion-thread/142111/316

@@ -72,20 +72,10 @@

<feature name="openhab.tp-httpclient" version="${project.version}">
<capability>openhab.tp;feature=httpclient;version=${jetty.version}</capability>
<feature dependency="true">pax-web-jetty-extras</feature>
Copy link
Contributor

Choose a reason for hiding this comment

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

This change seems to have broken the Hue binding which is now complaining about a missing ALPN feature.

https://community.openhab.org/t/philips-hue-clip-2-api-v2-discussion-thread/142111/315?u=andrewfg

@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/philips-hue-clip-2-api-v2-discussion-thread/142111/319

wborn added a commit to wborn/openhab-core that referenced this pull request Oct 9, 2023
The pax-web-jetty-http2-jdk9 feature should not have been removed in openhab#3814.

Signed-off-by: Wouter Born <github@maindrain.net>
@wborn wborn mentioned this pull request Oct 9, 2023
J-N-K pushed a commit that referenced this pull request Oct 9, 2023
The pax-web-jetty-http2-jdk9 feature should not have been removed in #3814.

Signed-off-by: Wouter Born <github@maindrain.net>
@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/philips-hue-clip-2-api-v2-discussion-thread/142111/330

@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/openhab-4-1-milestone-discussion/149502/82

@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/not-getting-org-openhab-core-auth-client-oauth2-oauthfactory-injected-by-activation/152526/7

wborn added a commit to wborn/openhab-core that referenced this pull request Jan 3, 2024
Karaf does not automatically install the feature after `dependency="true"` was added in openhab#3814.

Signed-off-by: Wouter Born <github@maindrain.net>
J-N-K pushed a commit that referenced this pull request Jan 3, 2024
Karaf does not automatically install the feature after `dependency="true"` was added in #3814.

Signed-off-by: Wouter Born <github@maindrain.net>
wborn added a commit that referenced this pull request Jan 4, 2024
Karaf does not automatically install the feature after `dependency="true"` was added in #3814.

Signed-off-by: Wouter Born <github@maindrain.net>
cipianpascu pushed a commit to cipianpascu/openhab-core that referenced this pull request Jan 17, 2024
Karaf does not automatically install the feature after `dependency="true"` was added in openhab#3814.

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants