Autoconf based SDL in CVS!

When running configure I get the following message between the checks
for ALSA and ARTS:

ls: audio/alsa/*.c: No such file or directory

This appears to be caused by the line

SDL_SRCS="$SDL_SRCS (cd $srcdir && ls audio/alsa/*.c)"

in configure.in. It seems this is the only place where SDL_SRCS shows up
in the entire SDL12 source tree. Maybe a leftover?

Yep, fixed in CVS, thanks!

-Sam Lantinga, Senior Software Engineer, Blizzard Entertainment

confirmed, works nicely now
-mikeOn Monday 20 February 2006 15:12, Sam Lantinga wrote:

again, configure picks up my custom LDFLAGS, but the build system doesnt
utilize them …

This should be fixed in CVS, thanks!

hmm, should include/SDL_config.h even be in cvs ? it is generated
automatically from include/SDL_config.h.in isnt it ?
-mikeOn Sunday 19 February 2006 22:17, Sam Lantinga wrote:

I’ve fully removed automake from SDL in CVS. For those of you who build
SDL with ./configure; make; make install, please grab the latest CVS
snapshot and let me know if there are any problems:
http://www.libsdl.org/cvs/SDL-1.2.10.tar.gz

ok, latest cvs, and hopefully ive doing this from clean sources (it worked
last nite at least), but my build now fails at the final link due to ALSA
issues …

$ ./configure --enable-alsa

checking for dmedia audio support… no
checking for ALSA CFLAGS…
checking for ALSA LDFLAGS… -lasound -lm -ldl -lpthread
checking for libasound headers version >= 0.9.0… found.
checking for snd_ctl_open in -lasound… yes
– /usr/lib/libasound.so.* -> libasound.so.2
– /usr/lib/libasound.so.* -> libasound.so.2
checking for dlvsym in -ldl… (cached) yes

SOURCES="./src/.c ./src/audio/.c ./src/cdrom/.c ./src/cpuinfo/.c ./src/events/.c ./src/file/.c ./src/joystick/.c ./src/stdlib/.c ./src/thread/.c ./src/timer/.c ./src/video/.c ./src/video/dummy/.c ./src/audio/disk/.c ./src/loadso/dlopen/.c ./src/audio/dsp/.c ./src/audio/dma/.c ./src/audio/alsa/.c ./src/audio/alsa/.c ./src/audio/arts/.c ./src/audio/esd/.c ./src/video/x11/.c ./src/video/Xext/Xxf86dga/.c ./src/video/dga/.c ./src/video/Xext/Xxf86vm/.c ./src/video/Xext/Xv/.c ./src/video/Xext/Xinerama/.c ./src/video/Xext/XME/.c ./src/video/fbcon/.c ./src/thread/pthread/SDL_systhread.c ./src/thread/pthread/SDL_syssem.c ./src/thread/pthread/SDL_sysmutex.c ./src/thread/pthread/SDL_syscond.c ./src/joystick/linux/.c ./src/cdrom/linux/.c ./src/timer/unix/*.c"

$ make

build-deps:980: warning: overriding commands for target
build/SDL_alsa_audio.lo' build-deps:959: warning: ignoring old commands for targetbuild/SDL_alsa_audio.lo’

build/.libs/SDL_alsa_audio.o:(.data.rel.local+0x0): multiple definition of
`ALSA_bootstrap’
build/.libs/SDL_alsa_audio.o:(.data.rel.local+0x0): first defined here

the reason for the duplicate definition is that SDL_alsa_audio.lo is listed
twice in the call to libtool --mode=link -o build/libSDL.la … and this is
because SOURCES="" is fed ./src/audio/alsa/*.c twice in configure …
-mikeOn Sunday 19 February 2006 22:17, Sam Lantinga wrote:

I’ve fully removed automake from SDL in CVS. For those of you who build
SDL with ./configure; make; make install, please grab the latest CVS
snapshot and let me know if there are any problems:
http://www.libsdl.org/cvs/SDL-1.2.10.tar.gz

the reason for the duplicate definition is that SDL_alsa_audio.lo is listed
twice in the call to libtool --mode=link -o build/libSDL.la … and this is
because SOURCES="" is fed ./src/audio/alsa/*.c twice in configure …

Fixed!
-Sam Lantinga, Senior Software Engineer, Blizzard Entertainment

hmm, should include/SDL_config.h even be in cvs ? it is generated
automatically from include/SDL_config.h.in isnt it ?

Yes, for configure-based builds. There are also a bunch of build projects
which don’t use the configure system (e.g. OS/2, Dev Studio, etc.), that
need to have a valid SDL_config.h. I’m probably going to create a default
that uses C preprocessor defines to do the right thing on most platforms,
and then the configure process will create a custom header if needed.

I’m still not entirely sure what the best way to handle it is.

-Sam Lantinga, Senior Software Engineer, Blizzard Entertainment

confirmed, cheers !
-mikeOn Monday 20 February 2006 18:19, Sam Lantinga wrote:

the reason for the duplicate definition is that SDL_alsa_audio.lo is
listed twice in the call to libtool --mode=link -o build/libSDL.la
and this is because SOURCES="" is fed ./src/audio/alsa/*.c twice in
configure …

Fixed!