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

Add arm64 variant build #5135

Merged
merged 2 commits into from
Dec 1, 2022
Merged

Add arm64 variant build #5135

merged 2 commits into from
Dec 1, 2022

Conversation

iamer
Copy link
Collaborator

@iamer iamer commented Nov 23, 2022

No description provided.

@cla-bot cla-bot bot added the cla: yes label Nov 23, 2022
@iamer iamer requested a review from kneth November 23, 2022 12:31
@@ -0,0 +1,16 @@
set(CMAKE_SYSTEM_NAME Linux)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this file be in core instead?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I wasn't sure how easy it would be to get it there, and it would be good to see it work here first. If it is easy to get it into core and then update core in realm-js we can do it that way.

Copy link
Member

Choose a reason for hiding this comment

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

I agree, we should just add this file to Core, even if Core's CI won't use it just yet.

@@ -0,0 +1,16 @@
set(CMAKE_SYSTEM_NAME Linux)
Copy link
Member

Choose a reason for hiding this comment

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

I agree, we should just add this file to Core, even if Core's CI won't use it just yet.

@iamer iamer requested a review from kneth November 30, 2022 10:14
Copy link
Contributor

@kneth kneth left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks!

@iamer iamer marked this pull request as ready for review December 1, 2022 14:57
@iamer iamer merged commit bc5cb40 into master Dec 1, 2022
@iamer iamer deleted the iamer/add-arm64-variant branch December 1, 2022 14:58
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants