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 133

Summary: X11 build failure on MacOS X
Product: SDL Reporter: Sam Lantinga <slouken>
Component: videoAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2    
Version: HG 1.2   
Hardware: PowerPC   
OS: Mac OS X (All)   

Description Sam Lantinga 2006-02-02 01:44:36 UTC
configure reports:
checking for Cocoa framework... yes
checking for X... no
checking for OpenGL (GLX) support... no
checking for pthreads... yes
checking for recursive mutexes... no
checking for pthread semaphores... yes
...

Then building the library fails:
In file included from SDL_QuartzVideo.h:63,
                 from SDL_QuartzEvents.m:23:
../../../include/SDL_syswm.h:58:22: X11/Xlib.h: No such file or directory
../../../include/SDL_syswm.h:59:23: X11/Xatom.h: No such file or directory
In file included from SDL_QuartzVideo.h:63,
                 from SDL_QuartzEvents.m:23:
../../../include/SDL_syswm.h:76: error: parse error before "XEvent"
../../../include/SDL_syswm.h:76: warning: no semicolon at end of struct or union
../../../include/SDL_syswm.h:76: warning: no semicolon at end of struct or union
../../../include/SDL_syswm.h:77: warning: type defaults to `int' in declaration of `event'
etc.
Comment 1 Sam Lantinga 2006-02-02 01:46:06 UTC
We can't really default on anything in SDL_syswm.h that may not exist on a platform.  This header is included by the application code and needs to work wherever the library is installed.
Comment 2 Ryan C. Gordon 2006-02-02 02:06:03 UTC
Huh, I get this:

checking for Cocoa framework... yes
checking for X... libraries /usr/X11R6/lib, headers


I always assumed this was just part of the Mac OS SDK, but it probably relies on installing the X11 package, which is unchecked by default in the OS (not developer tools!) install. Hmm.

--ryan.

Comment 3 Sam Lantinga 2006-03-09 10:21:00 UTC
SDL_config.h takes care of this now. :)