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 234

Summary: sdl 1.2.10 fails compile on ggi
Product: SDL Reporter: Robin Cook <rcook>
Component: buildAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2 CC: bobishmax
Version: 1.2.9   
Hardware: x86   
OS: Linux   

Description Robin Cook 2006-05-18 20:07:24 UTC 
gcc 4.1.0, libgii 1.0.1, libggi 2.2.1

 gcc -march=athlon-mp -mmmx -mfpmath=sse -msse -m3dnow -pipe -DPIC -fPIC -O3 -I./include -D_GNU_SOURCE=1 -fvisibility=hidden -DXTHREADS -D_REENTRANT$/bin/sh ./libtool --mode=compile gcc -march=athlon-mp -mmmx -mfpmath=sse -msse -m3dnow -pipe -DPIC -fPIC -O3  -I./include -D_GNU_SOURCE=1 -fvisibili$ gcc -march=athlon-mp -mmmx -mfpmath=sse -msse -m3dnow -pipe -DPIC -fPIC -O3 -I./include -D_GNU_SOURCE=1 -fvisibility=hidden -DXTHREADS -D_REENTRANT$./src/video/ggi/SDL_ggievents.c: In function 'GGI_PumpEvents':
./src/video/ggi/SDL_ggievents.c:65: error: 'uint32' undeclared (first use in this function)
./src/video/ggi/SDL_ggievents.c:65: error: (Each undeclared identifier is reported only once
./src/video/ggi/SDL_ggievents.c:65: error: for each function it appears in.)
./src/video/ggi/SDL_ggievents.c:65: error: expected ';' before 'sym'
make: *** [build/SDL_ggievents.lo] Error 1
make: *** Waiting for unfinished jobs....
 gcc -march=athlon-mp -mmmx -mfpmath=sse -msse -m3dnow -pipe -DPIC -fPIC -O3 -I./include -D_GNU_SOURCE=1 -fvisibility=hidden -DXTHREADS -D_REENTRANT$make: *** [build/SDL_x11dyn.lo] Error 1
Comment 1 Matt Turner 2006-05-20 22:30:35 UTC 
Unrelated, but:

You seem a little CFLAGS happy.  A few notes: -march=athlon-mp implies -msse -m3dnow and -mmmx (you don't need to specify them).  Also, -mfpmath=sse should only be used on P4s, last I heard.  The only reason for this was because the P4's FPU was god-awful
Comment 2 Ryan C. Gordon 2006-05-20 22:51:24 UTC 
I can't build the GGI target from here, but I just patched this error in the Subversion repository. Can you see if this fixes your problem?

--ryan.
Comment 3 Robin Cook 2006-05-21 00:17:58 UTC 
Those flags are set by the distribution I'm running but will bring it up with them.  I was able to get it to build by removing the  uint32 sym;

But I will check the cvs as well.
Comment 4 Ryan C. Gordon 2006-05-21 00:39:02 UTC 
All I did was remove that line, so we'll assume it's good to go, then.

--ryan.