On 09/15/2012 06:04 PM, Török Edwin wrote:
On 09/13/2012 11:19 AM, Oliver McFadden wrote:
Signed-off-by: Oliver McFadden <oliver.mcfad...@linux.intel.com>
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index abce52e..63bb3da 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -952,18 +952,14 @@ _mesa_initialize_context(struct gl_context *ctx,
-#if FEATURE_ES1
case API_OPENGLES:
ctx->Exec = _mesa_create_exec_table_es1();
break;
-#endif
This breaks r600 gallium build:
r600_dri.so.tmp: undefined reference to `_mesa_create_exec_table_es1'
The definition of that function in api_exec_es1.c is protected by #ifdef
FEATURE_ES1,
did you mean to remove that one as well?
P.S.: I called configure like this on Debian:
./configure --prefix=/opt/xorg --with-driver=dri --with-state-trackers=egl dri
--with-dri-drivers= --with-gallium-drivers=r600
LLVM_CONFIG=/usr/bin/llvm-config-3.1 --enable-r600-llvm-compiler
--enable-openvg --enable-vdpau --enable-glx-tls --enable-shared-glapi
--enable-texture-float --enable-egl --enable-gallium-egl
The problem is that _mesa_create_exec_table_es1 only exists with
--enable-es1. I had several other review comments on this series, but
I'm traveling and jetlagged, which has delayed things.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev