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

Unified Diff: chrome/BUILD.gn

Issue 1999513002: [Mac/GN] Implement dSYM generation and stripping. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Global configuration Created 4 years, 7 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
« build/config/BUILDCONFIG.gn ('K') | « build/toolchain/mac/linker_driver.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/BUILD.gn
diff --git a/chrome/BUILD.gn b/chrome/BUILD.gn
index c66526065a4b1ae97fae471a3351b24c80cb4f0b..575a75f517d3e7c0cff2172a341fa55d25492e47 100644
--- a/chrome/BUILD.gn
+++ b/chrome/BUILD.gn
@@ -21,6 +21,7 @@ if (is_android) {
} else if (is_mac) {
import("//build/compiled_action.gni")
import("//build/config/mac/rules.gni")
+ import("//build/config/mac/symbols.gni")
import("//build/mac/tweak_info_plist.gni")
import("//build/util/branding.gni")
import("//build/util/version.gni")
@@ -426,6 +427,13 @@ if (is_win) {
":chrome_versioned_bundle_data",
"//chrome/common:version_header",
]
+
+ if (enable_stripping) {
Dirk Pranke 2016/05/31 19:58:19 maybe add a comment explaining what this is doing
Robert Sesek 2016/06/01 20:21:05 Done.
+ remove_configs = [ "//build/config/mac:strip_all" ]
+
+ ldflags =
+ [ "-Wcrl,strip,-s," + rebase_path("app/app.saves", root_build_dir) ]
+ }
}
compiled_action("chrome_app_strings") {
@@ -568,6 +576,13 @@ if (is_win) {
"-rpath",
"@loader_path/../../../../../../..",
]
+
+ if (enable_stripping) {
+ remove_configs = [ "//build/config/mac:strip_all" ]
+
+ ldflags +=
+ [ "-Wcrl,strip,-s," + rebase_path("app/app.saves", root_build_dir) ]
+ }
}
bundle_data("chrome_framework_locales") {
« build/config/BUILDCONFIG.gn ('K') | « build/toolchain/mac/linker_driver.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698