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

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
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 'CHROMIUM_STRIP_SAVE_FILE': 'app/app_asan.saves',
Mark Mentovai 2011/09/02 13:37:21 This also deserves a comment saying that it replac
Alexander Potapenko 2011/09/02 14:33:42 Done.
1044 },
1045 }],
1041 ], 1046 ],
1042 }, # target helper_app 1047 }, # target helper_app
1043 { 1048 {
1044 # This produces the app mode loader, but not as a bundle. Chromium 1049 # This produces the app mode loader, but not as a bundle. Chromium
1045 # itself is responsible for producing bundles. 1050 # itself is responsible for producing bundles.
1046 'target_name': 'app_mode_app', 1051 'target_name': 'app_mode_app',
1047 'type': 'executable', 1052 'type': 'executable',
1048 'product_name': '<(mac_product_name) App Mode Loader', 1053 'product_name': '<(mac_product_name) App Mode Loader',
1049 'sources': [ 1054 'sources': [
1050 'app/app_mode_loader_mac.mm', 1055 'app/app_mode_loader_mac.mm',
(...skipping 686 matching lines...) Expand 10 before | Expand all | Expand 10 after
1737 'destination': '<(PRODUCT_DIR)', 1742 'destination': '<(PRODUCT_DIR)',
1738 'files': [ 1743 'files': [
1739 '<(INTERMEDIATE_DIR)/repack/chrome.pak' 1744 '<(INTERMEDIATE_DIR)/repack/chrome.pak'
1740 ], 1745 ],
1741 }, 1746 },
1742 ], 1747 ],
1743 }], # targets 1748 }], # targets
1744 }], # OS != "mac" 1749 }], # OS != "mac"
1745 ], # 'conditions' 1750 ], # 'conditions'
1746 } 1751 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698