Hi Quentin,

On 18 August 2016 at 10:15, Quentin Glidic
<sardemff7+wayl...@sardemff7.net> wrote:
> @@ -116,7 +116,9 @@ libweston_@LIBWESTON_MAJOR@_la_SOURCES =                  
>   \
>  lib_LTLIBRARIES += libweston-desktop-@LIBWESTON_MAJOR@.la
>  libweston_desktop_@LIBWESTON_MAJOR@_la_CPPFLAGS = $(AM_CPPFLAGS) -DIN_WESTON
>  libweston_desktop_@LIBWESTON_MAJOR@_la_CFLAGS = $(AM_CFLAGS) 
> $(COMPOSITOR_CFLAGS)
> -libweston_desktop_@LIBWESTON_MAJOR@_la_LIBADD = 
> libweston-@LIBWESTON_MAJOR@.la $(COMPOSITOR_LIBS)
> +libweston_desktop_@LIBWESTON_MAJOR@_la_LIBADD =                \
> +       libweston-@LIBWESTON_MAJOR@.la                  \
> +       $(COMPOSITOR_LIBS)
>  libweston_desktop_@LIBWESTON_MAJOR@_la_LDFLAGS = -version-info 
> $(LT_VERSION_INFO)

COMPOSITOR_LIBS is full of -Lfoo -lsyslib. To the best of my
knowledge, these belong in LDADD, which is things to throw in the ld
command line; LIBADD is local libraries to add, which will be
dependency-tracked and also thrown through the libtool mangling
machinery.

Could you please test with a split that has $(COMPOSITOR_LIBS) et al
being set as LDADD, and only the local paths for LIBADD? Such a patch
would have my R-b, as the rest looks sound enough.

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to