SDL-cvs and XFree86 extension problem

Hello,

I always build SDL in a different directory, to not pollute the original
source tree, and to compile it for several targets (Linux and Atari) from
the same source tree. Unfortunately, a little problem came when compiling
the linux version: included Xfree includes in the SDL archive are not
found. It does not work for me (maybe a spelling error), but you get the
idea for what is needed. Hope to get a fixed CVS version soon…

— /home/patrice/src/cvs/Libsdl/SDL12/configure.in Fri Jun 14 14:35:07
2002 +++ configure.in Wed Jun 19 23:50:08 2002 @@ -548,7 +548,7 @@
AC_PATH_X
AC_PATH_XTRA
if test x$have_x = xyes; then

  •        CFLAGS="$CFLAGS $X_CFLAGS -DENABLE_X11 -Iinclude -Isrc/video"
    
  •        CFLAGS="$CFLAGS $X_CFLAGS -DENABLE_X11 -I\$(top_srcdir)/include -I\$(top_srcdir)/src/video"
           if test x$ac_cv_func_shmat != xyes; then
               CFLAGS="$CFLAGS -DNO_SHARED_MEMORY"
           fi-- 
    

Patrice Mandin
WWW: http://membres.lycos.fr/pmandin/
Programmeur Linux, Atari
Sp?cialit?: D?veloppement, jeux

Hello,

I always build SDL in a different directory, to not pollute the original
source tree, and to compile it for several targets (Linux and Atari) from
the same source tree. Unfortunately, a little problem came when compiling
the linux version: included Xfree includes in the SDL archive are not
found. It does not work for me (maybe a spelling error), but you get the
idea for what is needed. Hope to get a fixed CVS version soon…

Thanks, your fix is now in CVS.

See ya!
-Sam Lantinga, Software Engineer, Blizzard Entertainment

Dans l’article <mailman.1024595046.29612.sdl at libsdl.org>, “Sam Lantinga”
a tapot? avec ses petits doigts:

Thanks, your fix is now in CVS.

… and still does not work. Luckily, in CheckNasm function, the correct
spelling is done, so here is the RIGHT fix:

— /home/patrice/src/cvs/Libsdl/SDL12/configure.in Fri Jun 21 10:28:38 2002
+++ configure.in Fri Jun 21 11:16:37 2002
@@ -548,7 +548,7 @@
AC_PATH_X
AC_PATH_XTRA
if test x$have_x = xyes; then

  •        CFLAGS="$CFLAGS $X_CFLAGS -DENABLE_X11 -I\$(top_srcdir)/include -I\$(top_srcdir)/src/video"
    
  •        CFLAGS="$CFLAGS $X_CFLAGS -DENABLE_X11 -I$srcdir/include -I$srcdir/src/video"
           if test x$ac_cv_func_shmat != xyes; then
               CFLAGS="$CFLAGS -DNO_SHARED_MEMORY"
           fi
    

So everybody learning how to do correct configure scripts (like me),
$top_srcdir is only for Makefile.am scripts. Use $srcdir in the
configure.in script. $top_srcdir is used in the configure.in script to
generate the CFLAGS variable for Makefile scripts.

Now I tested it, and it works. It was very annoying to not have
fullscreen modes anymore under X11…–
Patrice Mandin
WWW: http://membres.lycos.fr/pmandin/
Programmeur Linux, Atari
Sp?cialit?: D?veloppement, jeux