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

Side by Side Diff: build/common.gypi

Issue 62006: Add 'variants' support on Linux for COVERAGE=, PROFILE= and SYMBOLS=. (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 | « DEPS ('k') | 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 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
216 '-O0', 216 '-O0',
217 '-g', 217 '-g',
218 ], 218 ],
219 }, 219 },
220 'Release': { 220 'Release': {
221 'cflags': [ 221 'cflags': [
222 '-O2', 222 '-O2',
223 ], 223 ],
224 }, 224 },
225 }, 225 },
226 'variants': {
227 'coverage': {
228 'cflags': ['-fprofile-arcs', '-ftest-coverage'],
229 'ldflags': ['-fprofile-arcs'],
230 },
231 'profile': {
232 'cflags': ['-pg', '-g'],
233 'ldflags': ['-pg'],
234 },
235 'symbols': {
236 'cflags': ['-g'],
237 },
238 },
226 }, 239 },
227 }], 240 }],
228 ['OS=="mac"', { 241 ['OS=="mac"', {
229 'target_defaults': { 242 'target_defaults': {
230 'mac_bundle': 0, 243 'mac_bundle': 0,
231 'xcode_settings': { 244 'xcode_settings': {
232 'ALWAYS_SEARCH_USER_PATHS': 'NO', 245 'ALWAYS_SEARCH_USER_PATHS': 'NO',
233 'GCC_C_LANGUAGE_STANDARD': 'c99', 246 'GCC_C_LANGUAGE_STANDARD': 'c99',
234 'GCC_CW_ASM_SYNTAX': 'NO', 247 'GCC_CW_ASM_SYNTAX': 'NO',
235 'GCC_DYNAMIC_NO_PIC': 'YES', 248 'GCC_DYNAMIC_NO_PIC': 'YES',
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
369 # settings in target dicts. SYMROOT is a special case, because many other 382 # settings in target dicts. SYMROOT is a special case, because many other
370 # Xcode variables depend on it, including variables such as 383 # Xcode variables depend on it, including variables such as
371 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 384 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
372 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 385 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
373 # files to appear (when present) in the UI as actual files and not red 386 # files to appear (when present) in the UI as actual files and not red
374 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 387 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
375 # and therefore SYMROOT, needs to be set at the project level. 388 # and therefore SYMROOT, needs to be set at the project level.
376 'SYMROOT': '<(DEPTH)/xcodebuild', 389 'SYMROOT': '<(DEPTH)/xcodebuild',
377 }, 390 },
378 } 391 }
OLDNEW
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698