ALSA sound has problems on some sound cards. This couse high cpu speed aboud 2000%. Could you add option into configure.ac to use SDL sound with GTK-UI. SDL sound works with out any issue.
So at the moment it's indeed impossible to select the SDL audio driver unless you also use the SDL UI, and it's also impossible to use the SDL UI but have a different audio driver.
But it's not just that, we have 4 different audio drivers for Linux (alsa, sdl, oss, libao) and perhaps we might have more in the future (pulseaudio?). So instead of adding options to enable/disable each one of them I thought it would be much better to have a single generic option to select which one to enable. If the option is not used a default driver would be chosen, the same as before.
I split the work in three patches for clarity:
1) Remove duplicate AM_PATH_SDL in configure.ac
2) Add new --with-audio-driver option
3) Fix detection of Solaris and HP/UX audio drivers
The patches were created with git-format-patch so you can apply them with "git am <patch-file>" and keep the commit messages.</patch-file>
Here's a second version of the patches. I thought that it's probably better to give the SDL driver preference over libao and oss, so that's how it differs from the first set of patches.
So at the moment it's indeed impossible to select the SDL audio driver unless you also use the SDL UI, and it's also impossible to use the SDL UI but have a different audio driver.
But it's not just that, we have 4 different audio drivers for Linux (alsa, sdl, oss, libao) and perhaps we might have more in the future (pulseaudio?). So instead of adding options to enable/disable each one of them I thought it would be much better to have a single generic option to select which one to enable. If the option is not used a default driver would be chosen, the same as before.
I split the work in three patches for clarity:
1) Remove duplicate AM_PATH_SDL in configure.ac
2) Add new --with-audio-driver option
3) Fix detection of Solaris and HP/UX audio drivers
The patches were created with git-format-patch so you can apply them with "git am <patch-file>" and keep the commit messages.</patch-file>
Looks really good to me. I'll commit before the next release, barring objections.
Here's a second version of the patches. I thought that it's probably better to give the SDL driver preference over libao and oss, so that's how it differs from the first set of patches.
Thanks all. Committed in [96b62a], [77cbce] and [8a5a13].
Related
Commit: [77cbce]
Commit: [8a5a13]
Commit: [96b62a]