diff options
author | Rob Crittenden <rcritten@redhat.com> | 2009-08-11 17:08:09 -0400 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2009-08-12 13:18:15 -0400 |
commit | 8780751330871033b13fc8fbb7eb0588baf4417d (patch) | |
tree | 0b21cac0974b193c67c43e22fb7b21983766fbf8 /ipapython/sysrestore.py | |
parent | c781e8a57d3d05fa07729dbccff07bc1fab9d8e8 (diff) | |
download | freeipa-8780751330871033b13fc8fbb7eb0588baf4417d.tar.gz freeipa-8780751330871033b13fc8fbb7eb0588baf4417d.tar.xz freeipa-8780751330871033b13fc8fbb7eb0588baf4417d.zip |
Clean up some problems discovered with pylint and pychecker
Much of this is formatting to make pylint happy but it also fixes some
real bugs.
Diffstat (limited to 'ipapython/sysrestore.py')
-rw-r--r-- | ipapython/sysrestore.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/ipapython/sysrestore.py b/ipapython/sysrestore.py index 503f38b24..78c9b6283 100644 --- a/ipapython/sysrestore.py +++ b/ipapython/sysrestore.py @@ -25,7 +25,6 @@ import os import os.path -import errno import shutil import logging import ConfigParser @@ -110,13 +109,13 @@ class FileStore: logging.debug(" -> Not backing up - '%s' doesn't exist", path) return - (reldir, file) = os.path.split(path) + (reldir, backupfile) = os.path.split(path) filename = "" for i in range(8): h = "%02x" % self.random.randint(0,255) filename += h - filename += "-"+file + filename += "-"+backupfile backup_path = os.path.join(self._path, filename) if os.path.exists(backup_path): |