diff options
author | csmartdalton <csmartdalton@google.com> | 2016-11-09 16:34:53 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-10 12:52:14 +0000 |
commit | 2087dda1631594819f7290b79392d2e73d7e2742 (patch) | |
tree | 778b17daca1497df8e6183618dbaa0149e6e8fcb /tools/skpbench/skpbench.py | |
parent | 1f2f64b4ce586348cf951b09894b858d1cea8836 (diff) |
skpbench: remove --path flag
Replaces the confusing --path flag with a positional argument that
gives the path to the skpbench binary.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4635
Change-Id: I2b5eeca61ec85e7fe45fd3370625eddf34a2fd0e
Reviewed-on: https://skia-review.googlesource.com/4635
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'tools/skpbench/skpbench.py')
-rwxr-xr-x | tools/skpbench/skpbench.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/tools/skpbench/skpbench.py b/tools/skpbench/skpbench.py index 372d66f7e9..42263f073b 100755 --- a/tools/skpbench/skpbench.py +++ b/tools/skpbench/skpbench.py @@ -29,13 +29,13 @@ unacceptable stddev. """) +__argparse.add_argument('skpbench', + help="path to the skpbench binary") __argparse.add_argument('--adb', action='store_true', help="execute skpbench over adb") __argparse.add_argument('-s', '--device-serial', help="if using adb, ID of the specific device to target " "(only required if more than 1 device is attached)") -__argparse.add_argument('-p', '--path', - help="directory to execute ./skpbench from") __argparse.add_argument('-m', '--max-stddev', type=float, default=4, help="initial max allowable relative standard deviation") @@ -97,7 +97,7 @@ class SubprocessMonitor(Thread): self._queue.put(Message(Message.EXIT)) class SKPBench: - ARGV = ['skpbench', '--verbosity', str(FLAGS.verbosity)] + ARGV = [FLAGS.skpbench, '--verbosity', str(FLAGS.verbosity)] if FLAGS.duration: ARGV.extend(['--duration', str(FLAGS.duration)]) if FLAGS.sample_ms: @@ -106,13 +106,11 @@ class SKPBench: ARGV.extend(['--gpuClock', 'true']) if FLAGS.fps: ARGV.extend(['--fps', 'true']) - if FLAGS.path: - ARGV[0] = _path.join(FLAGS.path, ARGV[0]) if FLAGS.adb: if FLAGS.device_serial is None: - ARGV = ['adb', 'shell'] + ARGV + ARGV[:0] = ['adb', 'shell'] else: - ARGV = ['adb', '-s', FLAGS.device_serial, 'shell'] + ARGV + ARGV[:0] = ['adb', '-s', FLAGS.device_serial, 'shell'] @classmethod def print_header(cls): |