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

Unified Diff: chrome/chrome_dll.gypi

Issue 11968015: Workaround for pdb size being exceeded for Debug non-shared_library builds (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/empty_pdb_workaround.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_dll.gypi
diff --git a/chrome/chrome_dll.gypi b/chrome/chrome_dll.gypi
index 169ac1e64771bd972bd8fa9042474138304a7efa..65b70969d82a478cb8547a0aec1e05439442c59a 100644
--- a/chrome/chrome_dll.gypi
+++ b/chrome/chrome_dll.gypi
@@ -68,6 +68,21 @@
]
},
{
+ # This target is only depended upon on Windows.
+ 'target_name': 'chrome_dll_pdb_workaround',
+ 'type': 'static_library',
+ 'sources': [ 'empty_pdb_workaround.cc' ],
+ 'msvs_settings': {
+ 'VCCLCompilerTool': {
+ # This *in the compile phase* must match the pdb name that's
+ # output by the final link. See empty_pdb_workaround.cc for
+ # more details.
+ 'DebugInformationFormat': '3',
+ 'ProgramDataBaseFileName': '<(PRODUCT_DIR)/chrome.dll.pdb',
+ },
+ },
+ },
+ {
'target_name': 'chrome_main_dll',
'type': 'shared_library',
'variables': {
@@ -93,6 +108,7 @@
'dependencies': [
# On Windows, link the dependencies (libraries) that make
# up actual Chromium functionality into this .dll.
+ 'chrome_dll_pdb_workaround',
'chrome_resources.gyp:chrome_resources',
'chrome_version_resources',
'../chrome/chrome_resources.gyp:chrome_unscaled_resources',
« no previous file with comments | « no previous file | chrome/empty_pdb_workaround.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698