Skip to content

Commit

Permalink
Fixed bug #812
Browse files Browse the repository at this point in the history
 Max Horn      2009-09-27 11:58:18 PDT

I got build failure reports from various people for SDL-mixer on OS X when
compiling in 64bit mode, both in 10.5 and 10.6. The reason: Deprecated
QuickTime APIs that have been removed for 64bit binaries, see below.

Proposed short-term workaround: Disable or just remove the crappy "native" MIDI
code. It's rather bad anyway (I should know, i wrote it ;).

Mid/long-term solution: Write new CoreAudio based code to replace it. That code
then would work on all OS X versions (well, at least on 10.4 upwards for sure),
be more accurate, sound better and take up less system resources. And be
simpler, did I mention that? Generating MIDI output with CoreAudio is pretty
easy, we do it in ScummVM, Exult, Pentagram, DOSBox, ...
  • Loading branch information
slouken committed Sep 27, 2009
1 parent a93f52f commit 7b44d0d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions configure.in
Expand Up @@ -221,8 +221,10 @@ AC_HELP_STRING([--enable-music-native-midi], [enable native MIDI music output [[
EXTRA_LDFLAGS="$EXTRA_LDFLAGS -lwinmm"
;;
*-*-darwin*)
use_music_native_midi=yes
EXTRA_LDFLAGS="$EXTRA_LDFLAGS -Wl,-framework,QuickTime -Wl,-framework,CoreServices"
# This doesn't work on Mac OS X 10.5+
# Max Horn (the original author) recommends disabling it for now.
#use_music_native_midi=yes
#EXTRA_LDFLAGS="$EXTRA_LDFLAGS -Wl,-framework,QuickTime -Wl,-framework,CoreServices"
;;
esac
if test x$use_music_native_midi = xyes; then
Expand Down

0 comments on commit 7b44d0d

Please sign in to comment.