public inbox for gentoo-catalyst@lists.gentoo.org
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: gentoo-catalyst@lists.gentoo.org
Subject: [gentoo-catalyst] [PATCH] lint: clean up global statement warnings
Date: Tue,  6 Oct 2015 00:16:35 -0400	[thread overview]
Message-ID: <1444104995-16709-1-git-send-email-vapier@gentoo.org> (raw)

Delete unused calls and add disables for the ones we actively want.  We
generally want to avoid calls to "global" so force people to disable the
warning whenever they try to use it.
---
 catalyst/support.py | 2 --
 catalyst/version.py | 2 ++
 setup.py            | 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/catalyst/support.py b/catalyst/support.py
index a879eaf..1e3eeef 100644
--- a/catalyst/support.py
+++ b/catalyst/support.py
@@ -294,8 +294,6 @@ def ismount(path):
 
 def addl_arg_parse(myspec,addlargs,requiredspec,validspec):
 	"helper function to help targets parse additional arguments"
-	global valid_config_file_values
-
 	messages = []
 	for x in addlargs.keys():
 		if x not in validspec and x not in valid_config_file_values and x not in requiredspec:
diff --git a/catalyst/version.py b/catalyst/version.py
index d5c5cde..5deb4d7 100644
--- a/catalyst/version.py
+++ b/catalyst/version.py
@@ -18,6 +18,7 @@ _ver = None
 
 def get_git_version(version=__version__):
 	"""Return: a string describing our version."""
+	# pylint: disable=global-statement
 	global _ver
 	cwd = os.path.dirname(os.path.abspath(__file__))
 	version_info = get_ver(cwd)
@@ -39,6 +40,7 @@ def get_version(reset=False):
 	'''Returns a saved release version string or the
 	generated git release version.
 	'''
+	# pylint: disable=global-statement
 	global __version__, _ver
 	if _ver and not reset:
 		return _ver
diff --git a/setup.py b/setup.py
index 27ed2de..e4569ee 100755
--- a/setup.py
+++ b/setup.py
@@ -55,7 +55,6 @@ _data_files.extend(_files('share/catalyst/targets', 'targets'))
 class set_version(_Command):
 	'''Saves the specified release version information
 	'''
-	global __version__
 	description = "hardcode script's version using VERSION from environment"
 	user_options = []  # [(long_name, short_name, desc),]
 
@@ -66,6 +65,8 @@ class set_version(_Command):
 		pass
 
 	def run(self):
+		# pylint: disable=global-statement
+		global __version__
 		try:
 			version = _os.environ['VERSION']
 		except KeyError:
-- 
2.5.2



             reply	other threads:[~2015-10-06  4:16 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-06  4:16 Mike Frysinger [this message]
2015-10-06  6:12 ` [gentoo-catalyst] [PATCH] lint: clean up global statement warnings Brian Dolbec

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1444104995-16709-1-git-send-email-vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=gentoo-catalyst@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox