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

Unified Diff: build/toolchain/mac/BUILD.gn

Issue 2731313004: GN: Generate appropriate .dSYMs for .frameworks and .apps (Closed)
Patch Set: Modify linker_driver.py to create .dSYMs with correct names. Created 3 years, 9 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 | « build/config/mac/rules.gni ('k') | build/toolchain/mac/linker_driver.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/toolchain/mac/BUILD.gn
diff --git a/build/toolchain/mac/BUILD.gn b/build/toolchain/mac/BUILD.gn
index b417e76a3b0b9d985f1442503b7b9fd93d6f2e0f..e5783ed98657ef4509f8c5190a6ff3f3288adae9 100644
--- a/build/toolchain/mac/BUILD.gn
+++ b/build/toolchain/mac/BUILD.gn
@@ -172,9 +172,13 @@ template("mac_toolchain") {
# If dSYMs are enabled, this flag will be added to the link tools.
if (_enable_dsyms) {
- dsym_switch = " -Wcrl,dsym,{{root_out_dir}} "
- dsym_output_dir =
- "{{root_out_dir}}/{{target_output_name}}{{output_extension}}.dSYM"
+ if (defined(invoker.dsym_name)) {
themblsha 2017/03/17 15:50:58 This doesn't actually work, as it seems that I nee
Robert Sesek 2017/03/17 16:59:36 I think this is because tool()s are actually speci
+ dsym_name = invoker.dsym_name
+ } else {
+ dsym_name = "{{target_output_name}}{{output_extension}}.dSYM"
+ }
+ dsym_switch = " -Wcrl,dsym,\{{root_out_dir}},\"$dsym_name\" "
+ dsym_output_dir = "{{root_out_dir}}/$dsym_name"
dsym_output = [
"$dsym_output_dir/",
"$dsym_output_dir/Contents/Info.plist",
« no previous file with comments | « build/config/mac/rules.gni ('k') | build/toolchain/mac/linker_driver.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698