From 10e7c2c64d46da38d30d04a2c10c8c3cd5a30f7a Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Fri, 4 Aug 2017 17:49:08 +0100 Subject: loader: rework xmlconfig dependency MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently xmlconfig is conditionally used, only when --enable-dri is available. As the library has moved to src/util and has wider wisebase, this guard is no longer correct. Strictly speaking - it wasn't since the introduction of xmlconfig into st/nine a while ago. Unconditionally enable xmlconfig and drop the linking. As said before there's other users of the library, so depending on the configure options we will get multiple definitions of said symbols. NOTE: To avoid breaking other combinations, this commit adds the xmlconfig link to the required places - throughout gallium and the DRI loaders. Cc: Aaron Watry Signed-off-by: Emil Velikov Reviewed-by: Nicolai Hähnle Tested-by: Dieter Nützel --- src/loader/Makefile.am | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-) (limited to 'src/loader') diff --git a/src/loader/Makefile.am b/src/loader/Makefile.am index 8b197f2995c..74ac6c51e77 100644 --- a/src/loader/Makefile.am +++ b/src/loader/Makefile.am @@ -26,6 +26,8 @@ EXTRA_DIST = SConscript noinst_LTLIBRARIES = libloader.la AM_CPPFLAGS = \ + -I$(top_builddir)/src/util/ \ + -DUSE_DRICONF \ $(DEFINES) \ -I$(top_srcdir)/include \ -I$(top_srcdir)/src \ @@ -37,19 +39,6 @@ libloader_la_CPPFLAGS = $(AM_CPPFLAGS) libloader_la_SOURCES = $(LOADER_C_FILES) libloader_la_LIBADD = -if HAVE_DRICOMMON -libloader_la_CPPFLAGS += \ - -I$(top_builddir)/src/util/ \ - -I$(top_srcdir)/src/mesa/drivers/dri/common/ \ - -I$(top_srcdir)/src/mesa/ \ - -I$(top_srcdir)/src/mapi/ \ - -DUSE_DRICONF - -libloader_la_LIBADD += \ - $(top_builddir)/src/util/libxmlconfig.la - -endif - if HAVE_LIBDRM libloader_la_LIBADD += \ $(LIBDRM_LIBS) -- cgit v1.2.3