-
Notifications
You must be signed in to change notification settings - Fork 20
Macos Arm build #173
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: development
Are you sure you want to change the base?
Macos Arm build #173
Conversation
HeliumAnt
commented
Dec 20, 2024
- fix macos build
- add macos ARM CI
What extra work needs to be done here? Just testing etc? Also- can we distribute for ARM Linux too? FWIW I was running native ARM when I was developing on Mac M1 and it worked perfectly, so I don't imagine we should see any issues here. |
yes just testing. The linux flatpak release already includes ARM builds, all that needs to be done there is add CI builds. I'll probably want to move the linux CI to target the Freedesktop SDK as well. |
@@ -101,8 +101,8 @@ void WindowMan::Initialize() { | |||
|
|||
SDL_GL_SetAttribute(SDL_GL_DOUBLEBUFFER, 1); | |||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_MAJOR_VERSION, 3); | |||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_MINOR_VERSION, 3); | |||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK, SDL_GL_CONTEXT_PROFILE_CORE); | |||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_MINOR_VERSION, 1); |
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.
Are these changes still necessary/valid with the changes in b5b183e?
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.
Oh, that CL is older. No idea why it's merge conflicting, but eh