Re: [PATCHv3 spice-gtk] build-sys: fix make check when polkit not available

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

 



Hmm, that initial empty noinst_PROGRAMS looks a bit like a mistake. Perhaps add
a $(NULL) to that to make it explicit?

ACK either way

Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>



On Fri, 2016-03-11 at 20:04 +0100, Marc-André Lureau wrote:
> acl-helper needs polkit (and unix atm) to build.
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
> ---
>  tests/Makefile.am | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 24d45c9..c1d95c1 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -1,18 +1,21 @@
>  NULL =
>  
> +noinst_PROGRAMS =
>  TESTS = coroutine				\
>  	util					\
>  	session					\
> -	usb-acl-helper				\
>  	$(NULL)
>  
>  if WITH_PHODAV
>  TESTS += pipe
>  endif
>  
> -noinst_PROGRAMS = $(TESTS) \
> -		  mock-acl-helper \
> -		  $(NULL)
> +if WITH_POLKIT
> +TESTS += usb-acl-helper
> +noinst_PROGRAMS += mock-acl-helper
> +endif
> +
> +noinst_PROGRAMS += $(TESTS)
>  
>  AM_CPPFLAGS =					\
>  	$(COMMON_CFLAGS)			\
_______________________________________________
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]