Bug 412035 – media-libs/sdl-mixer: Make Timidity default and warn about FluidSynth

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description James Le Cuirot 2012-04-14 22:18:09 UTC]

Created attachment 308983 [details, diff]
Patch against sdl-mixer-1.2.12.ebuild

FluidSynth support was added to SDL_mixer as of 1.2.12. I implemented this myself. :) The ebuild has already been updated accordingly (thanks!) but there is potential for users to get caught out. Because FluidSynth requires a SoundFont and none are automatically provided, this support doesn't quite work out of the box. Failing to set SDL_SOUNDFONTS to something valid will either result in Timidity being used or silence if Timidity support is missing. For this reason, a pkg_postinst warning should be added and the timidity USE flag should be enabled by default. Please see the attached patch.

[Comment 1 Mr. Bones. (RETIRED) 2012-04-17 05:37:06 UTC]

added the warning.  No default use flag though - that would pull in timidity for a lot of people who previously didn't care at all about anything midi-related (like me).

[Comment 2 James Le Cuirot 2012-04-17 08:35:06 UTC]

Created attachment 309229 [details, diff]
Second patch against sdl-mixer-1.2.12.ebuild

Thanks for that. What if you wrap the dependencies in midi like so?

[Comment 3 Mr. Bones. (RETIRED) 2012-04-17 14:46:15 UTC]

i can dig it.

[Comment 4 James Le Cuirot 2012-04-17 20:04:37 UTC]

Great. Just one problem, I forgot to put IUSE="+timidity" in that second patch and you didn't notice either. :(