[build] Enable gn check for most of v8
Currently there are 32 gn check failures across v8 (not including third_party). These are due to issues in cppgc and inspector, which are not trivially resolveable, so for now 4 targets (+ third_party) are being excluded from the checks. Bug: v8:7330 Change-Id: I506abe8e452c26239262499fc1fe2a6347a4c84b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851886 Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#74255}
This commit is contained in:
parent
fee82d7fee
commit
dbd5ae5296
8
.gn
8
.gn
@ -11,7 +11,13 @@ buildconfig = "//build/config/BUILDCONFIG.gn"
|
|||||||
# matching these patterns (see "gn help label_pattern" for format) will have
|
# matching these patterns (see "gn help label_pattern" for format) will have
|
||||||
# their includes checked for proper dependencies when you run either
|
# their includes checked for proper dependencies when you run either
|
||||||
# "gn check" or "gn gen --check".
|
# "gn check" or "gn gen --check".
|
||||||
check_targets = []
|
no_check_targets = [
|
||||||
|
"//:cppgc_base",
|
||||||
|
"//:v8_internal_headers",
|
||||||
|
"//src/inspector:inspector",
|
||||||
|
"//test/cctest:cctest_sources",
|
||||||
|
"//third_party/*",
|
||||||
|
]
|
||||||
|
|
||||||
# These are the list of GN files that run exec_script. This whitelist exists
|
# These are the list of GN files that run exec_script. This whitelist exists
|
||||||
# to force additional review for new uses of exec_script, which is strongly
|
# to force additional review for new uses of exec_script, which is strongly
|
||||||
|
Loading…
Reference in New Issue
Block a user