diff options
author | Brian Dolbec <dolsen@gentoo.org> | 2013-06-01 00:31:59 -0700 |
---|---|---|
committer | Brian Dolbec <dolsen@gentoo.org> | 2013-11-21 22:01:49 -0800 |
commit | 1cba3b8ec73d458e53a65f0be7275f7585dc2186 (patch) | |
tree | c4372ab76544c9174f0aeb4bc6df3b32faea13b0 | |
parent | Fix a long line. (diff) | |
download | catalyst-1cba3b8ec73d458e53a65f0be7275f7585dc2186.tar.gz catalyst-1cba3b8ec73d458e53a65f0be7275f7585dc2186.tar.bz2 catalyst-1cba3b8ec73d458e53a65f0be7275f7585dc2186.zip |
Rename local unpack var to _unpack
The method name was unpack, so rename the local var to _unpack to help avoid confusion.
-rw-r--r-- | catalyst/base/stagebase.py | 18 | ||||
-rw-r--r-- | catalyst/targets/livecd_stage2.py | 6 |
2 files changed, 12 insertions, 12 deletions
diff --git a/catalyst/base/stagebase.py b/catalyst/base/stagebase.py index 759afee5..e2a98cf4 100644 --- a/catalyst/base/stagebase.py +++ b/catalyst/base/stagebase.py @@ -657,7 +657,7 @@ class StageBase(TargetBase, ClearBase, GenBase): target, print_traceback=True) def unpack(self): - unpack=True + _unpack=True clst_unpack_hash = self.resume.get("unpack") @@ -707,49 +707,49 @@ class StageBase(TargetBase, ClearBase, GenBase): if os.path.isdir(self.settings["source_path"]) \ and self.resume.is_enabled("unpack"): """ Autoresume is valid, SEEDCACHE is valid """ - unpack=False + _unpack=False invalid_snapshot=False elif os.path.isfile(self.settings["source_path"]) \ and self.settings["source_path_hash"]==clst_unpack_hash: """ Autoresume is valid, tarball is valid """ - unpack=False + _unpack=False invalid_snapshot=True elif os.path.isdir(self.settings["source_path"]) \ and self.resume.is_disabled("unpack"): """ Autoresume is invalid, SEEDCACHE """ - unpack=True + _unpack=True invalid_snapshot=False elif os.path.isfile(self.settings["source_path"]) \ and self.settings["source_path_hash"]!=clst_unpack_hash: """ Autoresume is invalid, tarball """ - unpack=True + _unpack=True invalid_snapshot=True else: """ No autoresume, SEEDCACHE """ if "seedcache" in self.settings["options"]: """ SEEDCACHE so let's run rsync and let it clean up """ if os.path.isdir(self.settings["source_path"]): - unpack=True + _unpack=True invalid_snapshot=False elif os.path.isfile(self.settings["source_path"]): """ Tarball so unpack and remove anything already there """ - unpack=True + _unpack=True invalid_snapshot=True """ No autoresume, no SEEDCACHE """ else: """ Tarball so unpack and remove anything already there """ if os.path.isfile(self.settings["source_path"]): - unpack=True + _unpack=True invalid_snapshot=True elif os.path.isdir(self.settings["source_path"]): """ We should never reach this, so something is very wrong """ raise CatalystError( "source path is a dir but seedcache is not enabled") - if unpack: + if _unpack: self.mount_safety_check() if invalid_snapshot: diff --git a/catalyst/targets/livecd_stage2.py b/catalyst/targets/livecd_stage2.py index 7e24bcab..82c69b03 100644 --- a/catalyst/targets/livecd_stage2.py +++ b/catalyst/targets/livecd_stage2.py @@ -87,7 +87,7 @@ class livecd_stage2(StageBase): myf.close() def unpack(self): - unpack=True + _unpack=True display_msg=None clst_unpack_hash = self.resume.get("unpack") @@ -103,12 +103,12 @@ class livecd_stage2(StageBase): if os.path.isdir(self.settings["source_path"]) and \ self.resume.is_enabled("unpack"): print "Resume point detected, skipping unpack operation..." - unpack=False + _unpack=False elif "source_path_hash" in self.settings: if self.settings["source_path_hash"] != clst_unpack_hash: invalid_snapshot=True - if unpack: + if _unpack: self.mount_safety_check() if invalid_snapshot: print "No Valid Resume point detected, cleaning up ..." |