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

Native loading refactor #162

Merged
merged 84 commits into from
Dec 19, 2024
Merged

Conversation

DariusIMP
Copy link
Member

@DariusIMP DariusIMP commented Dec 18, 2024

Native loading refactor

Aligning with zenoh-kotlin's native lib loading logic:

  • Adding extra checks for determining the platform on top of which zenoh-java is running
  • Defaulting javaClass's class loader in case the system class loader fails to find the native lib

…allback subclasses + tidying up documentation.
Aligning with zenoh-kotlin's native lib loading logic:
- Adding extra checks for determining the platform on top of which zenoh-java is running
- Defaulting javaClass's class loader in case the system class loader fails to find the native lib
@DariusIMP DariusIMP changed the title Zenoh jni loading Native loading refactor Dec 18, 2024
Copy link

PR missing one of the required labels: {'dependencies', 'enhancement', 'bug', 'documentation', 'breaking-change', 'new feature', 'internal'}

Copy link

PR missing one of the required labels: {'new feature', 'bug', 'internal', 'documentation', 'dependencies', 'breaking-change', 'enhancement'}

@DariusIMP DariusIMP added release Part of the next release internal Changes not included in the changelog labels Dec 19, 2024
@DariusIMP DariusIMP requested a review from milyin December 19, 2024 12:56
@milyin milyin merged commit 61b6f26 into eclipse-zenoh:main Dec 19, 2024
9 of 10 checks passed
@DariusIMP DariusIMP deleted the zenoh_jni_loading branch December 19, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes not included in the changelog release Part of the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants