Add asset management scripts

infra/bots/assets/scripts/download.py

Index: infra/bots/assets/scripts/download.py
diff --git a/infra/bots/assets/scripts/download.py b/infra/bots/assets/scripts/download.py
new file mode 100755
index 0000000000000000000000000000000000000000..553a6ec46542295814a6c40f72af153d16dd73fb
--- /dev/null
+++ b/infra/bots/assets/scripts/download.py
@@ -0,0 +1,25 @@
+#!/usr/bin/env python
+#
+# Copyright 2016 Google Inc.
+#
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+
+"""Download the current version of the asset."""
+
+
+import os
+import sys
+
+FILE_DIR = os.path.dirname(os.path.abspath(__file__))
+INFRA_BOTS_DIR = os.path.realpath(os.path.join(FILE_DIR, os.pardir, os.pardir))
+
+sys.path.insert(0, INFRA_BOTS_DIR)
+from assets import assets
+
+ASSET_NAME = os.path.basename(FILE_DIR)
+
+
+if __name__ == '__main__':
+  assets.main(['download', ASSET_NAME] + sys.argv[1:])

[rmistry, 2016/06/15 13:47:18]

sys.exit(...) here and in all scripts with "if __name__ == '__main__'"
MA always asks for this and I tend to agree.

[borenet, 2016/06/15 14:31:27]

I'm not sure what you mean.  We'll exit with code zero if there's no exception,
and since we don't trap exceptions, those will cause us to exit with non-zero
code.

[rmistry, 2016/06/15 14:49:40]

Right, the advantage of using sys.exit is when the function you call returns
specific ret codes. You do not need it here because you control everything you
call. I like using it in general because it handles ret codes in addition to
Exceptions incase anything changes in the future. What you have right now is
fine.