> > Hi > > On Fri, Jan 22, 2016 at 12:36 PM, Marc-André Lureau > <marcandre.lureau@xxxxxxxxx> wrote: > > Hi > > > > On Tue, Jan 19, 2016 at 11:41 AM, Frediano Ziglio <fziglio@xxxxxxxxxx> > > wrote: > >> Instead of using libserver.a to include necessary functions we include > >> single files to check for dependencies. > >> > >> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > >> --- > >> server/tests/Makefile.am | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >> diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am > >> index 8caff04..892a60b 100644 > >> --- a/server/tests/Makefile.am > >> +++ b/server/tests/Makefile.am > >> @@ -91,6 +91,8 @@ libstat_test3_a_CPPFLAGS = $(AM_CPPFLAGS) > >> -DTEST_COMPRESS_STAT=1 -DTEST_RED_WORK > >> libstat_test4_a_SOURCES = stat-test.c > >> libstat_test4_a_CPPFLAGS = $(AM_CPPFLAGS) -DTEST_COMPRESS_STAT=1 > >> -DTEST_RED_WORKER_STAT=1 -DTEST_NAME=stat_test4 > >> > >> +# Here we include only needed files to make sure code does not depend > >> +# on extra files. > >> test_qxl_parsing_SOURCES = \ > > > > What's the point of doing that? > > > > anyway, if it matters for some reason, I'll drop my cleanup patch. > > You didn't pick my patch, any reason for having those files seperate > from the rest of the server? Imho it's not useful and not very > future-proof either. > Looks like not many people are worried about design, software layers and dependencies. When I wrote this part of the Makefile with the intention of making sure parsing messages was a separate layer in the code. Not writing nothing about was a mistake. If nobody care I can remove the command and apply your patch. Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel