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

Checked compat with JNA 5.11.0 #139

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

mkitti
Copy link
Contributor

@mkitti mkitti commented Jun 14, 2022

Update scijava-pom to 31.1.0
Checked compat with JNA 5.11.0
Use Native.load rather than Native.loadLibrary

xref: scijava/pom-scijava#163

@ctrueden
Copy link
Member

ctrueden commented Sep 13, 2022

@mkitti Uh oh, I just noticed this. Now that we've upgraded Fiji to JNA 5.12.1, but not merged this PR... are there problems with SPIM_Registration? Shall we rebase/update this PR and get it merged?

@mkitti
Copy link
Contributor Author

mkitti commented Sep 13, 2022

Native.loadLibrary is deprecated and should be changed to Native.load.

* Update scijava-pom to 33.0.0.
* Checked compat with JNA 5.11.0.
* Use Native.load rather than Native.loadLibrary.

Signed-off-by: Curtis Rueden <[email protected]>
@ctrueden ctrueden merged commit 1e9e98b into fiji:master Sep 13, 2022
@ctrueden
Copy link
Member

Native.loadLibrary is deprecated and should be changed to Native.load.

@mkitti Noted. But it's just deprecation, not actual breakage, right?

In any case, I rebased the PR, fixed the conflicts, and merged this. Let me know if an immediate release is needed or not.

@mkitti
Copy link
Contributor Author

mkitti commented Sep 13, 2022

I do not think an immediate release is needed.

https://java-native-access.github.io/jna/5.5.0/javadoc/com/sun/jna/Native.html#loadLibrary-java.lang.Class-

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants