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

Unified Diff: masters/master.client.webrtc/master_android_cfg.py

Issue 505153002: WebRTC: Remove android_apk recipe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Rebased Created 6 years, 4 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
« no previous file with comments | « no previous file | masters/master.client.webrtc/slaves.cfg » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: masters/master.client.webrtc/master_android_cfg.py
diff --git a/masters/master.client.webrtc/master_android_cfg.py b/masters/master.client.webrtc/master_android_cfg.py
index a1495ca40a3bfec66936c2f6d4891a312970d183..e33c1d350a8f25f194319868ee32f7df955f43ca 100644
--- a/masters/master.client.webrtc/master_android_cfg.py
+++ b/masters/master.client.webrtc/master_android_cfg.py
@@ -15,22 +15,20 @@ def Update(c):
branch='trunk',
treeStableTimer=30,
builderNames=[
- 'Android',
- 'Android (dbg)',
+ 'Android Builder',
+ 'Android Builder (dbg)',
'Android Clang (dbg)',
'Android ARM64 (dbg)',
'Android GN',
'Android GN (dbg)',
- 'Android Chromium-APK Builder',
- 'Android Chromium-APK Builder (dbg)',
]),
Triggerable(name='android_trigger_dbg', builderNames=[
- 'Android Chromium-APK Tests (KK Nexus5)(dbg)',
- 'Android Chromium-APK Tests (JB Nexus7.2)(dbg)',
+ 'Android Tests (KK Nexus5)(dbg)',
+ 'Android Tests (JB Nexus7.2)(dbg)',
]),
Triggerable(name='android_trigger_rel', builderNames=[
- 'Android Chromium-APK Tests (KK Nexus5)',
- 'Android Chromium-APK Tests (JB Nexus7.2)',
+ 'Android Tests (KK Nexus5)',
+ 'Android Tests (JB Nexus7.2)',
]),
])
@@ -38,57 +36,42 @@ def Update(c):
# of the builders are pooled over multiple slave machines.
specs = [
{
- 'name': 'Android',
- 'recipe': 'webrtc/standalone',
- 'slavebuilddir': 'android',
+ 'name': 'Android Builder',
+ 'triggers': ['android_trigger_rel'],
},
{
- 'name': 'Android (dbg)',
- 'recipe': 'webrtc/standalone',
- 'slavebuilddir': 'android',
+ 'name': 'Android Builder (dbg)',
+ 'triggers': ['android_trigger_dbg'],
},
{
'name': 'Android Clang (dbg)',
- 'recipe': 'webrtc/standalone',
'slavebuilddir': 'android_clang',
},
{
'name': 'Android ARM64 (dbg)',
- 'recipe': 'webrtc/standalone',
'slavebuilddir': 'android_arm64',
},
{
'name': 'Android GN',
- 'recipe': 'webrtc/standalone',
'slavebuilddir': 'android_gn',
},
{
'name': 'Android GN (dbg)',
- 'recipe': 'webrtc/standalone',
'slavebuilddir': 'android_gn',
},
- {
- 'name': 'Android Chromium-APK Builder',
- 'triggers': ['android_trigger_rel'],
- },
- {
- 'name': 'Android Chromium-APK Builder (dbg)',
- 'triggers': ['android_trigger_dbg'],
- },
- {'name': 'Android Chromium-APK Tests (KK Nexus5)(dbg)'},
- {'name': 'Android Chromium-APK Tests (JB Nexus7.2)(dbg)'},
- {'name': 'Android Chromium-APK Tests (KK Nexus5)'},
- {'name': 'Android Chromium-APK Tests (JB Nexus7.2)'},
+ {'name': 'Android Tests (KK Nexus5)(dbg)'},
+ {'name': 'Android Tests (JB Nexus7.2)(dbg)'},
+ {'name': 'Android Tests (KK Nexus5)'},
+ {'name': 'Android Tests (JB Nexus7.2)'},
]
c['builders'].extend([
{
'name': spec['name'],
- 'factory': m_annotator.BaseFactory(
- spec.get('recipe', 'webrtc/android_apk'),
- triggers=spec.get('triggers')),
+ 'factory': m_annotator.BaseFactory('webrtc/standalone',
+ triggers=spec.get('triggers')),
'notify_on_missing': True,
'category': 'android',
- 'slavebuilddir': spec.get('slavebuilddir', 'android_apk'),
+ 'slavebuilddir': spec.get('slavebuilddir', 'android'),
} for spec in specs
])
« no previous file with comments | « no previous file | masters/master.client.webrtc/slaves.cfg » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698