Re: [PATCH 2/3] autotools: compile gst-test only if gstreamer is available

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Jan 15, 2017 at 06:12:21PM +0200, Snir Sheriber wrote:
> Needed in order to avoid compilation error when gstreamer is
> unavailable
> ---
>  server/tests/Makefile.am | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am
> index f696f9f..55ace5b 100644
> --- a/server/tests/Makefile.am
> +++ b/server/tests/Makefile.am
> @@ -47,6 +47,10 @@ check_PROGRAMS =				\
>  	test-stat-file				\
>  	$(NULL)
>  
> +if HAVE_GSTREAMER
> +TEST_GST=test-gst
> +endif
> +
>  noinst_PROGRAMS =				\
>  	test-display-no-ssl			\
>  	test-display-streaming			\
> @@ -59,7 +63,7 @@ noinst_PROGRAMS =				\
>  	test-vdagent				\
>  	test-display-width-stride		\
>  	spice-server-replay			\
> -	test-gst				\
> +	$(TEST_GST)				\
>  	$(check_PROGRAMS)			\
>  	$(NULL)
>  
> @@ -109,6 +113,7 @@ libtest_stat4_a_CPPFLAGS = $(AM_CPPFLAGS) -DTEST_COMPRESS_STAT=1 -DTEST_RED_WORK
>  
>  test_qxl_parsing_LDADD = ../libserver.la $(LDADD)
>  
> +if HAVE_GSTREAMER

Alternatively, it's probably possible to do
noinst_PROGRAMS += test-gst here to avoid the if HAVE_GSTREAMER block at
the very beginning of the file.
But I'm fine with your version

Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]