Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1709)

Unified Diff: scripts/slave/recipe_modules/webrtc/gclient_config.py

Issue 75163006: WebRTC Android APK trybot recipe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/webrtc/gclient_config.py
diff --git a/scripts/slave/recipe_modules/webrtc/gclient_config.py b/scripts/slave/recipe_modules/webrtc/gclient_config.py
new file mode 100644
index 0000000000000000000000000000000000000000..b030bc92c1e24babfc47da6f0d65f2d0c6195b52
--- /dev/null
+++ b/scripts/slave/recipe_modules/webrtc/gclient_config.py
@@ -0,0 +1,46 @@
+# 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.
+
+from RECIPE_MODULES.gclient import CONFIG_CTX
+from slave.recipe_config import BadConf
+from slave.recipe_modules.gclient.config import ChromeSvnSubURL,\
+ ChromiumSvnSubURL
iannucci 2013/11/18 18:49:26 I'm 'meh' on this, but I really don't have a bette
kjellander_chromium 2013/11/20 14:29:16 Yeah, I guess such paths could be refactored into
+
+
+@CONFIG_CTX(includes=['chromium', '_webrtc_additional_solutions'])
+def webrtc_android_apk_try_builder(c):
+ c.target_os = ['android']
iannucci 2013/11/18 18:49:26 I wonder if gclient's config.py should be informed
kjellander_chromium 2013/11/20 14:29:16 I guess it could, but it would confuse a bit as we
+
+ solution = c.solutions.add()
+ solution.name = 'src/third_party/webrtc'
+ solution.url = 'http://webrtc.googlecode.com/svn/trunk/webrtc'
+
+ # Make the Chromium solution build the ToT revision.
+ cr_solution = c.solutions[0]
+ cr_solution.revision = 'HEAD'
+
+ # Must set src/third_party/webrtc to None to avoid an error due to the
+ # separate WebRTC solution we have configured in there.
+ cr_solution.custom_deps['src/third_party/webrtc'] = None
+
+
+@CONFIG_CTX()
+def _webrtc_additional_solutions(c):
+ """Helper config for loading additional solutions.
+
+ The webrtc-limited solution contains non-redistributable code.
+ The webrtc.DEPS solution pulls in additional DEPS for building Android tests
+ and running special test setups.
+ """
+ if c.GIT_MODE:
+ raise BadConf('WebRTC only supports svn')
+
+ s = c.solutions.add()
+ s.name = 'webrtc-limited'
+ s.url = ChromeSvnSubURL(c, 'chrome-internal', 'trunk', 'webrtc-limited')
+
+ s = c.solutions.add()
+ s.name = 'webrtc.DEPS'
+ s.url = ChromiumSvnSubURL(c, 'chrome', 'trunk', 'deps', 'third_party',
+ 'webrtc', 'webrtc.DEPS')

Powered by Google App Engine
This is Rietveld 408576698