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

Unified Diff: .gn

Issue 936953004: Add more targets to GN check. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move net function Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/BUILD.gn » ('j') | skia/BUILD.gn » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: .gn
diff --git a/.gn b/.gn
index 1dea1071fd74f573449ff0d93e01eeb8c527ba5b..ee8f2471947412126fcebe204aac10685d1e7c32 100644
--- a/.gn
+++ b/.gn
@@ -15,16 +15,63 @@ secondary_source = "//build/secondary/"
# their includes checked for proper dependencies when you run either
# "gn check" or "gn gen --check".
check_targets = [
+ #"//apps/*", # Medium-hard.
+ #"//ash/*", # Medium-hard.
# Requires GN binary >= 314794 and to mark the "...__generate_enum" action in
# build/config/android/rules.gni "java_cpp_enum" template with
# check_includes = false.
#"//base/*",
+ "//blink/*",
+ "//breakpad/*",
"//build/*",
"//cc/*",
+
+ #"//chrome/*", # Epic number of errors.
+ "//chromecast/*",
+ "//chrome_elf/*",
+ "//cloud_print/*",
+
+ #"//components/*", # Lots of errors.
+ #"//content/*", # A whole lot of errors.
+ "//courgette/*",
+ "//crypto/*",
+ "//data/*",
"//dbus/*",
"//device/*",
+
+ #"//extensions/*", # Lots of errors.
+ #"//gin/*", # Easy.
+ #"//google_apis/*", # Easy.
+ "//google_update/*",
+
+ #"//gpu/*", # Lots of errors.
+ #"//ios/*",
"//ipc/*",
+
+ #"//jingle/*",
+ #"//media/*", # Lots of errors.
+ #"//mojo/*",
+ #"//native_client/*",
+ "//net/*",
+
+ #"//pdf/*", # Medium-hard.
+ #"//ppapi/*", # Lots of errors.
+ "//printing/*",
+
+ #"//remoting/*", # Medium-hard.
+ #"//rlz/*", # Needs checking on Windows.
+ #"//sandbox/*", # Medium-hard.
+ "//sdch/*",
+ "//skia/*",
"//sql/*",
"//storage/*",
+ "//sync/*",
+ "//testing/*",
+
+ #"//third_party/*", # May not ever want this.
+ "//tools/*",
+
+ #"//ui/*", # Just a few problems.
"//url/*",
+ "//v8/*",
]
« no previous file with comments | « no previous file | net/BUILD.gn » ('j') | skia/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698