can't find SDL on newer Ubuntu

[GoogleCodeExporter]

What steps will reproduce the problem?
1. sudo make
2. sudo ./configure
3. sudo make

What is the expected output? What do you see instead?
It should compile the game but instead it gives errors regarding SDL.h

What version of the product are you using? On what operating system?
Quadra Trunk 1.3
Ubuntu Intrepid 8.10

Please provide any additional information below.
I found out that in ubuntu SDL library installs in /usr/include/SDL
in the source files in quadra this library is included as "SDL.h" which
means that system can't find it. Replaced every instanace of "SDL_*.H" with
<SDL/SDL*.H> *being the name of the sub header SDL_video, SDL_sound etc.
Then it compiled successfully.
I uploaded the working copy to 

http://www.josecgomez.com/quadra/quadra%20_1.3_intrepid.tar.gz

I tried to attach it here but it wouldn't let me because its too big. I
don't know if the developers would like to make this an actual change to
the source code or not but it will be useful for those that can't compile
it in Ubuntu Intrepid.

Original issue reported on code.google.com by jose.go...@gmail.com on 19 Feb 2009 at 9:35

[GoogleCodeExporter]

Ah, yes, I found this problem too. The location of the SDL headers has been 
changed
in a recent version of Ubuntu (not sure when?). Most programs using SDL use the
"sdl-config" program to obtain the location of the SDL headers and library, but
Quadra didn't, so when they were moved, Quadra didn't follow. Compiling with the
following "configure" option seems to work at the moment without needing to 
modify
the source code:

./configure CPPFLAGS=-I/usr/include/SDL

Note that you do not need to use "sudo" to compile quadra, only for the eventual
"make install".

I'm not considering this a fix, just a workaround, the proper fix will be to 
improve
the detection of SDL, so I'm leaving this issue open until it is properly 
resolved.
Thanks for reporting it!

Original comment by pphaneuf on 19 Feb 2009 at 10:09

• Changed title: can't find SDL on newer Ubuntu
• Changed state: Accepted
• Added labels: Type-Defect, Priority-Medium, OpSys-Linux

[GoogleCodeExporter]

Original comment by pphaneuf on 19 Feb 2009 at 10:30

[GoogleCodeExporter]

Should be fixed by r732.

Original comment by pphaneuf on 21 Feb 2009 at 11:03

• Changed state: Fixed