aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrian Dolbec <dolsen@gentoo.org>2013-06-01 00:31:59 -0700
committerBrian Dolbec <dolsen@gentoo.org>2013-11-21 22:01:49 -0800
commit1cba3b8ec73d458e53a65f0be7275f7585dc2186 (patch)
treec4372ab76544c9174f0aeb4bc6df3b32faea13b0
parentFix a long line. (diff)
downloadcatalyst-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.py18
-rw-r--r--catalyst/targets/livecd_stage2.py6
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 ..."