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

Unified Diff: scripts/slave/recipes/webrtc/ios_api_framework.py

Issue 2011973003: WebRTC: Add trybot for iOS SDK Framework build. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 7 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/recipes/webrtc/ios_api_framework.py
diff --git a/scripts/slave/recipes/webrtc/ios_api_framework.py b/scripts/slave/recipes/webrtc/ios_api_framework.py
index b22843894e70d8c834b64429d73f68d62889fe84..3d5a926c9a5b9a1cc3dd075bc5d79036f49948d7 100644
--- a/scripts/slave/recipes/webrtc/ios_api_framework.py
+++ b/scripts/slave/recipes/webrtc/ios_api_framework.py
@@ -7,6 +7,7 @@ DEPS = [
'commit_position',
'depot_tools/bot_update',
'depot_tools/gclient',
+ 'depot_tools/tryserver',
'file',
'gsutil',
'ios',
@@ -29,24 +30,26 @@ def RunSteps(api):
build_script = api.path['checkout'].join('webrtc', 'build', 'ios',
'build_ios_libs.sh')
- api.step('cleanup', [build_script, '-c'], cwd=api.path['checkout'])
+ if not api.tryserver.is_tryserver:
+ api.step('cleanup', [build_script, '-c'], cwd=api.path['checkout'])
api.step('build', [build_script, '-r', commit_pos], cwd=api.path['checkout'])
- output_dir = api.path['checkout'].join('out_ios_libs')
- zip_out = api.path['slave_build'].join('webrtc_ios_api_framework.zip')
- pkg = api.zip.make_package(output_dir, zip_out)
- pkg.add_directory(output_dir.join('WebRTC.framework'))
- pkg.add_directory(output_dir.join('WebRTC.framework.dSYM'))
- pkg.add_file(output_dir.join('LICENSE.html'))
- pkg.zip('zip archive')
+ if not api.tryserver.is_tryserver:
+ output_dir = api.path['checkout'].join('out_ios_libs')
+ zip_out = api.path['slave_build'].join('webrtc_ios_api_framework.zip')
+ pkg = api.zip.make_package(output_dir, zip_out)
+ pkg.add_directory(output_dir.join('WebRTC.framework'))
+ pkg.add_directory(output_dir.join('WebRTC.framework.dSYM'))
+ pkg.add_file(output_dir.join('LICENSE.html'))
+ pkg.zip('zip archive')
- api.gsutil.upload(
- zip_out,
- 'chromium-webrtc',
- 'ios_api_framework/webrtc_ios_api_framework_%d.zip' % commit_pos,
- args=['-a', 'public-read'],
- unauthenticated_url=True)
+ api.gsutil.upload(
+ zip_out,
+ 'chromium-webrtc',
+ 'ios_api_framework/webrtc_ios_api_framework_%d.zip' % commit_pos,
+ args=['-a', 'public-read'],
+ unauthenticated_url=True)
def GenTests(api):
@@ -62,3 +65,9 @@ def GenTests(api):
buildername='iOS API Framework Builder') +
api.step_data('build', retcode=1)
)
+
+ yield (
+ api.test('trybot_biuld') +
phoglund_chromium 2016/05/26 07:30:07 biuld? That doesn't look right. I guess it's not a
kjellander_chromium 2016/05/26 08:11:59 Right, it only affects the name of the expectation
+ api.properties.tryserver(mastername='tryserver.webrtc',
+ buildername='ios_api_framework')
+ )
« no previous file with comments | « masters/master.tryserver.webrtc/slaves.cfg ('k') | scripts/slave/recipes/webrtc/ios_api_framework.expected/trybot_biuld.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698