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

Side by Side Diff: chrome/chrome.gyp

Issue 8824003: Breakpad: Compile Breakpad into Chromium by default on Mac (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Split out mac_breakpad_symbols too Created 9 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
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 757 matching lines...) Expand 10 before | Expand all | Expand 10 after
768 'install_name_tool', 768 'install_name_tool',
769 '-change', 769 '-change',
770 '@executable_path/../Versions/<(version_full)/<(mac_product_name ) Framework.framework/<(mac_product_name) Framework', 770 '@executable_path/../Versions/<(version_full)/<(mac_product_name ) Framework.framework/<(mac_product_name) Framework',
771 '@executable_path/../../../<(mac_product_name) Framework.framewo rk/<(mac_product_name) Framework', 771 '@executable_path/../../../<(mac_product_name) Framework.framewo rk/<(mac_product_name) Framework',
772 '${BUILT_PRODUCTS_DIR}/${EXECUTABLE_PATH}' 772 '${BUILT_PRODUCTS_DIR}/${EXECUTABLE_PATH}'
773 ], 773 ],
774 }, 774 },
775 { 775 {
776 # Modify the Info.plist as needed. The script explains why this 776 # Modify the Info.plist as needed. The script explains why this
777 # is needed. This is also done in the chrome and chrome_dll 777 # is needed. This is also done in the chrome and chrome_dll
778 # targets. In this case, -b0, -k0, and -s0 are used because 778 # targets. In this case, --breakpad=0, -k0, and -s0 are used
779 # Breakpad, Keystone, and Subersion keys are never placed into 779 # because Breakpad, Keystone, and Subersion keys are never
780 # the helper. 780 # placed into the helper.
781 'postbuild_name': 'Tweak Info.plist', 781 'postbuild_name': 'Tweak Info.plist',
782 'action': ['<(tweak_info_plist_path)', 782 'action': ['<(tweak_info_plist_path)',
783 '-b0', 783 '--breakpad=0',
784 '-k0', 784 '-k0',
785 '-s0', 785 '-s0',
786 '<(branding)', 786 '<(branding)',
787 '<(mac_bundle_id)'], 787 '<(mac_bundle_id)'],
788 }, 788 },
789 { 789 {
790 # Make sure there isn't any Objective-C in the helper app's 790 # Make sure there isn't any Objective-C in the helper app's
791 # executable. 791 # executable.
792 'postbuild_name': 'Verify No Objective-C', 792 'postbuild_name': 'Verify No Objective-C',
793 'action': [ 793 'action': [
794 'tools/build/mac/verify_no_objc.sh', 794 'tools/build/mac/verify_no_objc.sh',
795 ], 795 ],
796 }, 796 },
797 ], 797 ],
798 'conditions': [ 798 'conditions': [
799 ['mac_breakpad==1', { 799 ['mac_breakpad_symbols==1', {
800 'variables': { 800 'variables': {
801 # A real .dSYM is needed for dump_syms to operate on. 801 # A real .dSYM is needed for dump_syms to operate on.
802 'mac_real_dsym': 1, 802 'mac_real_dsym': 1,
803 }, 803 },
804 'xcode_settings': { 804 'xcode_settings': {
805 # With mac_real_dsym set, strip_from_xcode won't be used. 805 # With mac_real_dsym set, strip_from_xcode won't be used.
806 # Specify CHROMIUM_STRIP_SAVE_FILE directly to Xcode. 806 # Specify CHROMIUM_STRIP_SAVE_FILE directly to Xcode.
807 'STRIPFLAGS': '-s $(CHROMIUM_STRIP_SAVE_FILE)', 807 'STRIPFLAGS': '-s $(CHROMIUM_STRIP_SAVE_FILE)',
808 }, 808 },
809 }], 809 }],
(...skipping 492 matching lines...) Expand 10 before | Expand all | Expand 10 after
1302 'configurations': { 1302 'configurations': {
1303 'Common_Base': { 1303 'Common_Base': {
1304 'msvs_target_platform': 'x64', 1304 'msvs_target_platform': 'x64',
1305 }, 1305 },
1306 }, 1306 },
1307 }, 1307 },
1308 ]}, # 'targets' 1308 ]}, # 'targets'
1309 ], # OS=="win" 1309 ], # OS=="win"
1310 ], # 'conditions' 1310 ], # 'conditions'
1311 } 1311 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698