Small SDL patch to improve building under Linux

Could someone add this patch to the SDL CVS?
It fixes building of extensions that depend
on SDL runtime linker settings. After adding
this patch, I can build SMPEG even if SDL is
installed into a dir other than /usr/local.

Index: configure.in===================================================================
RCS file: /cvs/SDL/configure.in,v
retrieving revision 1.49
diff -u -r1.49 configure.in
configure.in 2000/03/14 19:59:26 1.49
+++ configure.in 2000/05/14 01:53:36
@@ -763,6 +763,17 @@
AM_CONDITIONAL(TARGET_BEOS, test $ARCH = beos)
AM_CONDITIONAL(TARGET_MACOS, test $ARCH = macos)

+# Set runtime shared library paths as needed
+SDL_RLD_FLAGS=""
+
+if test $ARCH = linux; then

  • SDL_RLD_FLAGS="-Wl,-rpath=${exec_prefix}/lib"
    +fi

+if test $ARCH = solaris; then

  • SDL_RLD_FLAGS="-R=${exec_prefix}/lib"
    +fi

dnl Output the video drivers we use
if test x$enable_video = xtrue; then
if test “$VIDEO_SUBDIRS” = “”; then
@@ -778,6 +789,7 @@
dnl Expand the cflags and libraries needed by apps using SDL
AC_SUBST(SDL_CFLAGS)
AC_SUBST(SDL_LIBS)
+AC_SUBST(SDL_RLD_FLAGS)

dnl Expand the include directories for building SDL
CFLAGS="$CFLAGS -I$(top_srcdir)/include"
Index: sdl-config.in

RCS file: /cvs/SDL/sdl-config.in,v
retrieving revision 1.5
diff -u -r1.5 sdl-config.in
sdl-config.in 2000/02/28 19:04:41 1.5
+++ sdl-config.in 2000/05/14 01:53:36
@@ -45,19 +45,11 @@
echo $includes -I at includedir@/SDL @SDL_CFLAGS@
;;
–libs)

  •  if [ "`uname`" = "SunOS" ]; then
    
  •    libdirs="-L at libdir@ -R at libdir@"
    
  •  else
    
  •    libdirs="-L at libdir@"
    
  •  fi
    
  •  libdirs="-L at libdir@ @SDL_RLD_FLAGS@"
     echo $libdirs @SDL_LIBS@
     ;;
    
    –static-libs)
  •  if [ "`uname`" = "SunOS" ]; then
    
  •    libdirs="-L at libdir@ -R at libdir@"
    
  •  else
    
  •    libdirs="-L at libdir@"
    
  •  fi
    
  •  libdirs="-L at libdir@ @SDL_RLD_FLAGS@"
     echo $libdirs @SDL_LIBS@ @SYSTEM_LIBS@
     ;;
    
    *)

thanks
Mo DeJong
Red Hat Inc.

Could someone add this patch to the SDL CVS?
It fixes building of extensions that depend
on SDL runtime linker settings. After adding
this patch, I can build SMPEG even if SDL is
installed into a dir other than /usr/local.

I thought I had. I’ll apply it Monday.

BTW, E3 rocked. If anybody was there and I missed you, sorry!

See ya,
-Sam Lantinga, Lead Programmer, Loki Entertainment Software

P.S. I’ll put up a picture of me and Duke Nukem later. :slight_smile: