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

Side by Side Diff: scripts/slave/recipes/flutter/engine.py

Issue 1919193002: build: roll infra_paths changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: merge 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import contextlib 5 import contextlib
6 6
7 DEPS = [ 7 DEPS = [
8 'depot_tools/bot_update', 8 'depot_tools/bot_update',
9 'depot_tools/gclient', 9 'depot_tools/gclient',
10 'depot_tools/infra_paths',
10 'file', 11 'file',
11 'gsutil', 12 'gsutil',
12 'recipe_engine/path', 13 'recipe_engine/path',
13 'recipe_engine/platform', 14 'recipe_engine/platform',
14 'recipe_engine/properties', 15 'recipe_engine/properties',
15 'recipe_engine/step', 16 'recipe_engine/step',
16 'zip', 17 'zip',
17 ] 18 ]
18 19
19 BUCKET_NAME = 'flutter_infra' 20 BUCKET_NAME = 'flutter_infra'
(...skipping 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 api.gclient.c.got_revision_mapping['src'] = 'got_engine_revision' 226 api.gclient.c.got_revision_mapping['src'] = 'got_engine_revision'
226 # TODO(eseidel): According to iannucci force=True is required. 227 # TODO(eseidel): According to iannucci force=True is required.
227 # See https://codereview.chromium.org/1690713003#msg6 228 # See https://codereview.chromium.org/1690713003#msg6
228 api.bot_update.ensure_checkout(force=True) 229 api.bot_update.ensure_checkout(force=True)
229 api.gclient.runhooks() 230 api.gclient.runhooks()
230 231
231 232
232 def RunSteps(api): 233 def RunSteps(api):
233 # buildbot sets 'clobber' to the empty string which is falsey, check with 'in' 234 # buildbot sets 'clobber' to the empty string which is falsey, check with 'in'
234 if 'clobber' in api.properties: 235 if 'clobber' in api.properties:
235 api.file.rmcontents('everything', api.path['slave_build']) 236 api.file.rmcontents('everything', api.infra_paths['slave_build'])
236 237
237 GetCheckout(api) 238 GetCheckout(api)
238 239
239 checkout = api.path['checkout'] 240 checkout = api.path['checkout']
240 dart_bin = checkout.join('third_party', 'dart-sdk', 'dart-sdk', 'bin') 241 dart_bin = checkout.join('third_party', 'dart-sdk', 'dart-sdk', 'bin')
241 env = { 'PATH': api.path.pathsep.join((str(dart_bin), '%(PATH)s')) } 242 env = { 'PATH': api.path.pathsep.join((str(dart_bin), '%(PATH)s')) }
242 243
243 # The context adds dart to the path, only needed for the analyze step for now. 244 # The context adds dart to the path, only needed for the analyze step for now.
244 with api.step.context({'env': env}): 245 with api.step.context({'env': env}):
245 AnalyzeDartUI(api) 246 AnalyzeDartUI(api)
(...skipping 11 matching lines...) Expand all
257 GenerateXcodeProject(api) 258 GenerateXcodeProject(api)
258 259
259 260
260 def GenTests(api): 261 def GenTests(api):
261 # A valid commit to flutter/engine, to make the gsutil urls look real. 262 # A valid commit to flutter/engine, to make the gsutil urls look real.
262 for platform in ('mac', 'linux'): 263 for platform in ('mac', 'linux'):
263 yield (api.test(platform) + api.platform(platform, 64) 264 yield (api.test(platform) + api.platform(platform, 64)
264 + api.properties(mastername='client.flutter', 265 + api.properties(mastername='client.flutter',
265 buildername='%s Engine' % platform.capitalize(), 266 buildername='%s Engine' % platform.capitalize(),
266 slavename='fake-m1', clobber='')) 267 slavename='fake-m1', clobber=''))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698