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

Side by Side Diff: build/common.gypi

Issue 56179: Add per-configuration flags that were previously being set by the... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 # .gyp files should set chromium_code to 1 if they build Chromium-specific 7 # .gyp files should set chromium_code to 1 if they build Chromium-specific
8 # code, as opposed to external code. This variable is used to control 8 # code, as opposed to external code. This variable is used to control
9 # such things as the set of warnings to enable, and whether warnings are 9 # such things as the set of warnings to enable, and whether warnings are
10 # treated as errors. 10 # treated as errors.
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 ], 162 ],
163 'scons_propagate_variables': [ 163 'scons_propagate_variables': [
164 'CC', 164 'CC',
165 'CCACHE_DIR', 165 'CCACHE_DIR',
166 'CXX', 166 'CXX',
167 'DISTCC_DIR', 167 'DISTCC_DIR',
168 'DISTCC_HOSTS', 168 'DISTCC_HOSTS',
169 'HOME', 169 'HOME',
170 'LINK', 170 'LINK',
171 ], 171 ],
172 'configurations': {
173 'Debug': {
174 'defines': [
175 '_DEBUG',
176 ],
177 'cflags': [
178 '-O0',
179 '-g',
180 ],
181 },
182 'Release': {
183 'cflags': [
TVL 2009/04/02 15:44:30 drive by: do you need NDEBUG to be defined?
184 '-O2',
185 ],
186 },
187 },
172 }, 188 },
173 }], 189 }],
174 ['OS=="mac"', { 190 ['OS=="mac"', {
175 'target_defaults': { 191 'target_defaults': {
176 'mac_bundle': 0, 192 'mac_bundle': 0,
177 'xcode_settings': { 193 'xcode_settings': {
178 'ALWAYS_SEARCH_USER_PATHS': 'NO', 194 'ALWAYS_SEARCH_USER_PATHS': 'NO',
179 'GCC_C_LANGUAGE_STANDARD': 'c99', 195 'GCC_C_LANGUAGE_STANDARD': 'c99',
180 'GCC_CW_ASM_SYNTAX': 'NO', 196 'GCC_CW_ASM_SYNTAX': 'NO',
181 'GCC_DYNAMIC_NO_PIC': 'YES', 197 'GCC_DYNAMIC_NO_PIC': 'YES',
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
315 # settings in target dicts. SYMROOT is a special case, because many other 331 # settings in target dicts. SYMROOT is a special case, because many other
316 # Xcode variables depend on it, including variables such as 332 # Xcode variables depend on it, including variables such as
317 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 333 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
318 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 334 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
319 # files to appear (when present) in the UI as actual files and not red 335 # files to appear (when present) in the UI as actual files and not red
320 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 336 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
321 # and therefore SYMROOT, needs to be set at the project level. 337 # and therefore SYMROOT, needs to be set at the project level.
322 'SYMROOT': '<(DEPTH)/xcodebuild', 338 'SYMROOT': '<(DEPTH)/xcodebuild',
323 }, 339 },
324 } 340 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698