Adding apprtc and downloading of google appengine sdk to webrtc.DEPS.

download_appengine_sdk.py

+#!/usr/bin/python
+# Copyright 2013 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""Downloads the appengine SDK from WebRTC storage and unpacks it.
+
+Requires that depot_tools is installed and in the PATH."""
+
+import os
+import sys
+import subprocess
+import zipfile
+
+
+def _DownloadAppEngineZipFile(webrtc_deps_path):
+  print 'Downloading files in %s...' % webrtc_deps_path
+  cmd = ['download_from_google_storage.py',
+         '--bucket=chromium-webrtc-resources',
+         '--directory', webrtc_deps_path]
+  subprocess.check_call(cmd)

[kjellander_chromium, 2013/10/10 09:02:23]

Does this work on windows without providing sys.executable?

[phoglund_chromium, 2013/10/10 09:38:16]

Nope, and if I do that I will have to locate depot_tools on the system to find
the python file since I can't do PATH lookup. Fortunately I can call the .bat
file on Win instead, which is in the PATH.

[kjellander_chromium, 2013/10/10 09:53:03]

SG, great!

+
+ 
+def _Unzip(path):
+  print 'Unzipping %s...' % path
+  zip_file = zipfile.ZipFile(path)
+  try:
+    zip_file.extractall()
+  finally:
+    zip_file.close()
+
+
+def main(argv):
+  if len(argv) == 1:
+    return 'Usage: %s <path to webrtc.DEPS>' % argv[0]
+
+  webrtc_deps_path = argv[1]
+  _DownloadAppEngineZipFile(webrtc_deps_path)
+  _Unzip(os.path.join(webrtc_deps_path, 'google-appengine.zip'))

[kjellander_chromium, 2013/10/10 09:02:23]

Hmm, is it the right thing to do to exact this into a source controlled folder? 
I was thinking more of something like ~/google_appengine (if it doesn't already
exist)

[phoglund_chromium, 2013/10/10 09:38:16]

It extracts into the level above the source controlled folders, e.g. where the
.gclient file is. I think that's cleaner than writing into a user's home dir.

[kjellander_chromium, 2013/10/10 09:53:03]

That sounds like the best approach, yes.

+
+
+if __name__ == '__main__':
+  sys.exit(main(sys.argv))