On Fri, 2017-08-04 at 11:53 +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle <nicolai.haeh...@amd.com>
> 
> This aligns with the fact that we also check for EXPAT_LIBS
> unconditionally in configure.ac now. It should make all the
> various build permutations of Clover work (whether DRI is
> enabled or disabled in the build).
> 
> Cc: Aaron Watry <awa...@gmail.com>
> Cc: Emil Velikov <emil.veli...@collabora.com>
> --
> This change keeps everything green on Travis, and it should fix
> the duplicate-symbol linker error seen by Aaron and others when
> building Clover.

It does. This patch fixes last of the problems I had since the driconf
changes.
Tested-by: Jan Vesely <jan.ves...@rutgers.edu>

thanks,
Jan

> ---
>  src/gallium/targets/opencl/Makefile.am |  1 -
>  src/loader/Makefile.am                 | 13 +++++--------
>  2 files changed, 5 insertions(+), 9 deletions(-)
> 
> diff --git a/src/gallium/targets/opencl/Makefile.am 
> b/src/gallium/targets/opencl/Makefile.am
> index e88fa0fd382..c9d2be7afd0 100644
> --- a/src/gallium/targets/opencl/Makefile.am
> +++ b/src/gallium/targets/opencl/Makefile.am
> @@ -19,7 +19,6 @@ lib@OPENCL_LIBNAME@_la_LIBADD = \
>       $(top_builddir)/src/gallium/state_trackers/clover/libclover.la \
>       $(top_builddir)/src/gallium/auxiliary/libgallium.la \
>       $(top_builddir)/src/util/libmesautil.la \
> -     $(top_builddir)/src/util/libxmlconfig.la \
>       $(EXPAT_LIBS) \
>       $(LIBELF_LIBS) \
>       $(DLOPEN_LIBS) \
> diff --git a/src/loader/Makefile.am b/src/loader/Makefile.am
> index 8b197f2995c..5ed87820664 100644
> --- a/src/loader/Makefile.am
> +++ b/src/loader/Makefile.am
> @@ -33,21 +33,18 @@ AM_CPPFLAGS = \
>       $(XCB_DRI3_CFLAGS) \
>       $(LIBDRM_CFLAGS)
>  
> -libloader_la_CPPFLAGS = $(AM_CPPFLAGS)
> +libloader_la_CPPFLAGS = $(AM_CPPFLAGS) \
> +     -DUSE_DRICONF
>  libloader_la_SOURCES = $(LOADER_C_FILES)
> -libloader_la_LIBADD =
> +libloader_la_LIBADD = \
> +     $(top_builddir)/src/util/libxmlconfig.la
>  
>  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
> -
> +     -I$(top_srcdir)/src/mapi/
>  endif
>  
>  if HAVE_LIBDRM

Attachment: signature.asc
Description: This is a digitally signed message part

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

Reply via email to