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

Unified Diff: build/common.gypi

Issue 9288078: build/common.gypi: split out filename exclude rules (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ok Created 8 years, 11 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 | build/filename_rules.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index a4deacae86837f15bcfb9e1323406307ec1a4816..c8dd15771545b2a36170f2c9509f9f64024a3c43 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -1327,65 +1327,17 @@
}],
],
}, {
+ 'includes': [
+ # Rules for excluding e.g. foo_win.cc from the build on non-Windows.
+ 'filename_rules.gypi',
+ ],
# In Chromium code, we define __STDC_FORMAT_MACROS in order to get the
# C99 macros on Mac and Linux.
'defines': [
'__STDC_FORMAT_MACROS',
],
'conditions': [
- ['OS!="win"', {
- 'sources/': [ ['exclude', '_win(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)win/'],
- ['exclude', '(^|/)win_[^/]*\\.(h|cc)$'] ],
- }],
- ['OS!="mac"', {
- 'sources/': [ ['exclude', '_(cocoa|mac)(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)(cocoa|mac)/'],
- ['exclude', '\\.mm?$' ] ],
- }],
- ['use_x11!=1', {
- 'sources/': [
- ['exclude', '_(chromeos|x|x11)(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)x11_[^/]*\\.(h|cc)$'],
- ],
- }],
- ['toolkit_uses_gtk!=1', {
- 'sources/': [
- ['exclude', '_gtk(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)gtk/'],
- ['exclude', '(^|/)gtk_[^/]*\\.(h|cc)$'],
- ],
- }],
- ['OS!="linux" and OS!="openbsd" and OS!="freebsd"', {
- 'sources/': [
- ['exclude', '_xdg(_unittest)?\\.(h|cc)$'],
- ],
- }],
- ['use_wayland!=1', {
- 'sources/': [
- ['exclude', '_(wayland)(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)wayland/'],
- ['exclude', '(^|/)(wayland)_[^/]*\\.(h|cc)$'],
- ],
- }],
- # Do not exclude the linux files on *BSD since most of them can be
- # shared at this point.
- # In case a file is not needed, it is going to be excluded later on.
- ['OS!="linux" and OS!="openbsd" and OS!="freebsd"', {
- 'sources/': [
- ['exclude', '_linux(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)linux/'],
- ],
- }],
- ['OS!="android"', {
- 'sources/': [
- ['exclude', '_android(_unittest)?\\.cc$'],
- ['exclude', '(^|/)android/'],
- ],
- }],
- # We use "POSIX" to refer to all non-Windows operating systems.
['OS=="win"', {
- 'sources/': [ ['exclude', '_posix(_unittest)?\\.(h|cc)$'] ],
# turn on warnings for signed/unsigned mismatch on chromium code.
'msvs_settings': {
'VCCLCompilerTool': {
@@ -1398,23 +1350,6 @@
4251, # class 'std::xx' needs to have dll-interface.
],
}],
- ['chromeos!=1', {
- 'sources/': [ ['exclude', '_chromeos\\.(h|cc)$'] ]
- }],
- ['toolkit_views==0', {
- 'sources/': [ ['exclude', '_views\\.(h|cc)$'] ]
- }],
- ['use_aura==0', {
- 'sources/': [ ['exclude', '_aura(_unittest)?\\.(h|cc)$'],
- ['exclude', '(^|/)aura/'],
- ]
- }],
- ['use_aura==0 or use_x11==0', {
- 'sources/': [ ['exclude', '_aurax11\\.(h|cc)$'] ]
- }],
- ['use_aura==0 or OS!="win"', {
- 'sources/': [ ['exclude', '_aurawin\\.(h|cc)$'] ]
- }],
],
}],
], # target_conditions for 'target_defaults'
« no previous file with comments | « no previous file | build/filename_rules.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698