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

Bumping isa-l to v2.31.0 #7967

Merged
merged 1 commit into from Apr 18, 2024
Merged

Conversation

liranmauda
Copy link
Contributor

Explain the changes

Bumping isa-l to v2.31.0

Issues: Fixed #xxx / Gap #xxx

  1. Fixes: deps: update to latest isa-l v2.31 to allow building on mac arm64 #7945

Copy link
Member

@guymguym guymguym left a comment

Choose a reason for hiding this comment

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

should update versions.md

@guymguym
Copy link
Member

@liranmauda does this build successfully on mac arm64 now?

@liranmauda
Copy link
Contributor Author

@liranmauda does this build successfully on mac arm64 now?

I have managed to build it on my mac (M1)

@guymguym
Copy link
Member

@liranmauda It fails on mac with node for arm64.

We need to add a condition to the mac section to handle node_arch=x64 vs arm64 like we do for linux.

In the isa-l readme and makefile it shows that on arm the toolchain to use is gcc/gas.
However when I tried it I had some issues on my mac so it might need more configuration to work out.

If you want we can merge this anyway to start running regression with the updated code, and fix the mac arm build in next PR.

@liranmauda
Copy link
Contributor Author

@liranmauda It fails on mac with node for arm64.

We need to add a condition to the mac section to handle node_arch=x64 vs arm64 like we do for linux.

In the isa-l readme and makefile it shows that on arm the toolchain to use is gcc/gas. However when I tried it I had some issues on my mac so it might need more configuration to work out.

If you want we can merge this anyway to start running regression with the updated code, and fix the mac arm build in next PR.

@guymguym I agree, probably we need to do the same as in the Linux.
We can merge this so the code will be there and run.

I will rebase, and when it is approved I will merge it.

Bumping isa-l to v2.31.0

Signed-off-by: liranmauda <liran.mauda@gmail.com>
@liranmauda liranmauda merged commit ed421ac into noobaa:master Apr 18, 2024
10 checks passed
@liranmauda liranmauda deleted the liran-bump-dependency branch April 18, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

deps: update to latest isa-l v2.31 to allow building on mac arm64
2 participants