diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-11 23:06:17 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-11 23:06:17 +0000 |
| commit | 51f425785d503b53236c3597fd70bdc7d9b5ed27 (patch) | |
| tree | 91ee53c7e02f0fee2e3ac7071c8e6eba5b5b88bd /build/config | |
| parent | 164aa4ea183b18dbad33c2ab19bca9f1936153a2 (diff) | |
| download | chromium_src-51f425785d503b53236c3597fd70bdc7d9b5ed27.zip chromium_src-51f425785d503b53236c3597fd70bdc7d9b5ed27.tar.gz chromium_src-51f425785d503b53236c3597fd70bdc7d9b5ed27.tar.bz2 | |
GN Windows build fixes
This also removes some patterns from the sources assignment filter, and adds scary comments not to make it bigger. The GYP version of this is out of control so I want to set a clear policy of what is included and not.
I removed X-related stuff from the filter (there are only about 50 files around the tree) and added manual rules for the affected files.
BUG=
R=scottmg@chromium.org
Review URL: https://codereview.chromium.org/235983002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263394 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build/config')
| -rw-r--r-- | build/config/BUILDCONFIG.gn | 21 |
1 files changed, 18 insertions, 3 deletions
diff --git a/build/config/BUILDCONFIG.gn b/build/config/BUILDCONFIG.gn index 350d880..57a3486 100644 --- a/build/config/BUILDCONFIG.gn +++ b/build/config/BUILDCONFIG.gn @@ -200,6 +200,8 @@ if (is_win) { # muct match the pattern (so you need "*.cc" to match all .cc files, for # example). +# DO NOT ADD MORE PATTERNS TO THIS LIST, see set_sources_assignment_filter call +# below. windows_sources_filters = [ "*_win.cc", "*_win.h", @@ -240,8 +242,6 @@ linux_sources_filters = [ "*_linux_unittest.h", "*_linux_unittest.cc", "*\blinux/*", - "*_x11.cc", - "*_x11.h", ] android_sources_filters = [ "*_android.h", @@ -257,6 +257,8 @@ posix_sources_filters = [ "*_posix_unittest.cc", "*\bposix/*", ] +# DO NOT ADD MORE PATTERNS TO THIS LIST, see set_sources_assignment_filter call +# below. # Construct the full list of sources we're using for this platform. sources_assignment_filter = [] @@ -281,7 +283,20 @@ if (!is_android) { sources_assignment_filter += android_sources_filters } -# This is the actual set. +# Actually save this list. +# +# DO NOT ADD MORE PATTERNS TO THIS LIST. +# +# These patterns are executed for every file in the source tree of every run. +# Therefore, adding more patterns slows down the build for everybody. We should +# only add automatic patterns for configurations affecting hundreds of files +# across many projects in the tree. +# +# Therefore, we only add rules to this list corresponding to platforms on the +# Chromium waterfall. This is not for non-officially-supported platforms +# (FreeBSD, etc.) toolkits, (X11, GTK, etc.), or features. For these cases, +# write a conditional in the target to remove the file(s) from the list when +# your platform/toolkit/feature doesn't apply. set_sources_assignment_filter(sources_assignment_filter) # ============================================================================= |