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

Migrate to SFML 3.0.0 #13276

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoshuaVandaele
Copy link

@JoshuaVandaele JoshuaVandaele commented Jan 15, 2025

SFML 3.0.0 released last week, with some breaking changes. I'm taking the occasion to make SFML a submodule, as well as migrating the code to work with >=3.0.0.

So far, I've tested using Netplay on Linux with a copy of Mario Kart Wii and everything seems to works fine, but I'd appreciate more testing, particularly with Gamecube games since I do not own any.

@JoshuaVandaele JoshuaVandaele marked this pull request as ready for review January 15, 2025 13:47
@JoshuaVandaele JoshuaVandaele force-pushed the sfml-3.0.0 branch 2 times, most recently from 01d196c to 953ec83 Compare January 15, 2025 18:22
@JoshuaVandaele
Copy link
Author

The change in graphical rendering seems to happen to multiple PRs so I think this is unrelated, however I'm not sure why the OSX build fails and I'm unable to test it locally considering I don't own a single Apple device, I'm unsure how to proceed?

@BhaaLseN
Copy link
Member

OSX looks like it uses an old compiler for SFML, as it complains about constexpr and noexcept. Which is...weird.

(And yes, feel free to ignore fifoci.)

@JoshuaVandaele
Copy link
Author

Oh wow I fixed it with my first attempt, amazing! Let me rebase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants