[Spice-devel] [PATCH] make spice-server-replay utility smaller

[Christophe Fergeau]

Acked-by: Christophe Fergeau <cfergeau at redhat.com>

though it took me a bit to understand what this was about, the commit
log could probably be a bit clearer.

Christophe

On Mon, May 16, 2016 at 05:13:49PM +0100, Frediano Ziglio wrote:
> spice-server-replay is built inside tests.
> However tests are built linking to a convenience library causing all
> spice-server code to be linked inside of the executable.
> This is not a big problem however do not allow to easily use this
> utility to test different binary version libraries.
> Linking to just files and to the dynamic library allows to use
> with different binary versions.
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/tests/Makefile.am | 13 ++++++++++++-
>  1 file changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am
> index b3e9740..91edd7c 100644
> --- a/server/tests/Makefile.am
> +++ b/server/tests/Makefile.am
> @@ -67,7 +67,18 @@ noinst_LIBRARIES += \
>  	libstat_test4.a \
>  	$(NULL)
>  
> -spice_server_replay_SOURCES = replay.c
> +spice_server_replay_SOURCES = replay.c		\
> +	../event-loop.c				\
> +	basic_event_loop.c			\
> +	basic_event_loop.h
> +
> +spice_server_replay_LDADD =					\
> +	$(top_builddir)/spice-common/common/libspice-common.la	\
> +	$(top_builddir)/server/libspice-server.la		\
> +	$(GLIB2_LIBS)						\
> +	$(GOBJECT2_LIBS)					\
> +	$(SPICE_NONPKGCONFIG_LIBS)		                \
> +	$(NULL)
>  
>  stat_test_SOURCES = stat-main.c
>  stat_test_LDADD = \
> -- 
> 2.7.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20160517/0c3b09b4/attachment.sig>