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 - sdl 1.2.10 fails compile on ggi
Summary: sdl 1.2.10 fails compile on ggi
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: 1.2.9
Hardware: x86 Linux
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-18 20:07 UTC by Robin Cook
Modified: 2019-01-10 16:35 UTC (History)
1 user (show)

See Also:

Attachments
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 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.