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

Side by Side Diff: scripts/slave/recipes/chromium_gn.py

Issue 211473006: Add GN (the build tool) to the chromium.linux buildbots and try jobs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: update w/ review feedback Created 6 years, 8 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
(Empty)
1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 DEPS = [
6 'chromium',
7 'gclient',
8 'platform',
9 'properties',
10 ]
11
12
13 def GenSteps(api):
14 api.gclient.set_config('chromium')
15 yield api.gclient.checkout(revert=True)
16
17 # TODO(dpranke): The build_config should probably be a property passed
18 # in from slaves.cfg, but that doesn't exist today, so we need a lookup
19 # mechanism to map bot name to build_config.
20 build_config = {
21 'Linux GN (dbg)': 'Debug',
22 }[api.properties.get('buildername')]
23
24 api.chromium.set_config('chromium', BUILD_CONFIG=build_config)
25 yield api.chromium.runhooks(run_gyp=False)
26
27 yield api.chromium.run_gn('//out/' + build_config)
iannucci 2014/03/28 22:40:42 I think that if this must be not-really-a-path, th
Dirk Pranke 2014/03/28 23:52:06 I think once we can get rid of the need to specify
28
29 yield api.chromium.compile_with_ninja('compile', api.chromium.output_dir)
30
31 # TODO(dpranke): crbug.com/353854. Run gn_unittests and other tests
32 # when they are also being run as part of the try jobs.
33
34
35 def GenTests(api):
36 yield (
37 api.test('unittest_basic') +
38 api.properties.generic(buildername='Linux GN (dbg)') +
39 api.platform.name('linux')
40 )
41
42 # TODO(dpranke): This test should actually produce the same result
43 # as the previous test, but it specifically matches what is run
44 # on the "Linux GN (dbg)" bot. We should have one 'simulation' test
45 # for each bot config. Ideally this should live somewhere else
46 # closer to the master tests.
47 yield (
48 api.test('full_chromium_linux_Linux_GN__dbg_') +
49 api.properties.generic(buildername='Linux GN (dbg)') +
50 api.platform.name('linux')
51 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698