aboutsummaryrefslogtreecommitdiffstats
path: root/main/xf86-video-qxl/convert-xspice-python3.patch
blob: 145d3e3029a7ce91b32ecfedfed92985a475b84c (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau <cfergeau@redhat.com>
Date: Wed, 8 Feb 2017 15:23:56 +0100
Subject: [PATCH] Xspice: Use print("") instead of print ""

This allows Xspice to run when using python3 instead of python2
---
 scripts/Xspice | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/scripts/Xspice b/scripts/Xspice
index ada99d1..30d10fa 100755
--- a/scripts/Xspice
+++ b/scripts/Xspice
@@ -34,7 +34,7 @@ def which(x):
         candidate = os.path.join(p, x)
         if os.path.exists(candidate):
             return candidate
-    print 'Warning: failed to find executable %s' % x
+    print('Warning: failed to find executable %s' % x)
     return None
 
 if 'XSPICE_ENABLE_GDB' in os.environ:
@@ -128,10 +128,10 @@ args, xorg_args = parser.parse_known_args(sys.argv[1:])
 def agents_new_enough(args):
     for f in [args.vdagent_exec, args.vdagentd_exec]:
         if not f:
-            print 'please specify path to vdagent/vdagentd executables'
+            print('please specify path to vdagent/vdagentd executables')
             return False
         if not os.path.exists(f):
-            print 'error: file not found ', f
+            print('error: file not found ', f)
             return False
 
     for f in [args.vdagent_exec, args.vdagentd_exec]:
@@ -169,11 +169,11 @@ def tls_files(args):
 # and it isn't supplied spice will still abort, and Xorg with it.
 for key, filename in tls_files(args).items():
     if not os.path.exists(filename):
-        print "missing %s - %s does not exist" % (key, filename)
+        print("missing %s - %s does not exist" % (key, filename))
         sys.exit(1)
 
 def error(msg, exit_code=1):
-    print "Xspice: %s" % msg
+    print("Xspice: %s" % msg)
     sys.exit(exit_code)
 
 if not args.xorg:
@@ -319,7 +319,7 @@ for arg in xorg_args:
     if arg.startswith(":"):
         display = arg
 if not display:
-    print "Error: missing display on line (i.e. :3)"
+    print("Error: missing display on line (i.e. :3)")
     raise SystemExit
 os.environ ['DISPLAY'] = display
 
@@ -343,7 +343,7 @@ time.sleep(2)
 
 retpid,rc = os.waitpid(xorg.pid, os.WNOHANG)
 if retpid != 0:
-    print "Error: X server is not running"
+    print("Error: X server is not running")
 else:
     if args.vdagent_enabled and args.vdagent_launch:
         # XXX use systemd --user for this?
@@ -361,4 +361,4 @@ else:
         xorg.wait()
     except KeyboardInterrupt:
         # Catch Ctrl-C as that is the common way of ending this script
-        print "Keyboard Interrupt"
+        print("Keyboard Interrupt")

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau <cfergeau@redhat.com>
Date: Wed, 8 Feb 2017 17:07:39 +0100
Subject: [PATCH] Xspice: Remove extra space before assignment

---
 scripts/Xspice | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/Xspice b/scripts/Xspice
index 30d10fa..41fd80c 100755
--- a/scripts/Xspice
+++ b/scripts/Xspice
@@ -211,7 +211,7 @@ signal.signal(signal.SIGTERM, cleanup)
 atexit.register(cleanup)
 
 if args.auto:
-    temp_dir  = tempfile.mkdtemp(prefix="Xspice-")
+    temp_dir = tempfile.mkdtemp(prefix="Xspice-")
     cleanup_dirs.append(temp_dir)
 
     args.config = temp_dir + "/xorg.conf"
@@ -275,7 +275,7 @@ if args.vdagent_enabled:
             os.unlink(f)
 
     if not temp_dir:
-        temp_dir  = tempfile.mkdtemp(prefix="Xspice-")
+        temp_dir = tempfile.mkdtemp(prefix="Xspice-")
         cleanup_dirs.append(temp_dir)
 
     # Auto generate temporary files for vdagent

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau <cfergeau@redhat.com>
Date: Wed, 8 Feb 2017 17:07:56 +0100
Subject: [PATCH] Xspice: Fix Python3 str() vs bytes() confusion

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
 
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau <cfergeau@redhat.com>
Date: Thu, 17 May 2018 15:31:05 +0200
Subject: [PATCH] Xspice: Adjust shebang to explicitly mention python3

---
 scripts/Xspice | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/Xspice b/scripts/Xspice
index 927dcb1..78d0794 100755
--- a/scripts/Xspice
+++ b/scripts/Xspice
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 
 """
 Xspice