X-Git-Url: http://git.grml.org/?p=grml2usb.git;a=blobdiff_plain;f=grml2usb;h=93b1d115d52dc8826d1e2bed753c683924aef0ef;hp=92c2cbfd10972acc70c4a86eba0d44005102444b;hb=1d3c9dab80aa6b80764e852865d5e224f6548fd3;hpb=411daf1e3e9fcbb9c4e4588a9fbe92513efb1ed1 diff --git a/grml2usb b/grml2usb index 92c2cbf..93b1d11 100755 --- a/grml2usb +++ b/grml2usb @@ -28,6 +28,7 @@ import sys import tempfile import time import uuid +import shutil # The line following this line is patched by debian/rules and tarball.sh. PROG_VERSION = '***UNRELEASED***' @@ -161,6 +162,9 @@ def string2array(s): def cleanup(): """Cleanup function to make sure there aren't any mounted devices left behind. """ + def del_failed(fn, filepath, exc): + msg = "Deletion of %s failed in temporary folder %s" + logging.warn(msg % (filepath, path)) logging.info("Cleaning up before exiting...") proc = subprocess.Popen(["sync"]) @@ -169,14 +173,25 @@ def cleanup(): for device in MOUNTED: try: unmount(device, "") + logging.debug('Unmounted %s' % device) except RuntimeError: logging.debug('RuntimeError while umount %s, ignoring' % device) - for tmpfile in TMPFILES: + + for tmppath in TMPFILES: try: - os.unlink(tmpfile) + if os.path.isdir(tmppath) and not os.path.islink(tmppath): + # symbolic links to directories are ignored + # without the check it will throw an OSError + shutil.rmtree(tmppath, onerror=del_failed) + logging.debug('temporary directory %s deleted' % tmppath) + unregister_tmpfile(tmppath) + elif os.path.isfile: + os.unlink(tmppath) + logging.debug('temporary file %s deleted' % tmppath) + unregister_tmpfile(tmppath) except RuntimeError: msg = 'RuntimeError while removing temporary %s, ignoring' - logging.debug(msg % tmpfile) + logging.debug(msg % tmppath) def register_tmpfile(path):