Run pyformat on all the toolchain-utils files.
This gets rid of a lot of lint issues.
Ran by doing this:
for f in *.py; do echo -n "$f " ; if [ -x $f ]; then pyformat -i
--remove_trailing_comma --yapf --force_quote_type=double $f ; else
pyformat -i --remove_shebang --remove_trailing_comma --yapf
--force_quote_type=double $f ; fi ; done
BUG=chromium:567921
TEST=Ran simple crosperf run.
Change-Id: I59778835fdaa5f706d2e1765924389f9e97433d1
Reviewed-on: https://chrome-internal-review.googlesource.com/242031
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
diff --git a/remote_test.py b/remote_test.py
index f3ff3fc..59eb9eb 100755
--- a/remote_test.py
+++ b/remote_test.py
@@ -1,13 +1,12 @@
#!/usr/bin/python
#
# Copyright 2010 Google Inc. All Rights Reserved.
-
"""Script to wrap test_that script.
This script can login to the chromeos machine using the test private key.
"""
-__author__ = "asharif@google.com (Ahmad Sharif)"
+__author__ = 'asharif@google.com (Ahmad Sharif)'
import optparse
import os
@@ -19,26 +18,31 @@
def Usage(parser, message):
- print "ERROR: " + message
+ print 'ERROR: ' + message
parser.print_help()
sys.exit(0)
+
def Main(argv):
parser = optparse.OptionParser()
- parser.add_option("-c", "--chromeos_root", dest="chromeos_root",
- help="ChromeOS root checkout directory")
- parser.add_option("-r", "--remote", dest="remote",
- help="Remote chromeos device.")
+ parser.add_option('-c',
+ '--chromeos_root',
+ dest='chromeos_root',
+ help='ChromeOS root checkout directory')
+ parser.add_option('-r',
+ '--remote',
+ dest='remote',
+ help='Remote chromeos device.')
options = parser.parse_args(argv)[0]
if options.chromeos_root is None:
- Usage(parser, "chromeos_root must be given")
+ Usage(parser, 'chromeos_root must be given')
if options.remote is None:
- Usage(parser, "remote must be given")
+ Usage(parser, 'remote must be given')
options.chromeos_root = os.path.expanduser(options.chromeos_root)
- command = "ls -lt /"
+ command = 'ls -lt /'
ce = command_executer.GetCommandExecuter()
ce.CrosRunCommand(command,
chromeos_root=options.chromeos_root,
@@ -49,37 +53,37 @@
# Tests to copy directories and files to the chromeos box.
ce.CopyFiles(version_dir_path,
- "/tmp/" + version_dir,
+ '/tmp/' + version_dir,
dest_machine=options.remote,
dest_cros=True,
chromeos_root=options.chromeos_root)
ce.CopyFiles(version_dir_path,
- "/tmp/" + version_dir + "1",
+ '/tmp/' + version_dir + '1',
dest_machine=options.remote,
dest_cros=True,
chromeos_root=options.chromeos_root)
ce.CopyFiles(sys.argv[0],
- "/tmp/" + script_name,
+ '/tmp/' + script_name,
recursive=False,
dest_machine=options.remote,
dest_cros=True,
chromeos_root=options.chromeos_root)
ce.CopyFiles(sys.argv[0],
- "/tmp/" + script_name + "1",
+ '/tmp/' + script_name + '1',
recursive=False,
dest_machine=options.remote,
dest_cros=True,
chromeos_root=options.chromeos_root)
# Test to copy directories and files from the chromeos box.
- ce.CopyFiles("/tmp/" + script_name,
- "/tmp/hello",
+ ce.CopyFiles('/tmp/' + script_name,
+ '/tmp/hello',
recursive=False,
src_machine=options.remote,
src_cros=True,
chromeos_root=options.chromeos_root)
- ce.CopyFiles("/tmp/" + script_name,
- "/tmp/" + script_name,
+ ce.CopyFiles('/tmp/' + script_name,
+ '/tmp/' + script_name,
recursive=False,
src_machine=options.remote,
src_cros=True,
@@ -89,5 +93,5 @@
return 0
-if __name__ == "__main__":
+if __name__ == '__main__':
Main(sys.argv)