Comment 2 for bug 1982534

Revision history for this message
Matthew Ruffell (mruffell) wrote :

Hi Robie,

Thank you for your feedback, it is all very reasonable, and I have actioned upon it, by changing the subprocess.Popen() call to check_call(), and changed the rm to os.unlink().

I opened a new merge request at https://code.launchpad.net/~mruffell/ubuntu-release-upgrader/+git/ubuntu-release-upgrader/+merge/435410 with the changes.