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

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: 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
Index: chrome/BUILD.gn
diff --git a/chrome/BUILD.gn b/chrome/BUILD.gn
index 7052c70cef4ce3ff5a6850ecbca5b55526e02ed9..f1cc57182b5b0e7e9a62461a9726cc156195b2ca 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")
@@ -425,6 +426,13 @@ if (is_win) {
":chrome_versioned_bundle_data",
"//chrome/common:version_header",
]
+
+ extra_configs = [ "//build/config/mac:dsym" ]
+
+ if (enable_stripping) {
+ ldflags =
+ [ "-Wcrl,strip,-s," + rebase_path("app/app.saves", root_build_dir) ]
+ }
}
compiled_action("chrome_app_strings") {
@@ -567,6 +575,13 @@ if (is_win) {
"-rpath",
"@loader_path/../../../../../../..",
]
+
+ extra_configs = [ "//build/config/mac:dsym" ]
+
+ if (enable_stripping) {
+ ldflags +=
+ [ "-Wcrl,strip,-s," + rebase_path("app/app.saves", root_build_dir) ]
+ }
}
bundle_data("chrome_framework_locales") {
@@ -793,7 +808,11 @@ if (is_win) {
mac_framework_bundle("chrome_framework") {
output_name = chrome_framework_name
- configs += [ "//build/config/compiler:wexit_time_destructors" ]
+ configs += [
+ "//build/config/compiler:wexit_time_destructors",
+ "//build/config/mac:dsym",
+ "//build/config/mac:strip_all",
+ ]
info_plist_target = ":chrome_framework_plist"
extra_substitutions = [
« build/toolchain/mac/linker_driver.py ('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