Define all class attributes inside __init__

src/pylorax/__init__.py

@@ -80,6 +80,13 @@ class Lorax(BaseLoraxClass):
     def __init__(self):
         BaseLoraxClass.__init__(self)
         self._configured = False
+        self.conf = None
+        self.outputdir = None
+        self.workdir = None
+        self.inroot = None
+        self.arch = None
+        self.product = None
+        self.debug = False
 
         # set locale to C
         locale.setlocale(locale.LC_ALL, 'C')

src/pylorax/imgutils.py

@@ -328,6 +328,7 @@ def estimate_size(rootdir, graft={}, fstype=None, blocksize=4096, overhead=128):
 
 class LoopDev(object):
     def __init__(self, filename, size=None):
+        self.loopdev = None
         self.filename = filename
         if size:
             mksparse(self.filename, size)
@@ -340,6 +341,7 @@ class LoopDev(object):
 class DMDev(object):
     def __init__(self, dev, size, name=None):
         (self.dev, self.size, self.name) = (dev, size, name)
+        self.mapperdev = None
     def __enter__(self):
         self.mapperdev = dm_attach(self.dev, self.size, self.name)
         return self.mapperdev
@@ -364,6 +366,8 @@ class PartitionMount(object):
         returns True if it should be mounted.
         """
         self.mount_dir = None
+        self.mount_dev = None
+        self.mount_size = None
         self.disk_img = disk_img
         self.mount_ok = mount_ok
 

src/pylorax/ltmpl.py

@@ -44,6 +44,7 @@ class LoraxTemplate(object):
         # we have to add ["/"] to the template lookup directories or the
         # file includes won't work properly for absolute paths
         self.directories = ["/"] + directories
+        self.lines = []
 
     def parse(self, template_file, variables):
         lookup = TemplateLookup(directories=self.directories)
@@ -153,6 +154,7 @@ class LoraxTemplateRunner(object):
         self.yum = yum
         self.fatalerrors = fatalerrors
         self.templatedir = templatedir or "/usr/share/lorax"
+        self.templatefile = None
         # some builtin methods
         self.builtins = DataHolder(exists=lambda p: rexists(p, root=inroot),
                                    glob=lambda g: list(rglob(g, root=inroot)))

src/pylorax/treebuilder.py

@@ -186,6 +186,7 @@ class TreeBuilder(object):
         self.add_templates = add_templates or []
         self.add_template_vars = add_template_vars or {}
         self.templatedir = templatedir
+        self.treeinfo_data = None
 
     @property
     def kernels(self):