[gentoo-commits] proj/catalyst:master commit in: catalyst/targets/, catalyst/base/, catalyst/, /

[gentoo-commits] proj/catalyst:master commit in: catalyst/targets/, catalyst/base/, catalyst/, /

[Mike Frysinger]

commit:     228211c82aeb92e9aa346e4c5bfa53a2a0cf95b8
Author:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
AuthorDate: Tue Oct  6 04:06:05 2015 +0000
Commit:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
CommitDate: Tue Oct  6 04:06:05 2015 +0000
URL:        https://gitweb.gentoo.org/proj/catalyst.git/commit/?id=228211c8

lint: fix up unused variables

This variables aren't actually used which the linter likes to warn about,
so add a _ prefix to their name to silence the warnings.

 catalyst/base/resume.py     | 2 +-
 catalyst/main.py            | 2 +-
 catalyst/support.py         | 2 +-
 catalyst/targets/netboot.py | 5 +++--
 setup.py                    | 2 +-
 5 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/catalyst/base/resume.py b/catalyst/base/resume.py
index 608f574..e3f03e7 100644
--- a/catalyst/base/resume.py
+++ b/catalyst/base/resume.py
@@ -23,7 +23,7 @@ class AutoResumeError(Exception):
 	def __init__(self, message, print_traceback=False):
 		if message:
 			if print_traceback:
-				(type,value)=sys.exc_info()[:2]
+				(_type, value) = sys.exc_info()[:2]
 				if value!=None:
 					print
 					print "Traceback values found.  listing..."

diff --git a/catalyst/main.py b/catalyst/main.py
index dfa0609..04f689e 100644
--- a/catalyst/main.py
+++ b/catalyst/main.py
@@ -182,7 +182,7 @@ def main():
 
 	# parse out the command line arguments
 	try:
-		opts,args = getopt.getopt(sys.argv[1:], "apPThvdc:C:f:FVs:", ["purge", "purgeonly", "purgetmponly", "help", "version", "debug",\
+		opts, _args = getopt.getopt(sys.argv[1:], "apPThvdc:C:f:FVs:", ["purge", "purgeonly", "purgetmponly", "help", "version", "debug",
 			"clear-autoresume", "config=", "cli=", "file=", "fetch", "verbose","snapshot="])
 
 	except getopt.GetoptError:

diff --git a/catalyst/support.py b/catalyst/support.py
index 5563a15..a879eaf 100644
--- a/catalyst/support.py
+++ b/catalyst/support.py
@@ -88,7 +88,7 @@ class CatalystError(Exception):
 	def __init__(self, message, print_traceback=False):
 		if message:
 			if print_traceback:
-				(type,value)=sys.exc_info()[:2]
+				(_type, value) = sys.exc_info()[:2]
 				if value!=None:
 					print
 					print "Traceback values found.  listing..."

diff --git a/catalyst/targets/netboot.py b/catalyst/targets/netboot.py
index c41ed59..b0e322c 100644
--- a/catalyst/targets/netboot.py
+++ b/catalyst/targets/netboot.py
@@ -31,11 +31,12 @@ class netboot(StageBase):
 		self.required_values=[]
 
 		try:
+			# XXX: This code does nothing because the for loop below is disabled.
 			if "netboot/packages" in addlargs:
 				if type(addlargs["netboot/packages"]) == types.StringType:
-					loopy=[addlargs["netboot/packages"]]
+					_loopy = [addlargs["netboot/packages"]]
 				else:
-					loopy=addlargs["netboot/packages"]
+					_loopy = addlargs["netboot/packages"]
 
 		#	for x in loopy:
 		#		self.required_values.append("netboot/packages/"+x+"/files")

diff --git a/setup.py b/setup.py
index feca894..27ed2de 100755
--- a/setup.py
+++ b/setup.py
@@ -34,7 +34,7 @@ def _files(prefix, root):
 
 	Yielding `(target_dir, (file_source_paths, ...))` tuples.
 	"""
-	for dirpath, dirnames, filenames in _os.walk(root):
+	for dirpath, _dirnames, filenames in _os.walk(root):
 		reldir = _os.path.relpath(dirpath, root)
 		install_directory = _posix_path(
 			_os.path.join(prefix, reldir))