We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 221 - More flexible SDL_config_macosx.h
Summary: More flexible SDL_config_macosx.h
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: HG 1.2
Hardware: PowerPC Mac OS X (All)
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks: 53
  Show dependency tree / graph
 
Reported: 2006-05-09 15:48 UTC by Anders F Bj
Modified: 2006-05-10 03:26 UTC (History)
0 users

See Also:

Attachments
SDLsvn-config-macosx.patch (813 bytes, patch)
2006-05-09 15:49 UTC, Anders F Bj 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Anders F Bj 2006-05-09 15:48:29 UTC 
The current SDL_config_macosx.h is hardcoded to use Cocoa/Quartz,
it would be better if you could override this (if you wanted to)
to use another video driver like the Carbon/QuickDraw or even DSp.
Ditto for the two audio drivers, now both SoundManager/CoreAudio.
Comment 1 Anders F Bj 2006-05-09 15:49:20 UTC 
Created attachment 118 [details]
SDLsvn-config-macosx.patch
Comment 2 Sam Lantinga 2006-05-10 03:11:52 UTC 
The config file is supposed to define what is actually built for the current configuration.  Maybe we need SDL_config_carbon.h?
Comment 3 Anders F Bj 2006-05-10 03:16:25 UTC 
It was used conditionally in SDL_config_macos.h, so I continued with that.

i.e. (from include/SDL_config_macos.h)
/* Enable various cdrom drivers */
#if TARGET_API_MAC_CARBON
#define SDL_CDROM_DUMMY         1
#else
#define SDL_CDROM_MACOS         1
#endif

/* Enable various input drivers */
#if TARGET_API_MAC_CARBON
#define SDL_JOYSTICK_DUMMY      1
#else
#define SDL_JOYSTICK_MACOS      1
#endif

Splitting the Mac OS X SDL_config_macosx.h into "SDL_config_carbon.h" and "SDL_config_cocoa.h" could be done, not sure how popular it would be though ?
Comment 4 Sam Lantinga 2006-05-10 03:26:25 UTC 
This should be fixed in subversion now... I just added a special #if TARGET_API_MAC_CARBON section, like I did for MacOS Classic.