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

Side by Side Diff: build/common.gypi

Issue 11490019: Only build with _FORTIFY_SOURCE in Release mode. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years 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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 # IMPORTANT: 5 # IMPORTANT:
6 # Please don't directly include this file if you are building via gyp_chromium, 6 # Please don't directly include this file if you are building via gyp_chromium,
7 # since gyp_chromium is automatically forcing its inclusion. 7 # since gyp_chromium is automatically forcing its inclusion.
8 { 8 {
9 # Variables expected to be overriden on the GYP command line (-D) or by 9 # Variables expected to be overriden on the GYP command line (-D) or by
10 # ~/.gyp/include.gypi. 10 # ~/.gyp/include.gypi.
(...skipping 2248 matching lines...) Expand 10 before | Expand all | Expand 10 after
2259 ], 2259 ],
2260 }, { 2260 }, {
2261 'defines': [ 2261 'defines': [
2262 'DYNAMIC_ANNOTATIONS_ENABLED=1', 2262 'DYNAMIC_ANNOTATIONS_ENABLED=1',
2263 'WTF_USE_DYNAMIC_ANNOTATIONS=1', 2263 'WTF_USE_DYNAMIC_ANNOTATIONS=1',
2264 ], 2264 ],
2265 }], 2265 }],
2266 ['win_use_allocator_shim==0', { 2266 ['win_use_allocator_shim==0', {
2267 'defines': ['NO_TCMALLOC'], 2267 'defines': ['NO_TCMALLOC'],
2268 }], 2268 }],
2269 ['os_posix==1 and chromium_code==1', {
2270 # Non-chromium code is not guaranteed to compile cleanly
2271 # with _FORTIFY_SOURCE. Also, fortified build may fail
2272 # when optimizations are disabled, so only do that for Release
2273 # build.
2274 'defines': [
2275 '_FORTIFY_SOURCE=2',
2276 ],
2277 }],
2269 ['OS=="linux" or OS=="android"', { 2278 ['OS=="linux" or OS=="android"', {
2270 'target_conditions': [ 2279 'target_conditions': [
2271 ['_toolset=="target"', { 2280 ['_toolset=="target"', {
2272 'cflags': [ 2281 'cflags': [
2273 '<@(release_extra_cflags)', 2282 '<@(release_extra_cflags)',
2274 ], 2283 ],
2275 }], 2284 }],
2276 ], 2285 ],
2277 }], 2286 }],
2278 ], 2287 ],
(...skipping 26 matching lines...) Expand all
2305 'cflags': [ 2314 'cflags': [
2306 # TODO(phajdan.jr): Use -fstack-protector-strong when our gcc 2315 # TODO(phajdan.jr): Use -fstack-protector-strong when our gcc
2307 # supports it. 2316 # supports it.
2308 '-fstack-protector', 2317 '-fstack-protector',
2309 '--param=ssp-buffer-size=4', 2318 '--param=ssp-buffer-size=4',
2310 ], 2319 ],
2311 'ldflags': [ 2320 'ldflags': [
2312 '-Wl,-z,now', 2321 '-Wl,-z,now',
2313 '-Wl,-z,relro', 2322 '-Wl,-z,relro',
2314 ], 2323 ],
2315 'conditions': [
2316 ['chromium_code==1', {
2317 # Non-chromium code is not guaranteed to compile cleanly
2318 # with _FORTIFY_SOURCE.
2319 'defines': [
2320 '_FORTIFY_SOURCE=2',
2321 ],
2322 }],
2323 ],
2324 }, 2324 },
2325 }], 2325 }],
2326 ['os_posix==1 and OS!="mac" and OS!="ios"', { 2326 ['os_posix==1 and OS!="mac" and OS!="ios"', {
2327 'target_defaults': { 2327 'target_defaults': {
2328 # Enable -Werror by default, but put it in a variable so it can 2328 # Enable -Werror by default, but put it in a variable so it can
2329 # be disabled in ~/.gyp/include.gypi on the valgrind builders. 2329 # be disabled in ~/.gyp/include.gypi on the valgrind builders.
2330 'variables': { 2330 'variables': {
2331 'werror%': '-Werror', 2331 'werror%': '-Werror',
2332 'libraries_for_target%': '', 2332 'libraries_for_target%': '',
2333 }, 2333 },
(...skipping 1586 matching lines...) Expand 10 before | Expand all | Expand 10 after
3920 # settings in target dicts. SYMROOT is a special case, because many other 3920 # settings in target dicts. SYMROOT is a special case, because many other
3921 # Xcode variables depend on it, including variables such as 3921 # Xcode variables depend on it, including variables such as
3922 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 3922 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
3923 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 3923 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
3924 # files to appear (when present) in the UI as actual files and not red 3924 # files to appear (when present) in the UI as actual files and not red
3925 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 3925 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
3926 # and therefore SYMROOT, needs to be set at the project level. 3926 # and therefore SYMROOT, needs to be set at the project level.
3927 'SYMROOT': '<(DEPTH)/xcodebuild', 3927 'SYMROOT': '<(DEPTH)/xcodebuild',
3928 }, 3928 },
3929 } 3929 }
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