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

Libretro related buildfixes and new presets #3365

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

Conversation

Ghabry
Copy link
Member

@Ghabry Ghabry commented Mar 6, 2025

This fixes some flaws encountered while updating the libretro core to the latest version on their buildbot.

They offer a recent m1 builder so I was able to test building a libretro core for iOS and tvOS. It works besides some minor issues that are fixed here.

iOS/tvOS presets are not included (more work needed to make them useful)

Ghabry added 7 commits March 6, 2025 16:42
CMake thinks we are crosscompiling (prevents that it pulls in system headers) which makes --check fail.

Provide an empty CMAKE_CROSSCOMPILING_EMULATOR to fix this.
Also pass all supported OSX_ARCHITECTURES by default
Simplifies the preset for emscripten a bit and is also needed for iOS and tvOS builds
Small variable fixes in the CMake files
@Ghabry Ghabry added this to the 0.8.1 milestone Mar 6, 2025
@Ghabry
Copy link
Member Author

Ghabry commented Mar 6, 2025

warmerhoven on the RetroArch Discord confirmed that it works on iOS and tvOS and shared this screenshot with me:

grafik

@Ghabry
Copy link
Member Author

Ghabry commented Mar 11, 2025

Minor update: Small change to the logging callback registration because of a bug in RetroArch... (surprise surprise)

Was also asked to disable our logfile by default as its redundant (RetroArch / other frontends put it onto their own logfile via that log_cb).


Second minor update: The logging API does not automatically insert linebreaks 🤦

white-axe added a commit to white-axe/mkxp-z that referenced this pull request Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants