Skip to content

sdl3: Enable dummy audio driver - #2618

Closed
mborgerson wants to merge 1 commit into
mesonbuild:masterfrom
mborgerson:fix/sdl3-dummy-audio-driver
Closed

sdl3: Enable dummy audio driver#2618
mborgerson wants to merge 1 commit into
mesonbuild:masterfrom
mborgerson:fix/sdl3-dummy-audio-driver

Conversation

@mborgerson

Copy link
Copy Markdown
Contributor

No description provided.

@UnixY2K

UnixY2K commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

I would advise to put these changes in your other PR #2666 so the fixes can be included along with the latest version.

if possible please add these changes.

Audio is not working on macos for me. This diff makes it work on my machine but I also don't have much knowledge about this so I just share it here.

diff --git a/subprojects/packagefiles/sdl3/meson.build b/subprojects/packagefiles/sdl3/meson.build
index c5c62083..38329773 100644
--- a/subprojects/packagefiles/sdl3/meson.build
+++ b/subprojects/packagefiles/sdl3/meson.build
@@ -414,6 +414,7 @@ elif host_machine.subsystem() == 'macos'
   sdl_deps += dependency(
     'appleframeworks',
     modules: [
+      'AudioToolbox',
       'CoreFoundation',
       'Cocoa',
       'CoreAudio',
diff --git a/subprojects/packagefiles/sdl3/src/audio/meson.build b/subprojects/packagefiles/sdl3/src/audio/meson.build
index e988f373..66bcdc10 100644
--- a/subprojects/packagefiles/sdl3/src/audio/meson.build
+++ b/subprojects/packagefiles/sdl3/src/audio/meson.build
@@ -4,6 +4,7 @@ if host_machine.system() == 'windows'
   subdir('directsound')
   #  subdir('wasapi')
 elif host_machine.system() == 'darwin'
+  cdata.set('SDL_AUDIO_DRIVER_COREAUDIO', 1)
   subdir('coreaudio')
 else
   cdata.set('SDL_AUDIO_DRIVER_PIPEWIRE', 1)

Originally posted by @Ben3eeE in #2026

@mborgerson

Copy link
Copy Markdown
Contributor Author

@UnixY2K

I would advise to put these changes in your other PR #2666

They are separate concerns. I don't think combining this PR into #2666 is really necessary, but if you want to take over and work on a PR of your own and cherry pick my patches you are welcome to

@mborgerson mborgerson closed this May 5, 2026
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