Skip to content

sdl3: Enable dummy audio driver#2618

Open
mborgerson wants to merge 1 commit intomesonbuild:masterfrom
mborgerson:fix/sdl3-dummy-audio-driver
Open

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

Conversation

@mborgerson
Copy link
Contributor

No description provided.

@UnixY2K
Copy link
Contributor

UnixY2K commented Feb 26, 2026

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
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

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