On 11 October 2017 at 09:11, Hongxu Jia <hongxu....@windriver.com> wrote: > While built with "-fvisibility=default" > ... > |i586-oe-linux-gcc ... -fvisibility=default ... -o > common/.libs/common_libintel_common_la-gen_decoder.o > ... > > It triggered the failure > ... > |i586-oe-linux-g++ ... common/.libs/libintel_common.a ... -o > vulkan/.libs/libvulkan_intel.so > |common/.libs/libintel_common.a(common_libintel_common_la-gen_decoder.o): > |In function `start_element': > |/usr/src/debug/mesa/2_17.1.7-r0/mesa-17.1.7/src/intel/common/gen_decoder.c:371: > undefined reference to `XML_GetCurrentLineNumber' > ... > > explictly add EXPAT_LIBS to intel's VULKAN_LIB_DEPS > > Signed-off-by: Hongxu Jia <hongxu....@windriver.com> > --- > src/intel/Makefile.vulkan.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/intel/Makefile.vulkan.am b/src/intel/Makefile.vulkan.am > index 271b0a5..8fbe2c8 100644 > --- a/src/intel/Makefile.vulkan.am > +++ b/src/intel/Makefile.vulkan.am > @@ -144,6 +144,7 @@ VULKAN_LIB_DEPS = \ > $(LIBDRM_LIBS) \ > $(PTHREAD_LIBS) \ > $(DLOPEN_LIBS) \ > + $(EXPAT_LIBS) \ Expat is used in src/intel/common/, thus we should move the handling (cflags and libs) to Makefile.common.am. Seems like we've missed that as the decoder migrated tools ->common.
Care to send a patch, or shall I? Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
