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: support apple silicon (arm64 darwin) builds #49

Merged
merged 2 commits into from
Jul 10, 2020

Conversation

MarshallOfSound
Copy link
Member

Copy link
Contributor

@ryzokuken ryzokuken left a comment

Choose a reason for hiding this comment

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

LGTM overall. Is there absolutely any way to be able to test this though?

@cclauss
Copy link
Contributor

cclauss commented Jul 10, 2020

We will have to monitor https://blog.macstadium.com/category/macstadium-news to see when the new Apple hardware is added.

@MarshallOfSound
Copy link
Member Author

You can test the "build" part using any machine with the Xcode 12.0 UA preview on it, but you won't be able to ensure it actually runs / works without an apple-silicon machine somewhere. Definitely works on my DTK though 😄

@ryzokuken
Copy link
Contributor

@MarshallOfSound great. I'll merge this in after another review and once that's done I suppose we could make a minor release with this. Would you mind adding this to the changelog file?

@MarshallOfSound
Copy link
Member Author

@ryzokuken Updated

Copy link
Contributor

@ryzokuken ryzokuken left a comment

Choose a reason for hiding this comment

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

@ryzokuken ryzokuken merged commit 458a949 into nodejs:master Jul 10, 2020
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