toolchain-utils: fix remaining cros lint errors.
BUG=chromium:570458
TEST='crosperf/run_tests.sh' passes.
Change-Id: I5566349cc97233e1ddb6d98121cd6f44e14bf0a9
Reviewed-on: https://chrome-internal-review.googlesource.com/249482
Commit-Ready: Rahul Chaudhry <rahulchaudhry@google.com>
Tested-by: Rahul Chaudhry <rahulchaudhry@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>
diff --git a/crosperf/crosperf.py b/crosperf/crosperf.py
index 6b00cf7..e874277 100755
--- a/crosperf/crosperf.py
+++ b/crosperf/crosperf.py
@@ -15,6 +15,10 @@
from experiment_factory import ExperimentFactory
from experiment_file import ExperimentFile
from settings_factory import GlobalSettings
+
+# This import causes pylint to warn about "No name 'logger' in module 'utils'".
+# I do not understand why. The import works fine in python.
+# pylint: disable=no-name-in-module
from utils import logger
import test_flag
diff --git a/crosperf/suite_runner.py b/crosperf/suite_runner.py
index b7b26bf..b0b4c3a 100644
--- a/crosperf/suite_runner.py
+++ b/crosperf/suite_runner.py
@@ -1,4 +1,3 @@
-
# Copyright (c) 2013~2015 The Chromium OS Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
diff --git a/utils/buildbot_json.py b/utils/buildbot_json.py
index 3800b90..693a42c 100755
--- a/utils/buildbot_json.py
+++ b/utils/buildbot_json.py
@@ -40,8 +40,15 @@
import datetime
import functools
import json
+
+# Pylint recommends we use "from chromite.lib import cros_logging as logging".
+# Chromite specific policy message, we want to keep using the standard logging.
+# pylint: disable=cros-logging-import
import logging
+
+# pylint: disable=deprecated-module
import optparse
+
import time
import urllib
import urllib2
diff --git a/utils/misc.py b/utils/misc.py
index 1ffd235..ae234fe 100644
--- a/utils/misc.py
+++ b/utils/misc.py
@@ -470,6 +470,9 @@
### First of all, we need chromite libs
sys.path.append(os.path.join(chromeos_root, 'chromite'))
+ # Imports below are ok after modifying path to add chromite.
+ # Pylint cannot detect that and complains.
+ # pylint: disable=import-error
from lib import git
from lib import gerrit
manifest = git.ManifestCheckout(chromeos_root)
@@ -541,7 +544,8 @@
except KeyboardInterrupt:
# If the user hits CTRL+C, just exit the process.
print()
- Die('CTRL+C detected; exiting')
+ print('CTRL+C detected; exiting')
+ sys.exit()
if not response:
return default