From 8e2fb913a58aa5e305a12c00bf993347023cf41b Mon Sep 17 00:00:00 2001 From: BJ Dierkes Date: Fri, 26 Feb 2016 15:44:25 -0600 Subject: [PATCH] Fix PEP Issues --- ChangeLog | 4 +++- cement/core/foundation.py | 2 +- scripts/devtools.py | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index aa0e7c96..502922d2 100755 --- a/ChangeLog +++ b/ChangeLog @@ -28,7 +28,9 @@ development will happen as 2.9.x under the git master branch. Bugs: - * None + * :issue:`346` - AttributeError: 'module' object has no attribute 'SIGHUP' + on Windows + Features: diff --git a/cement/core/foundation.py b/cement/core/foundation.py index ef6cef93..7c0b2047 100644 --- a/cement/core/foundation.py +++ b/cement/core/foundation.py @@ -17,13 +17,13 @@ from ..utils import fs if sys.version_info[0] >= 3: from imp import reload # pragma: nocover - LOG = minimal_logger(__name__) if platform.system() == 'Windows': SIGNALS = [signal.SIGTERM, signal.SIGINT] else: SIGNALS = [signal.SIGTERM, signal.SIGINT, signal.SIGHUP] + def add_handler_override_options(app): """ This is a ``post_setup`` hook that adds the handler override options to diff --git a/scripts/devtools.py b/scripts/devtools.py index 07a054da..6be7a7a5 100755 --- a/scripts/devtools.py +++ b/scripts/devtools.py @@ -46,7 +46,7 @@ class CementDevtoolsController(ArgparseController): # make sure we don't have any un-added files print('Checking for Untracked Files') out, err, res = shell.exec_cmd(['git', 'status']) - if re.match('Untracked files', str(out): + if re.match('Untracked files', str(out)): self._do_error('There are untracked files. See `git status`.') # make sure there isn't an existing tag