Skip to content

Conversation

@mdales
Copy link
Contributor

@mdales mdales commented Jan 28, 2026

I have an Apple Silicon mac, but I do not have Rosetta installed, and need to avoid doing so for work reasons.

By default, macOS alerts me I need Rosetta with camp, which I understand to be because the executable identified in the bundle is a shell script, and macOS can not detect the platform type for that, so assumes x86.

This PR adds an extra key to the bundle, LSArchitecturePriority, at build time to insert the platform of the build architecture, on the assumption that therse match - it doesn't look. to me that the Makefile would build a fat binary and inspecting camp.exe it seems to be just built for the host platform type.

Unfortunately I do not have access to an Intel mac to verify this works for the other platform, however I asked on the fediverse what uname -m returns on Intel macs and I can cofirm that it matches what is specified on that documentation for LSArchitecturePriority. So in theory, we should be good, but I can't confirm that in practice. I can just say it works now on my Apple Silicon mac sans Rosetta!

@rossberg
Copy link
Owner

Thanks, this looks fine, not that I understand it :). I don't have an Intel Mac to test either, but you can add macos-15-intel to the os matrix in .github/workflows/build.yml, then Github will try building it in CI. That said, I don't know if Raylib even works on Intel Macs anymore, but it's worth a try.

@mdales
Copy link
Contributor Author

mdales commented Jan 28, 2026

Thanks @rossberg - I've added that, but if it fails I guess I can just revert it to be a less complex fix and just hard code arm64?

@rossberg
Copy link
Owner

The Mac-Intel build took an eternity but actually has succeeded, which I find rather amazing. So perhaps leave it in for now?

@rossberg rossberg merged commit bde9742 into rossberg:master Jan 28, 2026
4 checks passed
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.

2 participants