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

HV-2025 Remove OSGi testing and support #1413

Conversation

marko-bekhta
Copy link
Member

@marko-bekhta marko-bekhta commented Aug 5, 2024

https://hibernate.atlassian.net/browse/HV-2020
https://hibernate.atlassian.net/browse/HV-2025

Opening this as a draft for now. It

  • removes the felix plugin that was generating the bundle info for OSGi
  • removes most of the mentions of OSGi from the docs
  • removes direct dependency on Expressly as without OSGi support we can skip the step of trying to load the EL factory from the EL impl classloader.
  • removes the OSGi directory with tests and karaf features.
  • removes the corresponding relocation pom.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Aug 5, 2024

Thanks for your pull request!

This pull request appears to follow the contribution rules.

› This message was automatically generated.

@marko-bekhta marko-bekhta force-pushed the feat/HV-2025-Remove-OSGi-testing-and-support branch from b72904d to 97b9060 Compare September 3, 2024 15:25
@marko-bekhta marko-bekhta force-pushed the feat/HV-2025-Remove-OSGi-testing-and-support branch from 97b9060 to 2d0650a Compare September 3, 2024 15:26
@marko-bekhta marko-bekhta marked this pull request as ready for review September 4, 2024 15:32
@marko-bekhta
Copy link
Member Author

Thanks for taking a look! 🎉 🏆

@marko-bekhta marko-bekhta merged commit cd4be03 into hibernate:main Sep 4, 2024
7 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.

2 participants