Browse Source

misc python3 fixes

Wojtek Porczyk 7 years ago
parent
commit
1be75d9c83

+ 1 - 1
qubes/storage/__init__.py

@@ -91,7 +91,7 @@ class Volume(object):
         self.pool = str(pool)
         self.internal = internal
         self.removable = removable
-        self.revisions_to_keep = revisions_to_keep
+        self.revisions_to_keep = int(revisions_to_keep)
         self.rw = rw
         self.save_on_stop = save_on_stop
         self._size = int(size)

+ 1 - 1
qubes/storage/file.py

@@ -129,7 +129,7 @@ class FilePool(qubes.storage.Pool):
                              stdout=subprocess.PIPE)
         result = p.communicate()
 
-        m = re.match(r'^(/dev/loop\d+):\s', result[0])
+        m = re.match(r'^(/dev/loop\d+):\s', result[0].decode())
         if m is not None:
             loop_dev = m.group(1)
 

+ 1 - 1
qubes/storage/lvm.py

@@ -317,7 +317,7 @@ def init_cache(log=logging.getLogger('qube.storage.lvm')):
     result = {}
 
     for line in out.splitlines():
-        line = line.strip()
+        line = line.decode().strip()
         pool_name, pool_lv, name, size, usage_percent, attr = line.split(',', 5)
         if '' in [pool_name, pool_lv, name, size, usage_percent]:
             continue

+ 1 - 1
qubes/tests/__init__.py

@@ -660,7 +660,7 @@ class SystemTestsMixin(object):
         try:
             volumes = subprocess.check_output(
                 ['sudo', 'lvs', '--noheadings', '-o', 'vg_name,name',
-                    '--separator', '/'])
+                    '--separator', '/']).decode()
             if ('/' + prefix) not in volumes:
                 return
             subprocess.check_call(['sudo', 'lvremove', '-f'] +

+ 2 - 2
qubes/tests/integ/tools/qvm_run.py

@@ -19,8 +19,8 @@
 # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 #
 
+import io
 import os
-import StringIO
 import sys
 import tempfile
 import unittest
@@ -111,7 +111,7 @@ class TC_00_qvm_run(qubes.tests.SystemTestsMixin, qubes.tests.QubesTestCase):
 
     @unittest.skip('expected error')
     def test_005_colour_output(self):
-        sys.stdout = StringIO.StringIO()
+        sys.stdout = io.StringIO()
         qubes.tools.qvm_run.main(
             self.sharedopts + ['--colour-output', '32', self.vm1.name, 'true'])
         self.assertEqual('\033[0;32m\033[0m', sys.stdout.getvalue())

+ 2 - 1
qubes/vm/qubesvm.py

@@ -400,7 +400,8 @@ class QubesVM(qubes.vm.mix.net.NetVMMixin, qubes.vm.BaseVM):
 
     maxmem = qubes.property('maxmem', type=int,
         setter=_setter_positive_int,
-        default=(lambda self: min(self.app.host.memory_total / 1024 / 2, 4000)),
+        default=(lambda self:
+            int(min(self.app.host.memory_total / 1024 / 2, 4000))),
         doc='''Maximum amount of memory available for this VM (for the purpose
             of the memory balancer).''')
 

+ 1 - 0
rpm_spec/core-dom0.spec

@@ -85,6 +85,7 @@ Requires:       createrepo
 Requires:       gnome-packagekit
 Requires:       cronie
 Requires:       bsdtar
+Requires:       python3-jinja2
 # for qubes-hcl-report
 Requires:       dmidecode
 Requires:       PyQt4