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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 505153002: WebRTC: Remove android_apk recipe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Updated for https://codereview.chromium.org/500423004/ 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 | « scripts/slave/recipe_modules/webrtc/api.py ('k') | scripts/slave/recipe_modules/webrtc/chromium_android_config.py » ('j') | scripts/slave/recipes/webrtc/chromium.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/webrtc/builders.py
diff --git a/scripts/slave/recipe_modules/webrtc/builders.py b/scripts/slave/recipe_modules/webrtc/builders.py
index 8dc5f8db9d5d6e5af1a55a4f432601910727e704..f35ed940187bdf00bb1c2cb609013c2457d4232b 100644
--- a/scripts/slave/recipe_modules/webrtc/builders.py
+++ b/scripts/slave/recipe_modules/webrtc/builders.py
@@ -28,13 +28,11 @@ RECIPE_CONFIGS = {
},
'webrtc_android': {
'webrtc_config': 'webrtc_android',
+ 'test_suite': 'android',
},
'webrtc_android_clang': {
'webrtc_config': 'webrtc_android_clang',
},
- 'webrtc_android_apk': {
- 'webrtc_config': 'webrtc_android_apk',
- },
'webrtc_ios': {
'webrtc_config': 'webrtc_ios',
},
@@ -690,7 +688,7 @@ BUILDERS = {
'bot_type': 'builder_tester',
'testing': {'platform': 'linux'},
},
- 'Android': {
+ 'Android Builder': {
'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Release',
@@ -699,9 +697,10 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'builder',
+ 'build_gs_archive': 'android_apk_rel_archive',
'testing': {'platform': 'linux'},
},
- 'Android (dbg)': {
+ 'Android Builder (dbg)': {
'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Debug',
@@ -710,6 +709,7 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'builder',
+ 'build_gs_archive': 'android_apk_dbg_archive',
'testing': {'platform': 'linux'},
},
'Android ARM64 (dbg)': {
@@ -758,32 +758,8 @@ BUILDERS = {
'bot_type': 'builder',
'testing': {'platform': 'linux'},
},
- 'Android Chromium-APK Builder (dbg)': {
- 'recipe_config': 'webrtc_android_apk',
- 'webrtc_config_kwargs': {
- 'BUILD_CONFIG': 'Debug',
- 'TARGET_PLATFORM': 'android',
- 'TARGET_ARCH': 'arm',
- 'TARGET_BITS': 32,
- },
- 'bot_type': 'builder',
- 'build_gs_archive': 'android_apk_dbg_archive',
- 'testing': {'platform': 'linux'},
- },
- 'Android Chromium-APK Builder': {
- 'recipe_config': 'webrtc_android_apk',
- 'webrtc_config_kwargs': {
- 'BUILD_CONFIG': 'Release',
- 'TARGET_PLATFORM': 'android',
- 'TARGET_ARCH': 'arm',
- 'TARGET_BITS': 32,
- },
- 'bot_type': 'builder',
- 'build_gs_archive': 'android_apk_rel_archive',
- 'testing': {'platform': 'linux'},
- },
- 'Android Chromium-APK Tests (KK Nexus5)(dbg)': {
- 'recipe_config': 'webrtc_android_apk',
+ 'Android Tests (KK Nexus5)(dbg)': {
+ 'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Debug',
'TARGET_PLATFORM': 'android',
@@ -791,12 +767,12 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'tester',
- 'parent_buildername': 'Android Chromium-APK Builder (dbg)',
+ 'parent_buildername': 'Android Builder (dbg)',
'build_gs_archive': 'android_apk_dbg_archive',
'testing': {'platform': 'linux'},
},
- 'Android Chromium-APK Tests (KK Nexus5)': {
- 'recipe_config': 'webrtc_android_apk',
+ 'Android Tests (KK Nexus5)': {
+ 'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_PLATFORM': 'android',
@@ -804,12 +780,12 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'tester',
- 'parent_buildername': 'Android Chromium-APK Builder',
+ 'parent_buildername': 'Android Builder',
'build_gs_archive': 'android_apk_rel_archive',
'testing': {'platform': 'linux'},
},
- 'Android Chromium-APK Tests (JB Nexus7.2)(dbg)': {
- 'recipe_config': 'webrtc_android_apk',
+ 'Android Tests (JB Nexus7.2)(dbg)': {
+ 'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Debug',
'TARGET_PLATFORM': 'android',
@@ -817,12 +793,12 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'tester',
- 'parent_buildername': 'Android Chromium-APK Builder (dbg)',
+ 'parent_buildername': 'Android Builder (dbg)',
'build_gs_archive': 'android_apk_dbg_archive',
'testing': {'platform': 'linux'},
},
- 'Android Chromium-APK Tests (JB Nexus7.2)': {
- 'recipe_config': 'webrtc_android_apk',
+ 'Android Tests (JB Nexus7.2)': {
+ 'recipe_config': 'webrtc_android',
'webrtc_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_PLATFORM': 'android',
@@ -830,7 +806,7 @@ BUILDERS = {
'TARGET_BITS': 32,
},
'bot_type': 'tester',
- 'parent_buildername': 'Android Chromium-APK Builder',
+ 'parent_buildername': 'Android Builder',
'build_gs_archive': 'android_apk_rel_archive',
'testing': {'platform': 'linux'},
},
@@ -1124,7 +1100,7 @@ BUILDERS = {
'TARGET_ARCH': 'arm',
'TARGET_BITS': 32,
},
- 'bot_type': 'builder',
+ 'bot_type': 'builder_tester',
'testing': {'platform': 'linux'},
},
'android_rel': {
@@ -1135,7 +1111,7 @@ BUILDERS = {
'TARGET_ARCH': 'arm',
'TARGET_BITS': 32,
},
- 'bot_type': 'builder',
+ 'bot_type': 'builder_tester',
'testing': {'platform': 'linux'},
},
'android_clang': {
@@ -1184,28 +1160,6 @@ BUILDERS = {
'bot_type': 'builder',
'testing': {'platform': 'linux'},
},
- 'android_apk': {
- 'recipe_config': 'webrtc_android_apk',
- 'webrtc_config_kwargs': {
- 'BUILD_CONFIG': 'Debug',
- 'TARGET_PLATFORM': 'android',
- 'TARGET_ARCH': 'arm',
- 'TARGET_BITS': 32,
- },
- 'bot_type': 'builder_tester',
- 'testing': {'platform': 'linux'},
- },
- 'android_apk_rel': {
- 'recipe_config': 'webrtc_android_apk',
- 'webrtc_config_kwargs': {
- 'BUILD_CONFIG': 'Release',
- 'TARGET_PLATFORM': 'android',
- 'TARGET_ARCH': 'arm',
- 'TARGET_BITS': 32,
- },
- 'bot_type': 'builder_tester',
- 'testing': {'platform': 'linux'},
- },
},
},
}
« no previous file with comments | « scripts/slave/recipe_modules/webrtc/api.py ('k') | scripts/slave/recipe_modules/webrtc/chromium_android_config.py » ('j') | scripts/slave/recipes/webrtc/chromium.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698