[Spice-devel] [PATCH xf86-video-qxl v2] Xspice: handle parameters with value 0, allows --port 0

Christophe Fergeau cfergeau at redhat.com
Tue Nov 3 08:46:52 PST 2015


On Tue, Nov 03, 2015 at 10:39:33AM -0600, Jeremy White wrote:
> As of commit f73ca7e4cc090772652611a4e2b8b95ae64319bc, you can now
> disable the regular port by specifying 0.  This change to the Xspice
> script permits us to take advantage of that change.

Ah thanks for the explanation, makes much more sense now, thanks, ACK.

Christophe

> 
> Signed-off-by: Jeremy White <jwhite at codeweavers.com>
> ---
> Since v1: Commit message expanded to explain why this is useful.
> ---
>  scripts/Xspice | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/Xspice b/scripts/Xspice
> index 52b1b16..46c5c1e 100755
> --- a/scripts/Xspice
> +++ b/scripts/Xspice
> @@ -256,7 +256,7 @@ var_args = ['port', 'tls_port', 'disable_ticketing',
>      'vdagent_uid', 'vdagent_gid']
>  
>  for arg in var_args:
> -    if getattr(args, arg):
> +    if getattr(args, arg) != None:
>          # The Qxl code doesn't respect booleans, so pass them as 0/1
>          a = getattr(args, arg)
>          if a == True:
> -- 
> 2.1.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://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: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151103/0882a021/attachment.sig>


More information about the Spice-devel mailing list