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

Support arm64 (Apple Silicon) #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tsbehlman
Copy link

Treating the arm64 architecture as 64-bit appears to be enough to compile successfully on an M1 Pro, and likely the rest of the current Apple Silicon-based chips.

Treating the arm64 architecture as 64-bit appears to be enough to compile sucessfully on an M1 Pro, and likely the rest of the current Apple Silicon-based chips
@m4dEngi
Copy link
Owner

m4dEngi commented Jun 15, 2022

I know that project is already a mess and i'm considering reorganizing and migrating it to cmake, but still... maybe you can make it at least a little bit more specific about apple/osx instead of using linux build dirs and generic aarch64?

@tsbehlman
Copy link
Author

I know that project is already a mess and i'm considering reorganizing and migrating it to cmake, but still... maybe you can make it at least a little bit more specific about apple/osx instead of using linux build dirs and generic aarch64?

Sure thing, sorry for the late reply! I've updated the PR with these changes, let me know how you feel about them.

It might make more sense to generate the platform directories as $(UNAME)-$(UNAME_M) instead of including them in the git repository but I decided to follow the existing pattern for now. I can try making that change if you prefer.

@Ciel1996
Copy link

Hey I just found this nice repo and this issue and it's PR. Is it stable or still in progress? :)

Or can one assume that the branch can be used to remote play together?

Thanks

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