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

Side by Side Diff: chrome/chrome.gyp

Issue 7828043: If the 'asan' GYP variable is set to 1 on Mac, do not strip the __asan_init symbol from the binar... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 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 | « chrome/app/app_asan.saves ('k') | chrome/chrome_exe.gypi » ('j') | 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) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 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 'variables': { 5 'variables': {
6 'chromium_code': 1, 6 'chromium_code': 1,
7 7
8 'variables': { 8 'variables': {
9 'version_py_path': 'tools/build/version.py', 9 'version_py_path': 'tools/build/version.py',
10 'version_path': 'VERSION', 10 'version_path': 'VERSION',
(...skipping 1020 matching lines...) Expand 10 before | Expand all | Expand 10 after
1031 'variables': { 1031 'variables': {
1032 # A real .dSYM is needed for dump_syms to operate on. 1032 # A real .dSYM is needed for dump_syms to operate on.
1033 'mac_real_dsym': 1, 1033 'mac_real_dsym': 1,
1034 }, 1034 },
1035 'xcode_settings': { 1035 'xcode_settings': {
1036 # With mac_real_dsym set, strip_from_xcode won't be used. 1036 # With mac_real_dsym set, strip_from_xcode won't be used.
1037 # Specify CHROMIUM_STRIP_SAVE_FILE directly to Xcode. 1037 # Specify CHROMIUM_STRIP_SAVE_FILE directly to Xcode.
1038 'STRIPFLAGS': '-s $(CHROMIUM_STRIP_SAVE_FILE)', 1038 'STRIPFLAGS': '-s $(CHROMIUM_STRIP_SAVE_FILE)',
1039 }, 1039 },
1040 }], 1040 }],
1041 ['asan==1', {
1042 'xcode_settings': {
1043 # Override the outer definition of CHROMIUM_STRIP_SAVE_FILE.
1044 'CHROMIUM_STRIP_SAVE_FILE': 'app/app_asan.saves',
1045 },
1046 }],
1041 ], 1047 ],
1042 }, # target helper_app 1048 }, # target helper_app
1043 { 1049 {
1044 # This produces the app mode loader, but not as a bundle. Chromium 1050 # This produces the app mode loader, but not as a bundle. Chromium
1045 # itself is responsible for producing bundles. 1051 # itself is responsible for producing bundles.
1046 'target_name': 'app_mode_app', 1052 'target_name': 'app_mode_app',
1047 'type': 'executable', 1053 'type': 'executable',
1048 'product_name': '<(mac_product_name) App Mode Loader', 1054 'product_name': '<(mac_product_name) App Mode Loader',
1049 'sources': [ 1055 'sources': [
1050 'app/app_mode_loader_mac.mm', 1056 'app/app_mode_loader_mac.mm',
(...skipping 686 matching lines...) Expand 10 before | Expand all | Expand 10 after
1737 'destination': '<(PRODUCT_DIR)', 1743 'destination': '<(PRODUCT_DIR)',
1738 'files': [ 1744 'files': [
1739 '<(INTERMEDIATE_DIR)/repack/chrome.pak' 1745 '<(INTERMEDIATE_DIR)/repack/chrome.pak'
1740 ], 1746 ],
1741 }, 1747 },
1742 ], 1748 ],
1743 }], # targets 1749 }], # targets
1744 }], # OS != "mac" 1750 }], # OS != "mac"
1745 ], # 'conditions' 1751 ], # 'conditions'
1746 } 1752 }
OLDNEW
« no previous file with comments | « chrome/app/app_asan.saves ('k') | chrome/chrome_exe.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698