Problems with timidity

[Max Horn]

I get this when I try to compile current SDL_sound:

[...]

cc -DHAVE_CONFIG_H -I. -I../../../SDL_sound/decoders -I.. 
-I../../../SDL_sound -Itimidity -g -O2 -I/sw/include 
-I/sw/include/SDL -D_THREAD_SAFE 
-F/System/Library/Frameworks/Carbon.framework 
-F/System/Library/Frameworks/Cocoa.framework -I/sw/include 
-I/sw/include/smpeg -I/sw/include -I/sw/include/SDL -D_THREAD_SAFE 
-F/System/Library/Frameworks/Carbon.framework 
-F/System/Library/Frameworks/Cocoa.framework -I/sw/include 
-D_REENTRANT -c ../../../SDL_sound/decoders/midi.c 
-Wp,-MD,.deps/midi.TPlo  -fno-common -DPIC -o .libs/midi.lo
../../../SDL_sound/decoders/midi.c:47: header file 'timidity.h' not found
../../../SDL_sound/decoders/midi.c:94: undefined type, found `MidiSong'

[...a couple of dozen similiar errors].


The reason it doesn't find timidity.h is that I am not building 
inside my CVS dir, but inside a seperate build directory (that's very 
nice if you e.g. want multiple different builds with different 
paramaters).

So if you look at that build statments, it has "-Itimidity" which of 
course is not going to work in my case. You should never write 
something as what is in the current decoders/Makefile.am:

INCLUDES = -I$(top_srcdir) -Itimidity

This is bad.  Instead use:

INCLUDES = -I$(top_srcdir) -I$(top_srcdir)/decoders/timidity


I fixed this in CVS, but wanted to report it here anyway for future 
use :) Oh and for the CHANGELOG: we now have 2002, I fixed some 
entries accordingly.

Max
-- 
-----------------------------------------------
Max Horn
Software Developer

email: <mailto:max at quendi.de>
phone: (+49) 6151-494890