commit f7cf794890d4ba978152c5da05eaa9a53b4f05d9
author Nathaniel Nifong <nifong@google.com> Wed Sep 11 14:00:20 2019 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Sep 11 20:20:33 2019 +0000
tree 846c7f840a4e4e39a44134abebf546ef212a1f03
parent 50f377e275c34fc29fd142b62736b7bc553baa9b

Assert there are some sources when running skpbench

Change-Id: I6009dfc7c6eabe04b3b6fdf68f38995d718b4fef
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235278
Commit-Queue: Nathaniel Nifong <nifong@google.com>
Reviewed-by: Ben Wagner aka dogben <benjaminwagner@google.com>

tools/skpbench/_adb_path.py

diff --git a/tools/skpbench/_adb_path.py b/tools/skpbench/_adb_path.py
index 407f403..966ab44 100644
--- a/tools/skpbench/_adb_path.py
+++ b/tools/skpbench/_adb_path.py
@@ -27,7 +27,7 @@
   return __ADB.check('''\
     for PATHNAME in %s; do
       if [ -d "$PATHNAME" ]; then
-        find "$PATHNAME" -maxdepth 1 -name *.skp
+        find "$PATHNAME" -maxdepth 1 -name '*.skp' -o -name '*.mskp'
       else
         echo "$PATHNAME"
       fi

tools/skpbench/_os_path.py

diff --git a/tools/skpbench/_os_path.py b/tools/skpbench/_os_path.py
index 42b1c42..2c10265 100644
--- a/tools/skpbench/_os_path.py
+++ b/tools/skpbench/_os_path.py
@@ -17,6 +17,7 @@
   for skp in skps:
     if (path.isdir(skp)):
       pathnames.extend(glob.iglob(path.join(skp, '*.skp')))
+      pathnames.extend(glob.iglob(path.join(skp, '*.mskp')))
     else:
       pathnames.append(skp)
   return pathnames

tools/skpbench/skpbench.py

diff --git a/tools/skpbench/skpbench.py b/tools/skpbench/skpbench.py
index df26232..7e40df4 100755
--- a/tools/skpbench/skpbench.py
+++ b/tools/skpbench/skpbench.py
@@ -327,6 +327,7 @@
   DELIMITER = r'[, ](?!(?:[^(]*\([^)]*\))*[^()]*\))'
   configs = re.split(DELIMITER, FLAGS.config)
   srcs = _path.find_skps(FLAGS.srcs)
+  assert srcs
 
   if FLAGS.adb:
     adb = Adb(FLAGS.device_serial, FLAGS.adb_binary,