# HG changeset patch # User Sam Lantinga # Date 1144937646 0 # Node ID 10b4827452a829bc63ebd1cd94b59d459c7c9baa # Parent aee7ea396f598a2971787e9f227782b02f37ed24 Fixed bug #185 The SDL platform header shouldn't dictate the OS target API for other applications. The build system should define the target API for building SDL internally. diff -r aee7ea396f59 -r 10b4827452a8 configure.in --- a/configure.in Thu Apr 13 14:04:13 2006 +0000 +++ b/configure.in Thu Apr 13 14:14:06 2006 +0000 @@ -2319,10 +2319,13 @@ # Next line is broken, and a few files below require Mac OS X (full) ARCH=macosx - # HACK: Reset EXTRA_LDFLAGS; the only thing it contains at this point is - # -lm which is not needed under Mac OS X. But for some reasons it also - # tends to contain spurious -L switches, which we don't want to use - # here or in sdl-config. Hence we reset it. + # Mac OS X builds with both the Carbon and OSX APIs at the moment + EXTRA_CFLAGS="-DTARGET_API_MAC_CARBON -DTARGET_API_MAC_OSX" + + # HACK: Reset EXTRA_LDFLAGS; the only thing it contains at this point + # is -lm which is not needed under Mac OS X. But for some reasons it + # also tends to contain spurious -L switches, which we don't want to + # use here or in sdl-config. Hence we reset it. EXTRA_LDFLAGS="" CheckDummyVideo diff -r aee7ea396f59 -r 10b4827452a8 include/SDL_platform.h --- a/include/SDL_platform.h Thu Apr 13 14:04:13 2006 +0000 +++ b/include/SDL_platform.h Thu Apr 13 14:14:06 2006 +0000 @@ -64,16 +64,10 @@ #if defined(macintosh) #undef __MACOS__ #define __MACOS__ 1 -#define TARGET_API_MAC_OS8 1 -#define TARGET_API_MAC_CARBON 0 -#define TARGET_API_MAC_OSX 0 #endif #if defined(__GNUC__) && (defined(__APPLE_CC__) || defined(__APPLE_CPP__)) #undef __MACOSX__ #define __MACOSX__ 1 -#define TARGET_API_MAC_OS8 0 -#define TARGET_API_MAC_CARBON 1 -#define TARGET_API_MAC_OSX 1 #endif #if defined(__NetBSD__) #undef __NETBSD__