Skip to content

Update all to latest main#268

Merged
smoogipoo merged 2 commits into
ppy:masterfrom
Happypig375:patch-1
Feb 24, 2026
Merged

Update all to latest main#268
smoogipoo merged 2 commits into
ppy:masterfrom
Happypig375:patch-1

Conversation

@Happypig375

Copy link
Copy Markdown
Contributor

No description provided.

@hwsmm

hwsmm commented Feb 8, 2026

Copy link
Copy Markdown
Collaborator

Hmm... I'm not sure about using tagged releases for our builds since we haven't really done it so far.

Either way. we need to use the latest commit of SDL to include libsdl-org/SDL@bddf6d3 since osu!framework needs it.

@Happypig375

Copy link
Copy Markdown
Contributor Author

Ok, I updated all to main.

@Happypig375 Happypig375 changed the title Update SDL to 3.4.0, SDL_mixer to 3.1.2, SDL_image to 3.4.0, SDL_ttf to main Update all to latest main Feb 8, 2026
@Susko3

Susko3 commented Feb 8, 2026

Copy link
Copy Markdown
Member

I'm not sure about using tagged releases for our builds since we haven't really done it so far.

I would like to switch to tagged releases at some point (they should be more stable, sometimes they even get fixes backported), but if we need new functionality or fixes we must use main. So let's keep it on main for now.

we need to use the latest commit of SDL to include libsdl-org/SDL@bddf6d3 since osu!framework needs it.

We don't need it as we implemented a local workaround in ppy/osu-framework#6703 (confirmed working as expected), but it makes sense to pull it in.

@Susko3 Susko3 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've run bindgen locally and it produces the same changes. All the submodule commits are legit and they are recent enough. Basic tests pass on a local SDL3 build.

@peppy
peppy requested a review from smoogipoo February 9, 2026 05:49
@smoogipoo
smoogipoo merged commit 87f9535 into ppy:master Feb 24, 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.

4 participants