Merge lp:~vorlon/usb-creator/genisoimage-to-xorriso into lp:usb-creator

Proposed by Steve Langasek
Status: Merged
Merged at revision: 496
Proposed branch: lp:~vorlon/usb-creator/genisoimage-to-xorriso
Merge into: lp:usb-creator
Diff against target: 64 lines (+19/-9)
3 files modified
debian/changelog (+6/-0)
debian/control (+1/-1)
usbcreator/backends/udisks/backend.py (+12/-8)
To merge this branch: bzr merge lp:~vorlon/usb-creator/genisoimage-to-xorriso
Reviewer Review Type Date Requested Status
usb-creator hackers Pending
Review via email: mp+401342@code.launchpad.net

Description of the change

We want to migrate away from genisoimage to xorriso.

This should not be merged until 21.10 opens.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2021-04-02 15:52:52 +0000
+++ debian/changelog 2021-04-16 23:24:37 +0000
@@ -1,3 +1,9 @@
1usb-creator (0.3.10) UNRELEASED; urgency=medium
2
3 * Port from genisoimage (isoinfo) to xorriso (osirrox)
4
5 -- Steve Langasek <steve.langasek@ubuntu.com> Fri, 16 Apr 2021 16:23:06 -0700
6
1usb-creator (0.3.9) hirsute; urgency=medium7usb-creator (0.3.9) hirsute; urgency=medium
28
3 * debian/control: Add genisoimage to Depends as isoinfo is needed to look9 * debian/control: Add genisoimage to Depends as isoinfo is needed to look
410
=== modified file 'debian/control'
--- debian/control 2021-04-02 15:52:42 +0000
+++ debian/control 2021-04-16 23:24:37 +0000
@@ -17,7 +17,7 @@
17Package: usb-creator-common17Package: usb-creator-common
18Architecture: all18Architecture: all
19Depends: ${misc:Depends}, ${python3:Depends}, python3-dbus,19Depends: ${misc:Depends}, ${python3:Depends}, python3-dbus,
20 gir1.2-udisks-2.0, udisks2, genisoimage, python3-debian20 gir1.2-udisks-2.0, udisks2, xorriso, python3-debian
21Description: create a startup disk using a CD or disc image (common files)21Description: create a startup disk using a CD or disc image (common files)
22 Startup Disk Creator converts a USB key or SD card into a volume from which you22 Startup Disk Creator converts a USB key or SD card into a volume from which you
23 can start up and run Ubuntu. You can also store files and settings in any space23 can start up and run Ubuntu. You can also store files and settings in any space
2424
=== modified file 'usbcreator/backends/udisks/backend.py'
--- usbcreator/backends/udisks/backend.py 2019-06-30 14:23:02 +0000
+++ usbcreator/backends/udisks/backend.py 2021-04-16 23:24:37 +0000
@@ -1,5 +1,8 @@
1import dbus1import dbus
2import logging2import logging
3import os
4import subprocess
5import tempfile
3from dbus.mainloop.glib import DBusGMainLoop, threads_init6from dbus.mainloop.glib import DBusGMainLoop, threads_init
4import gi7import gi
5gi.require_version('UDisks', '2.0')8gi.require_version('UDisks', '2.0')
@@ -145,15 +148,16 @@
145148
146 # Device manipulation functions.149 # Device manipulation functions.
147 def _is_casper_cd(self, filename):150 def _is_casper_cd(self, filename):
151 tmpfile = tempfile.mkstemp()[1]
148 for search in ['/.disk/info', '/.disk/mini-info']:152 for search in ['/.disk/info', '/.disk/mini-info']:
149 cmd = ['isoinfo', '-J', '-i', filename, '-x', search]153 cmd = ['osirrox', '-dev', filename, '-extract', search, tmpfile]
150 try:154 subprocess.run(cmd, stderr=None, stdout=None)
151 output = misc.popen(cmd, stderr=None)155 if os.stat(tmpfile).st_size > 0:
152 if output:156 f = open(tmpfile)
153 return output157 output = f.read()
154 except misc.USBCreatorProcessException:158 f.close()
155 # TODO evand 2009-07-26: Error dialog.159 os.unlink(tmpfile)
156 logging.error('Could not extract .disk/info.')160 return output
157 return None161 return None
158162
159 def install(self, source, target, allow_system_internal=False):163 def install(self, source, target, allow_system_internal=False):

Subscribers

People subscribed via source and target branches