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

Side by Side Diff: content/content_shell.gypi

Issue 854753002: linux/gyp: Find missing symbols at build time, not runtime. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ozone Created 5 years, 11 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
« no previous file with comments | « content/content_browser.gypi ('k') | gpu/gpu_config.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) 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 { 5 {
6 'variables': { 6 'variables': {
7 'content_shell_product_name': 'Content Shell', 7 'content_shell_product_name': 'Content Shell',
8 # The "19" is so that sites that sniff for version think that this is 8 # The "19" is so that sites that sniff for version think that this is
9 # something reasonably current; the "77.34.5" is a hint that this isn't a 9 # something reasonably current; the "77.34.5" is a hint that this isn't a
10 # standard Chrome. 10 # standard Chrome.
(...skipping 714 matching lines...) Expand 10 before | Expand all | Expand 10 after
725 'xcode_settings': { 725 'xcode_settings': {
726 'GCC_SYMBOLS_PRIVATE_EXTERN': 'NO', 726 'GCC_SYMBOLS_PRIVATE_EXTERN': 'NO',
727 'INFOPLIST_FILE': 'shell/tools/plugin/mac/Info.plist', 727 'INFOPLIST_FILE': 'shell/tools/plugin/mac/Info.plist',
728 }, 728 },
729 }], 729 }],
730 ['os_posix == 1 and OS != "mac"', { 730 ['os_posix == 1 and OS != "mac"', {
731 'cflags': [ 731 'cflags': [
732 '-fvisibility=default', 732 '-fvisibility=default',
733 ], 733 ],
734 }], 734 }],
735 ['use_x11 == 1', {
736 'dependencies': [ '../build/linux/system.gyp:x11' ],
737 }],
735 ['OS=="win"', { 738 ['OS=="win"', {
736 'defines': [ 739 'defines': [
737 # This seems like a hack, but this is what Safari Win does. 740 # This seems like a hack, but this is what Safari Win does.
738 'snprintf=_snprintf', 741 'snprintf=_snprintf',
739 ], 742 ],
740 'sources': [ 743 'sources': [
741 'shell/tools/plugin/win/TestNetscapePlugin.def', 744 'shell/tools/plugin/win/TestNetscapePlugin.def',
742 'shell/tools/plugin/win/TestNetscapePlugin.rc', 745 'shell/tools/plugin/win/TestNetscapePlugin.rc',
743 ], 746 ],
744 # The .rc file requires that the name of the dll is np_test_netscape_p lugin.dll. 747 # The .rc file requires that the name of the dll is np_test_netscape_p lugin.dll.
(...skipping 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
1163 '--input_symbol', '<(PRODUCT_DIR)/content_shell.exe.pdb', 1166 '--input_symbol', '<(PRODUCT_DIR)/content_shell.exe.pdb',
1164 '--destination_dir', '<(dest_dir)', 1167 '--destination_dir', '<(dest_dir)',
1165 ], 1168 ],
1166 }, 1169 },
1167 ], 1170 ],
1168 }, 1171 },
1169 ], 1172 ],
1170 }], # OS=="win" and fastbuild==0 and target_arch=="ia32" 1173 }], # OS=="win" and fastbuild==0 and target_arch=="ia32"
1171 ] 1174 ]
1172 } 1175 }
OLDNEW
« no previous file with comments | « content/content_browser.gypi ('k') | gpu/gpu_config.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698