On 20/08/2010 15:34, Kevin H. Hobbs wrote:
When Mesa is built with :

./autogen.sh \
   --prefix=/home/kevin/mesa_xlib/ \
   --with-driver=xlib \
   --disable-gallium \
   --disable-gl-osmesa

The libGL.so that is produced has undefined symbols from talloc but
talloc is not linked.

... and this breaks the cygwin build (see tinderbox [1]), as PE/COFF requires all symbols to be defined at link time.

I think the attached patch is the way to fix this.

[1] http://tinderbox.freedesktop.org/builds/2010-08-27-0016/logs/libGL/#build

>From 6b8dac531b6a677a3df835eb68dc37878276ad29 Mon Sep 17 00:00:00 2001
From: Jon TURNEY <jon.tur...@dronecode.org.uk>
Date: Fri, 27 Aug 2010 13:31:54 +0100
Subject: [PATCH] Add talloc to dependencies for libGL built with xlib driver

Signed-off-by: Jon TURNEY <jon.tur...@dronecode.org.uk>
---
 configure.ac |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index 0b60837..757571a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -578,8 +578,8 @@ xlib)
         GL_PC_LIB_PRIV="$GL_LIB_DEPS"
         GL_PC_CFLAGS="$X11_INCLUDES"
     fi
-    GL_LIB_DEPS="$GL_LIB_DEPS $SELINUX_LIBS -lm -lpthread"
-    GL_PC_LIB_PRIV="$GL_PC_LIB_PRIV $SELINUX_LIBS -lm -lpthread"
+    GL_LIB_DEPS="$GL_LIB_DEPS $SELINUX_LIBS -lm -lpthread $TALLOC_LIBS"
+    GL_PC_LIB_PRIV="$GL_PC_LIB_PRIV $SELINUX_LIBS -lm -lpthread $TALLOC_LIBS"
 
     # if static, move the external libraries to the programs
     # and empty the libraries for libGL
-- 
1.7.1

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to