[PATCH v2 3/4] test: always add DIX_LIB and OS_LIB on XORG builds
Peter Hutterer
peter.hutterer at who-t.net
Wed Jun 20 22:42:17 PDT 2012
With --disable-xorg, We also disabled a bunch of tests because of their
perceived reliance on a DDX. The cause was libtool missing some object files
that never ended up in libxservertest.la. Only the xfree86 test has a true
dependency on XORG.
DIX_LIB was pointing to dix.O (instead of libdix.la) when
DTRACE_SPECIAL_OBJECTS was defined. libdix.la should be part of XSERVER_LIBS
but dix.O is not a recognised libtool object, so it got skipped for
libxservertest.a. Only in the XORG case would we add DIX_LIB and OS_LIB
manually, thus forcing linkage with the dtrace-generated objects.
Fixing this by packaging up the dtrace-generated files as part of
libdix.la/libos.la doesn't work for Solaris (and possible others), so simply
always force linkage against the DIX_LIB/OS_LIB in the case of dtrace objects.
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
Changes to v1:
we can't fix the dix/os libtool libraries previously proposed, Solaris
ends up without probes then. So instead of that, use the existing object
files but link all test against them.
test/Makefile.am | 2 --
1 file changed, 2 deletions(-)
diff --git a/test/Makefile.am b/test/Makefile.am
index 1f0e7fa..69e7f8a 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -21,11 +21,9 @@ INCLUDES += -I$(top_srcdir)/hw/xfree86/parser \
endif
TEST_LDADD=libxservertest.la $(XORG_SYS_LIBS) $(XSERVER_SYS_LIBS) $(GLX_SYS_LIBS)
-if XORG
if SPECIAL_DTRACE_OBJECTS
TEST_LDADD += $(OS_LIB) $(DIX_LIB)
endif
-endif
xkb_LDADD=$(TEST_LDADD)
input_LDADD=$(TEST_LDADD)
--
1.7.10.2
More information about the xorg-devel
mailing list