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

Unified Diff: scripts/slave/recipe_modules/chromium/chromium_fyi.py

Issue 829363004: Move CrWinClang and Tester to recipes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: . Created 5 years, 11 months 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/chromium/chromium_fyi.py
diff --git a/scripts/slave/recipe_modules/chromium/chromium_fyi.py b/scripts/slave/recipe_modules/chromium/chromium_fyi.py
index 0a043753dc5f0a861acab296eb37fded3865290e..4b21f84df092695ca0d200aad84037fa4e934ed3 100644
--- a/scripts/slave/recipe_modules/chromium/chromium_fyi.py
+++ b/scripts/slave/recipe_modules/chromium/chromium_fyi.py
@@ -388,5 +388,31 @@ SPEC = {
'platform': 'linux',
},
},
+ 'CrWinClang': {
+ 'recipe_config': 'chromium_win_clang',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ 'TARGET_BITS': 32,
+ },
+ 'bot_type': 'builder',
+ 'testing': {
+ 'platform': 'win',
+ },
+ },
+ 'CrWinClang Tester': {
iannucci 2015/01/14 05:30:08 I think there's a 'Tester' v. 'tester' discrepancy
Nico 2015/01/14 06:03:36 Excellent catch, thanks. Will fix.
+ 'recipe_config': 'chromium_win_clang',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ 'TARGET_BITS': 32,
+ },
+ 'test_generators': [
+ steps.generate_gtest,
+ ],
+ 'bot_type': 'tester',
+ 'parent_buildername': 'CrWinClang',
iannucci 2015/01/14 05:30:08 [aside, to no one in particular: WTF is this... we
Paweł Hajdan Jr. 2015/01/14 14:57:03 We don't have that at recipe expectation training
+ 'testing': {
+ 'platform': 'win',
+ },
+ },
},
}

Powered by Google App Engine
This is Rietveld 408576698