Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[iOS] Debug build to include dSYM for use in Instruments #6529

Closed
wants to merge 1 commit into from

Conversation

halset
Copy link
Contributor

@halset halset commented Sep 30, 2016

I use CocoaPods in my app. When using a pod built with "make ipackage", the Mapbox GL Native symbols did not show up in Instruments. This patch fixes that problem.

@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS build labels Sep 30, 2016
@friedbunny
Copy link
Contributor

Thanks for the PR! This is interesting — I’m not sure why the default Xcode setting in Debug for DEBUG_INFORMATION_FORMAT is DWARF, rather than DWARF + dSYM (as you propose).

Compilation speed, @1ec5?

screen shot 2016-09-30 at 6 10 12 am

@@ -2167,7 +2167,7 @@
CURRENT_COMMIT_HASH = deadbeef;
CURRENT_SEMANTIC_VERSION = 1.0.0;
CURRENT_SHORT_VERSION = 1.0;
DEBUG_INFORMATION_FORMAT = dwarf;
DEBUG_INFORMATION_FORMAT = "dwarf-with-dsym";
Copy link
Contributor

@1ec5 1ec5 Sep 30, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compilation speed is indeed part of the reason that the default is dwarf for Debug builds. I think we should keep it that way for builds within Xcode. A dSYM is unnecessary because the SDK is built with debug symbols.

make ipackage builds a Debug build by default but strips debug symbols (in a post-build step once #6531 lands). Therefore, package.sh should be modified to override this build setting with dwarf-with-dsym if symbols would be stripped (see also #6531).

Please make corresponding changes for the macOS SDK. Thanks!

@halset
Copy link
Contributor Author

halset commented Oct 14, 2016

Closing this. I will just turn on dsym when I need them when playing with Instruments.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants