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

Upgrade to SDL3 #1332

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Upgrade to SDL3 #1332

wants to merge 5 commits into from

Conversation

Taiko2k
Copy link
Owner

@Taiko2k Taiko2k commented Dec 6, 2024

Work in progress.

Will need to consider the impact this will have on packaging as SDL3 is not officially released yet.

@C0rn3j
Copy link
Collaborator

C0rn3j commented Dec 6, 2024

As a note, Dota 2 and Counter-Strike 2 have been running on SDL3 for a while, so SDL3 itself is pretty battle tested despite not having a stable release.

PySDL3 may be another story.

SDL3 defaults to Xwayland today on most setups due to libsdl-org/SDL#9383, and the necessary protocol implementations on Plasma (for example), which uses kwin, was pushed to 6.4 just last week... as a note Plasma is currently on 6.2 and new version releases aren't exactly speedy.

Xwayland isn't exactly a problem, but a thing to note.


This will need all the requirements files(root+extra) edited to bump pysdl2 to pysdl3, pysdl2-dll commented out (asked about SDL3 plans here a-hurst/pysdl2-dll#36), and spec files for all OSs edited to comment out pysdl2-dll.

The CI yaml files will need to be edited to install SDL3 - that one is going to be fun I imagine.

With that, I believe the CI should build fine.

Or maybe we'll run into pyinstaller not supporting pysdl3, who knows, adventure awaits!


pre-forcepush patchfile just in case - 1332.zip

Linux: No SDL3 in Ubuntu 24.04
macOS: No SDL3 in brew
Windows: No SDL3 in MINGW64 (Arch does not ship it outside of AUR either)

While CI builds, launching the image crashes on SDL3 not being found.
We have no pysdl3-dll to fall back on either.

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