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

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

Issue 1917243002: Revert "build: roll infra_paths changes" (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 unified diff | Download patch
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',
11 'file', 10 'file',
12 'gsutil', 11 'gsutil',
13 'recipe_engine/path', 12 'recipe_engine/path',
14 'recipe_engine/platform', 13 'recipe_engine/platform',
15 'recipe_engine/properties', 14 'recipe_engine/properties',
16 'recipe_engine/step', 15 'recipe_engine/step',
17 'zip', 16 'zip',
18 ] 17 ]
19 18
20 BUCKET_NAME = 'flutter_infra' 19 BUCKET_NAME = 'flutter_infra'
(...skipping 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 api.gclient.c.got_revision_mapping['src'] = 'got_engine_revision' 225 api.gclient.c.got_revision_mapping['src'] = 'got_engine_revision'
227 # TODO(eseidel): According to iannucci force=True is required. 226 # TODO(eseidel): According to iannucci force=True is required.
228 # See https://codereview.chromium.org/1690713003#msg6 227 # See https://codereview.chromium.org/1690713003#msg6
229 api.bot_update.ensure_checkout(force=True) 228 api.bot_update.ensure_checkout(force=True)
230 api.gclient.runhooks() 229 api.gclient.runhooks()
231 230
232 231
233 def RunSteps(api): 232 def RunSteps(api):
234 # buildbot sets 'clobber' to the empty string which is falsey, check with 'in' 233 # buildbot sets 'clobber' to the empty string which is falsey, check with 'in'
235 if 'clobber' in api.properties: 234 if 'clobber' in api.properties:
236 api.file.rmcontents('everything', api.infra_paths['slave_build']) 235 api.file.rmcontents('everything', api.path['slave_build'])
237 236
238 GetCheckout(api) 237 GetCheckout(api)
239 238
240 checkout = api.path['checkout'] 239 checkout = api.path['checkout']
241 dart_bin = checkout.join('third_party', 'dart-sdk', 'dart-sdk', 'bin') 240 dart_bin = checkout.join('third_party', 'dart-sdk', 'dart-sdk', 'bin')
242 env = { 'PATH': api.path.pathsep.join((str(dart_bin), '%(PATH)s')) } 241 env = { 'PATH': api.path.pathsep.join((str(dart_bin), '%(PATH)s')) }
243 242
244 # The context adds dart to the path, only needed for the analyze step for now. 243 # The context adds dart to the path, only needed for the analyze step for now.
245 with api.step.context({'env': env}): 244 with api.step.context({'env': env}):
246 AnalyzeDartUI(api) 245 AnalyzeDartUI(api)
(...skipping 11 matching lines...) Expand all
258 GenerateXcodeProject(api) 257 GenerateXcodeProject(api)
259 258
260 259
261 def GenTests(api): 260 def GenTests(api):
262 # A valid commit to flutter/engine, to make the gsutil urls look real. 261 # A valid commit to flutter/engine, to make the gsutil urls look real.
263 for platform in ('mac', 'linux'): 262 for platform in ('mac', 'linux'):
264 yield (api.test(platform) + api.platform(platform, 64) 263 yield (api.test(platform) + api.platform(platform, 64)
265 + api.properties(mastername='client.flutter', 264 + api.properties(mastername='client.flutter',
266 buildername='%s Engine' % platform.capitalize(), 265 buildername='%s Engine' % platform.capitalize(),
267 slavename='fake-m1', clobber='')) 266 slavename='fake-m1', clobber=''))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698