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.

Sign up for GitHub

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

Jump to bottom

Move away from SDL2 and use Qt Sound/Multimedia instead #2104

Open
jwrober opened this issue Dec 31, 2023 · 2 comments
Open

Move away from SDL2 and use Qt Sound/Multimedia instead #2104

jwrober opened this issue Dec 31, 2023 · 2 comments
Labels
enhancement New feature or request gui This issue requires changes to the user interface

Comments

@jwrober
Copy link
Collaborator

jwrober commented Dec 31, 2023

Is your feature request related to a problem? Please describe.
We have had some recent issues with SDL2 and SDL2_mixer autodetection and MacOS.

Describe the solution you'd like
@lmoureaux suggested that we move away from this and use native Qt for this. Good to include as part of the migration to Qt6 #1198

Describe alternatives you've considered
None. Removing a dependency on an external library is always a good idea.

Additional context
None
@jwrober jwrober added enhancement New feature or request gui This issue requires changes to the user interface labels Dec 31, 2023
@lmoureaux
Copy link
Contributor

None. Removing a dependency on an external library is always a good idea.

Note that we'd technically be replacing the dependency. The main advantage is that everything would be more Qt-centric.

@lmoureaux
Copy link
Contributor

Why migrate now: The Qt Multimedia API was overhauled in Qt6 and the SDL_mixer API was overhauled in SDL3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request gui This issue requires changes to the user interface
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lmoureaux @jwrober