Re: [qxl 2/2] Xspice: Fix Python3 str() vs bytes() confusion

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

 



ack

On Wed, 2017-02-08 at 17:15 +0100, Christophe Fergeau wrote:
> With python3, without universal_newlines=True, Popen().stdout.read()
> will return a byte array, while find(str) expects to operate on a
> string.
> I've checked that this still works with python2 as well.
> ---
>  scripts/Xspice | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/Xspice b/scripts/Xspice
> index 41fd80c..927dcb1 100755
> --- a/scripts/Xspice
> +++ b/scripts/Xspice
> @@ -135,7 +135,7 @@ def agents_new_enough(args):
>              return False
>  
>      for f in [args.vdagent_exec, args.vdagentd_exec]:
> -        if Popen(args=[f, '-h'], stdout=PIPE).stdout.read().find('-
> S') == -1:
> +        if Popen(args=[f, '-h'], stdout=PIPE,
> universal_newlines=True).stdout.read().find('-S') == -1:
>              return False
>      return True
>  
_______________________________________________
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]