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

Unified Diff: build/common.gypi

Issue 788193003: Create a build_config header file. (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: fixes for mseaborn Created 5 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 | « SConstruct ('k') | src/include/build_config.h » ('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 4c6289fa4ebc6331ec19a2675f2ec74cb9ff0e8d..2664dd996d8c0d7e840cb1e780f46bd5ad32e79a 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -297,6 +297,7 @@
'-pthread',
'-fno-exceptions',
'-Wall', # TODO(bradnelson): why does this disappear?!?
+ '-include', 'native_client/src/include/build_config.h'
],
'conditions': [
['nacl_standalone==1 and OS=="linux"', {
@@ -305,13 +306,10 @@
['OS=="android"', {
'target_conditions': [
['_toolset=="target"', {
- 'defines': ['NACL_ANDROID=1'],
- }, {
- 'defines': ['NACL_ANDROID=0'],
+ 'defines': ['ANDROID'],
}],
],
}, {
- 'defines': ['NACL_ANDROID=0'],
'link_settings': {
'libraries': [
'-lrt',
@@ -449,9 +447,6 @@
'-Wl,-z,noexecstack',
],
'defines': [
- 'NACL_LINUX=1',
- 'NACL_OSX=0',
- 'NACL_WINDOWS=0',
'_DEFAULT_SOURCE=1',
'_BSD_SOURCE=1',
'_POSIX_C_SOURCE=199506',
@@ -533,6 +528,9 @@
'USE_HEADERMAP': 'NO',
# TODO(bradnelson): -Werror ?!?
'WARNING_CFLAGS': ['-Wall', '-Wendif-labels', '-Wno-long-long'],
+ 'OTHER_CFLAGS': [
+ '-include', 'native_client/src/include/build_config.h'
+ ],
'conditions': [
['chromium_mac_pch', {'GCC_PRECOMPILE_PREFIX_HEADER': 'YES'},
{'GCC_PRECOMPILE_PREFIX_HEADER': 'NO'}],
@@ -643,12 +641,6 @@
],
}],
],
- 'defines': [
- 'NACL_LINUX=0',
- 'NACL_ANDROID=0',
- 'NACL_OSX=1',
- 'NACL_WINDOWS=0',
- ],
},
}],
['OS=="win"', {
@@ -703,11 +695,6 @@
'_SECURE_ATL',
'__STDC_LIMIT_MACROS=1',
'_HAS_EXCEPTIONS=0',
-
- 'NACL_LINUX=0',
- 'NACL_ANDROID=0',
- 'NACL_OSX=0',
- 'NACL_WINDOWS=1'
],
'conditions': [
['MSVS_VERSION=="2008"', {
@@ -730,6 +717,9 @@
'WarnAsError': 'true',
'DebugInformationFormat': '3',
'ExceptionHandling': '0',
+ 'AdditionalOptions': [
+ '/FI', 'native_client/src/include/build_config.h'
+ ],
},
'VCLibrarianTool': {
'AdditionalOptions': ['/ignore:4221'],
« no previous file with comments | « SConstruct ('k') | src/include/build_config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698