[Spice-devel] [PATCH spice-common v2 1/2] test-region: Add missing dependencies

Christophe Fergeau cfergeau at redhat.com
Tue Jan 8 12:59:25 UTC 2019


On Mon, Dec 17, 2018 at 06:56:19AM -0500, Frediano Ziglio wrote:
> > 
> > The test uses both GLib and pixman libraries.
> > This does not seem to affect Linux but make the test fails under Windows.
> > 
> > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > ---
> >  tests/Makefile.am | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > Changes since v1:
> > - use SPICE_COMMON_LIBS
> > 
> > diff --git a/tests/Makefile.am b/tests/Makefile.am
> > index 926ac99..f54e394 100644
> > --- a/tests/Makefile.am
> > +++ b/tests/Makefile.am
> > @@ -63,6 +63,7 @@ test_region_CFLAGS =			\
> >  	$(NULL)
> >  test_region_LDADD =					\
> >  	$(top_builddir)/common/libspice-common.la	\
> > +	$(SPICE_COMMON_LIBS)				\
> >  	$(NULL)
> >  
> >  # Avoid need for python(pyparsing) by end users
> 
> Looks like a bug in libtool. Dependencies are in libspice-common.la and
> the real executable (in tests/.libs/test_region.exe) correctly depends
> on glib and pixman dll files however the wrapper/launcher generated in
> tests/test_region.exe is generated not considering dependencies in
> libspice-common.la (one of wrapper job is to add some definitions to
> PATH variable).
> Still the patch solves the weird issue.

Actually, since test-region.c uses both pixman and glib, it should be
linking with glib/pixman explicitly, not through libspice-common.la,
it's a bug in our Makefile.am that it's not doing so.

Acked-by: Christophe Fergeau <cfergeau at redhat.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20190108/c336c887/attachment.sig>


More information about the Spice-devel mailing list