-
Notifications
You must be signed in to change notification settings - Fork 10
Native macOS Apple Silicon builds #49
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
base: slippi
Are you sure you want to change the base?
Native macOS Apple Silicon builds #49
Conversation
NikhilNarayana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i also want to understand why we're hard coding homebrew paths
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i say we copy this file and leave the original alone given the scope of our changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment applies to both files changed in this folder
do we need these changes or can we do without them? i'd like some justification so we can note it down and mark that we've made changes
| CMAKE_FLAGS+=" -DMACOS_CODE_SIGNING=OFF" | ||
| fi | ||
|
|
||
| if [[ "$ARCH" == 'arm64' ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
combine this block and the one on line 11
From https://github.com/jmlee337/dolphin/tree/universalmacos14
Only change I made was to not upgrade cmake
Have been using builds from this on my Macbook M1 for the last few weeks with no issue