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

Issue 1491743002: Add all components to "gn check". (Closed)

Created:
5 years ago by brettw
Modified:
5 years ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@nacl
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add all components to "gn check". All components now pass a "gn check" step. Whitelist with a wildcard. R=dpranke@chromium.org Committed: https://crrev.com/273b751649ebe3cc3579d9c87166ed3802827fb0 Cr-Commit-Position: refs/heads/master@{#362652}

Patch Set 1 #

Patch Set 2 : chromeos #

Patch Set 3 : more chromeos #

Patch Set 4 : more chromeos #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -160 lines) Patch
M .gn View 1 chunk +1 line, -152 lines 0 comments Download
M components/arc/BUILD.gn View 1 1 chunk +1 line, -0 lines 0 comments Download
M components/exo/BUILD.gn View 1 1 chunk +1 line, -0 lines 0 comments Download
M components/exo/wayland/BUILD.gn View 1 2 3 1 chunk +2 lines, -0 lines 0 comments Download
M components/wifi_sync/BUILD.gn View 1 2 3 2 chunks +18 lines, -7 lines 0 comments Download
M ui/ozone/BUILD.gn View 1 2 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 30 (13 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1491743002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1491743002/1
5 years ago (2015-12-02 00:08:53 UTC) #2
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_gn_chromeos_rel/builds/113997)
5 years ago (2015-12-02 01:59:23 UTC) #4
brettw
5 years ago (2015-12-02 02:20:29 UTC) #5
Dirk Pranke
<burns-voice>excellent</burns-voice> lgtm.
5 years ago (2015-12-02 02:27:44 UTC) #6
brettw
chromeos
5 years ago (2015-12-02 05:42:12 UTC) #7
brettw
I had to add some chromeos stuff.
5 years ago (2015-12-02 05:46:47 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1491743002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1491743002/20001
5 years ago (2015-12-02 05:47:06 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_gn_chromeos_rel/builds/114117)
5 years ago (2015-12-02 05:52:39 UTC) #14
brettw
more chromeos
5 years ago (2015-12-02 06:24:31 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1491743002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1491743002/40001
5 years ago (2015-12-02 06:25:24 UTC) #18
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_gn_chromeos_rel/builds/114125)
5 years ago (2015-12-02 06:34:09 UTC) #20
brettw
more chromeos
5 years ago (2015-12-02 06:34:11 UTC) #21
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1491743002/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1491743002/60001
5 years ago (2015-12-02 06:34:57 UTC) #24
commit-bot: I haz the power
Committed patchset #4 (id:60001)
5 years ago (2015-12-02 07:14:51 UTC) #26
commit-bot: I haz the power
Patchset 4 (id:??) landed as https://crrev.com/273b751649ebe3cc3579d9c87166ed3802827fb0 Cr-Commit-Position: refs/heads/master@{#362652}
5 years ago (2015-12-02 07:16:27 UTC) #28
dtu
Failing on perf Linux Builder: https://build.chromium.org/p/chromium.perf/builders/Linux%20Builder/builds/7266 Please fix or revert. The perf builders are Official ...
5 years ago (2015-12-02 15:49:58 UTC) #29
dtu
5 years ago (2015-12-02 16:07:19 UTC) #30
Message was sent while issue was closed.
A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1490313002/ by dtu@chromium.org.

The reason for reverting is: Failing Perf (Official) Linux Builder.
https://build.chromium.org/p/chromium.perf/builders/Linux%20Builder/builds/7266

ERROR at //components/version_info/version_info.cc:12:11: Include not allowed.
#include "ui/base/l10n/l10n_util.h"
          ^-----------------------
It is not in any dependency of
  //components/version_info:version_info
The include file is in the target(s):
  //ui/base:base
which should somehow be reachable.
GN gen failed: 1
.

Powered by Google App Engine
This is Rietveld 408576698