Rename local unpack var to _unpack

The method name was unpack, so rename the local var to _unpack to help avoid confusion.

diff --git a/catalyst/base/stagebase.py b/catalyst/base/stagebase.py
index c2042342b1137038e69c80bb060f6847f278300a..160c3a1bf552c263c018e352cc7d925975be9648 100644 (file)
--- 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 7e24bcab53439ed6fdeddba0b0414cb34c65845b..82c69b03603501e987c85f89aec1ac5f361f1e0c 100644 (file)
--- 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  ..."