commit f6ef4395fe1896ba68c80e52cb24763b0fcfe7f8
author Caroline Tice <cmtice@google.com> Thu Apr 06 17:16:05 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Sat Apr 08 19:40:20 2017 -0700
tree 7612123711db98f0746e56b66368d16b388192bd
parent 8c18be1425c8a4ecfc059a7c637fc93f33edab1f

[toolchain-utils] Fix remaining lint errors in toolchain-utils.

In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).

BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
     Ran afe_lock_machine.py (check machine status)
     Ran full crosperf test (octane, speedometer, BootPerf) on alex.

Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>

cros_login.py

diff --git a/cros_login.py b/cros_login.py
index 32dfcb7..06ff8ff 100755
--- a/cros_login.py
+++ b/cros_login.py
@@ -1,9 +1,8 @@
-#!/usr/bin/python2
+#!/usr/bin/env python2
 #
 # Copyright 2010~2015 Google Inc. All Rights Reserved.
-"""Script to get past the login screen of ChromeOS.
+"""Script to get past the login screen of ChromeOS."""
 
-"""
 from __future__ import print_function
 
 import argparse
@@ -82,15 +81,17 @@
 
   with open(host_login_script, 'w') as f:
     f.write(full_login_script_contents)
-  ce.CopyFiles(host_login_script,
-               device_login_script,
-               dest_machine=remote,
-               chromeos_root=chromeos_root,
-               recursive=False,
-               dest_cros=True)
-  ret = ce.CrosRunCommand('python %s' % device_login_script,
-                          chromeos_root=chromeos_root,
-                          machine=remote)
+  ce.CopyFiles(
+      host_login_script,
+      device_login_script,
+      dest_machine=remote,
+      chromeos_root=chromeos_root,
+      recursive=False,
+      dest_cros=True)
+  ret = ce.CrosRunCommand(
+      'python %s' % device_login_script,
+      chromeos_root=chromeos_root,
+      machine=remote)
   if os.path.exists(host_login_script):
     os.remove(host_login_script)
   return ret
@@ -99,14 +100,10 @@
 def Main(argv):
   """The main function."""
   parser = argparse.ArgumentParser()
-  parser.add_argument('-r',
-                      '--remote',
-                      dest='remote',
-                      help='The remote ChromeOS box.')
-  parser.add_argument('-c',
-                      '--chromeos_root',
-                      dest='chromeos_root',
-                      help='The ChromeOS root.')
+  parser.add_argument(
+      '-r', '--remote', dest='remote', help='The remote ChromeOS box.')
+  parser.add_argument(
+      '-c', '--chromeos_root', dest='chromeos_root', help='The ChromeOS root.')
 
   options = parser.parse_args(argv)