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

Update bindings (no merging needed) #103

Merged
merged 3 commits into from
Jul 31, 2024
Merged

Conversation

hwsmm
Copy link
Collaborator

@hwsmm hwsmm commented Jul 21, 2024

I'll keep this open to reduce duplicate effort on updating bindings.

Reviews are welcomed, but I don't think we need to update bindings every time SDL changes their API, so you don't need to merge this now. I'm doing this because I usually keep -git packages up to date. updating SDL3 broke osu! again

Feel free to take over maintaining if I look inactive here!

Relevant osu!framework branch

My ppy/osu-framework#6002 (SDL3 Audio) also follows this PR, so expect build failures there without this.


Some notable changes:

@IsaacMarovitz
Copy link

I'm looking into initial SDL3 support for Ryujinx and we need SDL_GetJoystickGUIDForID() which is not available without these changes.

Copy link
Contributor

@smoogipoo smoogipoo left a comment

Choose a reason for hiding this comment

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

lgtm

@smoogipoo smoogipoo merged commit 7455e1c into ppy:master Jul 31, 2024
1 check passed
@smoogipoo
Copy link
Contributor

New package available as https://www.nuget.org/packages/ppy.SDL3-CS/2024.731.0

@hwsmm hwsmm deleted the update-bindings branch August 11, 2024 15:20
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.

3 participants