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

Unified Diff: build/common.gypi

Issue 1085843004: mac: Add the flag "-gline-tables-only" to reduce dSYM size. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments from thakis. Created 5 years, 8 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 7de617cba6d6b09dfc39e1ed423ddc202c750857..2db290d2d95c53f44d4b6a47972e8d63847064c6 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -5130,6 +5130,17 @@
# specified or not.
'-fno-strict-aliasing', # See http://crbug.com/32204.
],
+ 'conditions': [
+ ['branding=="Chrome" and buildtype=="Official"', {
+ 'OTHER_CFLAGS': [
erikchen 2015/04/23 01:20:36 I don't understand GYP. I figured that this would
Nico 2015/04/23 01:26:24 gyp merges lists by appending them. This is docume
erikchen 2015/04/23 01:57:49 I see. It was unclear to me that lists got merged
+ # The Google Chrome Framework dSYM generated by dsymutil has grown
+ # larger than 4GB, which dsymutil can't handle. Reduce the amount
+ # of debug symbols.
+ '-gline-tables-only', # See http://crbug.com/479841
+ ]
+ }],
+ ],
+
},
'target_conditions': [
['_type=="executable"', {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698