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

Build native lib for arm64 #8

Merged
merged 3 commits into from
May 3, 2024
Merged

Build native lib for arm64 #8

merged 3 commits into from
May 3, 2024

Conversation

fab-10
Copy link
Contributor

@fab-10 fab-10 commented May 3, 2024

Use Dockercross to compile the native lib for different platform and architecture, include arm64.

@fab-10 fab-10 force-pushed the arm64-native-lib branch from dab7c76 to 55c8b92 Compare May 3, 2024 11:08
Signed-off-by: Fabio Di Fabio <[email protected]>
@fab-10 fab-10 force-pushed the arm64-native-lib branch from 55c8b92 to d31a695 Compare May 3, 2024 11:22
@fab-10 fab-10 marked this pull request as ready for review May 3, 2024 12:30
Signed-off-by: Fabio Di Fabio <[email protected]>
@ahamlat
Copy link
Contributor

ahamlat commented May 3, 2024

The docker images look heavier than usual, especially the windows one
image

@fab-10
Copy link
Contributor Author

fab-10 commented May 3, 2024

These Docker images are only used to cross compile, they are not published and are not the final artifact, that is the jar file

@ahamlat
Copy link
Contributor

ahamlat commented May 3, 2024

What is the best way to test in this case ?
Maybe add a documentation in Readme on how to generate cross plateform binaries

@ahamlat
Copy link
Contributor

ahamlat commented May 3, 2024

If I understand correctly, the jar will include both ARM64 and AMD64 library (libcompress_jni.so) thanks to cross plateform compilation. Is that correct ?

@fab-10
Copy link
Contributor Author

fab-10 commented May 3, 2024

Added a doc in the README on how to release native libs locally, and yes the published jar will contain all the built native libs

Copy link
Contributor

@ahamlat ahamlat left a comment

Choose a reason for hiding this comment

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

LGTM.

.github/workflows/release.yml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@fab-10 fab-10 force-pushed the arm64-native-lib branch from 5740d36 to d619ea4 Compare May 3, 2024 14:16
@fab-10 fab-10 merged commit 6e52d3e into main May 3, 2024
7 checks passed
@fab-10 fab-10 deleted the arm64-native-lib branch May 3, 2024 14:39
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