Chromium Code Reviews| 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') |
| + ) |