[virt-tools-list] [PATCH virt-viewer] Fail if neither vnc nor spice is detected

Christophe Fergeau cfergeau at redhat.com
Tue Oct 23 08:03:16 UTC 2012


On Mon, Oct 22, 2012 at 08:41:06PM +0200, Guido Günther wrote:
> Since the viewer makes little sense otherwise.
> ---
>  configure.ac |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/configure.ac b/configure.ac
> index e7ddd3e..d5feb0f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -192,6 +192,10 @@ if test "x$enable_plugin" = "xyes"; then
>  fi
>  AM_CONDITIONAL(ENABLE_PLUGIN, [test "x$enable_plugin" = "xyes"])
>  
> +if test "x$have_gtk_vnc" != "xyes" -a "x$have_gtk_spice" != "xyes"; then

IIRC
if test "x$have_gtk_vnc" != "xyes" && test "x$have_gtk_spice" != "xyes"; then
is better as it is more portable.
Apart from this, ACK

Christophe


> +    AC_MSG_ERROR([At least one of spice or vnc must be used])
> +fi
> +
>  AC_OUTPUT([
>      Makefile
>      data/Makefile
> -- 
> 1.7.10.4
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20121023/d19cb1cd/attachment.sig>


More information about the virt-tools-list mailing list