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

Ship both x86 and aarch64 darwin libs #79

Merged
merged 9 commits into from
Sep 18, 2022

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Sep 5, 2022

Signed-off-by: Antoine Toulme antoine@lunar-ocean.com

Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
Copy link
Contributor

@shemnon shemnon left a comment

Choose a reason for hiding this comment

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

Looks good for the most part.

Except that the Blake2F lib performs worse than the Java lib right now on mac. Performance is out of scope for this PR.

Copy link
Contributor

@garyschulte garyschulte left a comment

Choose a reason for hiding this comment

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

Are we not able to build a lipo/multi-arch darwin lib for blake2f in the same way we do for the other besu-native libs?

@atoulme
Copy link
Contributor Author

atoulme commented Sep 18, 2022

jna now uses separate folders for darwin, so you can't use a multi-arch lib.
And blake2bf has some ever so slightly differences in imports between m1 and arm, and required a change in the code for imports. You can probably reconcile and use a precompile condition like ifdef __APPLE__. I picked that up a bit late.

Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com>
@atoulme
Copy link
Contributor Author

atoulme commented Sep 18, 2022

I have verified locally that the m1 lib for ipa-multipoint now loads properly with the zulu java 17 SDK.

@atoulme atoulme merged commit 57dc1d4 into hyperledger:main Sep 18, 2022
@shemnon shemnon mentioned this pull request Sep 21, 2022
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.

3 participants