diff options
| author | dpranke <dpranke@chromium.org> | 2015-09-18 12:07:00 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-18 19:07:49 +0000 |
| commit | b218d91ef159033438a8b79fc0ca8da7abdc2b8e (patch) | |
| tree | 45e814c3b88e0ad57c11ba1178fb46364164819c /.gn | |
| parent | 31b2721a44330fb51fe59c2c6628e2a9593a384b (diff) | |
| download | chromium_src-b218d91ef159033438a8b79fc0ca8da7abdc2b8e.zip chromium_src-b218d91ef159033438a8b79fc0ca8da7abdc2b8e.tar.gz chromium_src-b218d91ef159033438a8b79fc0ca8da7abdc2b8e.tar.bz2 | |
Move `gn check` into the `generate_build_files` step.
The dedicated GN bots run a script test step called `gn_check`
that (unsurprisingly) runs `gn check` to validate that we
haven't introduced any bad build dependencies.
When we enabled GN on the regular Linux bots, we forgot to add
that step. It turns out that we can't easily add it, because
the Linux bots use a builder/tester split, and the script would
run on the tester, where we don't have a build directory and
don't know what GN args to check against.
This patch changes the MB/generate_build_files step to add the
--check flag to the normal `gn gen` invocation. This slows
down the generate_build_files step, but the time would be spent
somewhere anyway when we ran the gn check step, later.
This patch also removes the explicit gn_check test step from
all of the bots. This patch will have the side effect of starting
to run `gn check` on *every* GN configuration (GN check was only
being run on some of the configurations before).
R=jam@chromium.org, brettw@chromium.org
BUG=532230
CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg;tryserver.chromium.win:win8_chromium_gn_dbg,win_chromium_gn_x64_rel
Review URL: https://codereview.chromium.org/1342293002
Cr-Commit-Position: refs/heads/master@{#349731}
Diffstat (limited to '.gn')
| -rw-r--r-- | .gn | 7 |
1 files changed, 5 insertions, 2 deletions
@@ -24,7 +24,7 @@ check_targets = [ "//cc/*", #"//chrome/*", # Epic number of errors. - "//chrome/installer/*", + #"//chrome/installer/*", # A few errors, need to verify the size after fixing. "//chromecast/*", # TODO(brettw): Fix http://crbug.com/460828 and uncomment the following @@ -38,7 +38,10 @@ check_targets = [ "//components/bookmarks/*", "//components/google/*", "//components/history/*", - "//components/html_viewer/*", + + # TODO(dpranke): Reenable once v8 exports v8_base as a public_dep + #"//components/html_viewer/*", + "//components/mus/*", "//components/omnibox/*", "//components/os_crypt/*", |