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

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: make recipe actually work Created 6 years, 9 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
18 # TODO(dpranke): The build_config should probably be a property passed
19 # in from slaves.cfg, but that doesn't exist today, so we need a lookup
20 # mechanism to map bot name to build_config.
21 build_config = {
22 'Linux GN (dbg)': 'Debug',
23 }[api.properties.get('buildername')]
iannucci 2014/03/27 01:33:57 yeah, we need a real mechanism for this. Bug filed
24
25 api.chromium.set_config('chromium', BUILD_CONFIG=build_config)
26 yield api.chromium.runhooks(run_gyp=False)
27
28 # TODO(dpranke): GN doesn't support absolute paths for the -o arg
29 # so we have to pass the path relative to the checkout root.
30 yield api.chromium.run_gn('out/' + build_config)
iannucci 2014/03/27 01:33:57 Oh, that's actually worse than I thought :D. I gue
31
32 yield api.chromium.compile_with_ninja('compile', api.chromium.output_dir)
33
34 # TODO(dpranke): crbug.com/353854. Run gn_unittests and other tests
35 # when they are also being run as part of the try jobs.
36
37
38 def GenTests(api):
39 yield (
40 api.test('unittest_basic') +
41 api.properties.generic(buildername='Linux GN (dbg)') +
42 api.platform.name('linux')
43 )
44
45 # TODO(dpranke): This test should actually produce the same result
46 # as the previous test, but it specifically matches what is run
47 # on the "Linux GN (dbg)" bot. We should have one 'simulation' test
48 # for each bot config. Ideally this should live somewhere else
49 # closer to the master tests.
50 yield (
51 api.test('full_chromium_linux_Linux_GN__dbg_') +
52 api.properties.generic(buildername='Linux GN (dbg)') +
53 api.platform.name('linux')
54 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698