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

feat: jna over static jni #1007

Merged
merged 1 commit into from
Jun 13, 2024
Merged

feat: jna over static jni #1007

merged 1 commit into from
Jun 13, 2024

Conversation

sgammon
Copy link
Member

@sgammon sgammon commented Jun 13, 2024

Ready for review Powered by Pull Request Badge

Summary

Adopts java-native-access/jna#1608 to enable JNA statically in Elide's final binary target.

Changelog

  • chore: switch to jna snapshot
  • chore: add new jna graalvm artifact
  • chore: opt-in to static jna feature
  • chore: cleanup superfluous configs

@sgammon sgammon added feature Large PRs or issues with full-blown features 🧪 labs Experimental work labels Jun 13, 2024
@sgammon sgammon added this to the Release R6: Alpha 10 milestone Jun 13, 2024
@sgammon sgammon requested a review from a team June 13, 2024 07:45
@sgammon sgammon self-assigned this Jun 13, 2024
@sgammon sgammon enabled auto-merge (rebase) June 13, 2024 07:45
@sgammon sgammon linked an issue Jun 13, 2024 that may be closed by this pull request
@sgammon sgammon mentioned this pull request Jun 13, 2024
Copy link

codecov bot commented Jun 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.59%. Comparing base (9b3d37f) to head (7d00a24).
Report is 11 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1007   +/-   ##
=======================================
  Coverage   54.59%   54.59%           
=======================================
  Files         320      320           
  Lines       10151    10151           
  Branches     1759     1759           
=======================================
  Hits         5542     5542           
  Misses       4078     4078           
  Partials      531      531           
Flag Coverage Δ
gradle 54.59% <ø> (ø)
jvm 54.59% <ø> (ø)
lib 54.59% <ø> (ø)
plugin 54.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 616f63d...7d00a24. Read the comment docs.

- chore: switch to jna snapshot
- chore: add new jna graalvm artifact
- chore: opt-in to static jna feature
- chore: cleanup superfluous configs

Adopts java-native-access/jna#1608

Signed-off-by: Sam Gammon <[email protected]>
Copy link
Member

@darvld darvld left a comment

Choose a reason for hiding this comment

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

Amazing how much configuration we can elide now 🙂

@sgammon sgammon merged commit 0ecb6a8 into main Jun 13, 2024
22 checks passed
@sgammon sgammon deleted the feat/jna-static branch June 13, 2024 14:22
This was referenced Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Large PRs or issues with full-blown features 🧪 labs Experimental work
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Static JNI for JNA
2 participants