On Wed, Jan 20, 2016 at 10:15:42AM -0500, Frediano Ziglio wrote: > > diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am > > index 6f02c99..000b097 100644 > > --- a/server/tests/Makefile.am > > +++ b/server/tests/Makefile.am > > @@ -27,10 +27,15 @@ libtest_a_SOURCES = \ > > test_display_base.h \ > > $(NULL) > > > > +libtest_a_LIBADD = \ > > + $(top_builddir)/server/libserver.la \ > > + $(top_builddir)/spice-common/common/libspice-common.la \ > > + $(NULL) > > + > > I think this is void for a static library > > > LDADD = \ > > libtest.a \ > > + $(top_builddir)/server/libserver.la \ > > $(top_builddir)/spice-common/common/libspice-common.la \ > > - $(top_builddir)/server/libspice-server.la \ > > $(GLIB2_LIBS) \ > > $(SPICE_NONPKGCONFIG_LIBS) \ > > $(NULL) > > So by default you add any possible library? > > Isn't a bit overkill? What if you want to test a specific source? > I suppose we can just override the LDADD for specific tests. I'll refine the Makefile.am bits, my main goal was to see if it was possible to get things to compile, and if the tests were still working ;) > > Otherwise the patch is really good! Thanks, I'll send a proper series then after addressing the things you pointed out. Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel