commit d81977f51b865de191a859309055b8992f25698b
author Ravi Mistry <rmistry@google.com> Wed Jul 19 09:30:53 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Jul 19 13:44:34 2017 +0000
tree cd94742fb71fc5a177f04f0a48278b2bb961c7af
parent 430e4c84e9a2229e386741c7c06b51e679d9fef3

Download material design SVGs from Google Storage

NoTry: true
Bug: skia:6865
Change-Id: Id857d6ad0b5cdccbadb11810bf07a093467544ec
Reviewed-on: https://skia-review.googlesource.com/24086
Commit-Queue: Ravi Mistry <rmistry@google.com>
Reviewed-by: Eric Boren <borenet@google.com>

infra/bots/assets/svg/create.py

diff --git a/infra/bots/assets/svg/create.py b/infra/bots/assets/svg/create.py
index 2b9df46..312d18e 100755
--- a/infra/bots/assets/svg/create.py
+++ b/infra/bots/assets/svg/create.py
@@ -18,9 +18,10 @@
 
 SVG_TOOLS = os.path.join(common.INFRA_BOTS_DIR, os.pardir, os.pardir, 'tools',
                          'svg')
+SVG_GS_BUCKET = 'gs://skia-svgs'
 
 
-def create_asset(local_svgs_dir, target_dir):
+def create_asset(target_dir):
   """Create the asset."""
   target_dir = os.path.realpath(target_dir)
 
@@ -34,19 +35,21 @@
   ]
   subprocess.check_call(download_svgs_cmd)
 
-  # Copy over the SVGs from local_svgs_dir (if any).
-  if local_svgs_dir and os.path.exists(local_svgs_dir):
-    for svg_filename in os.listdir(local_svgs_dir):
-      shutil.copy(src=os.path.join(local_svgs_dir, svg_filename),
-                  dst=os.path.join(target_dir, svg_filename))
+  # Download SVGs from Google storage. These are material design SVGs from
+  # skbug.com/5757.
+  # There was no easy way to create URLs from them to specify in
+  # tools/svg/svgs.txt which is why they had been stored in Google storage.
+  subprocess.check_call([
+      'gsutil', '-m', 'cp', os.path.join(SVG_GS_BUCKET, 'skbug5757', '*'),
+      target_dir
+  ])
 
 
 def main():
   parser = argparse.ArgumentParser()
-  parser.add_argument('--local_svgs_dir', '-l', default='')
   parser.add_argument('--target_dir', '-t', required=True)
   args = parser.parse_args()
-  create_asset(args.local_svgs_dir, args.target_dir)
+  create_asset(args.target_dir)
 
 
 if __name__ == '__main__':