Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

macOS: save dSYM files #2162

Merged
merged 1 commit into from
Aug 29, 2024
Merged

macOS: save dSYM files #2162

merged 1 commit into from
Aug 29, 2024

Conversation

mitchellh
Copy link
Contributor

The focus of this commit is to store the dSYM files associated with official macOS builds. dSYM files allow us to map crash reports to source.

The dSYM files are primarily uploaded to our official blob storage where all releases are also stored. We also upload the dSYM files to Sentry since I'm experimenting with using that for crash reproting (note: manual crash reporting, no automatic network traffic).

This commit also changes our blob URLs for releases to use the full Git SHA rather than a build number. This is much easier to trace back.

The focus of this commit is to store the dSYM files associated with
official macOS builds. dSYM files allow us to map crash reports to
source.

The dSYM files are primarily uploaded to our official blob storage where
all releases are also stored. We also upload the dSYM files to Sentry
since I'm experimenting with using that for crash reproting (note:
manual crash reporting, no automatic network traffic).

This commit also changes our blob URLs for releases to use the full
Git SHA rather than a build number. This is much easier to trace back.
@mattrobenolt
Copy link
Collaborator

Hell yeah brother.

@mitchellh mitchellh merged commit fcb8b04 into main Aug 29, 2024
38 checks passed
@mitchellh mitchellh deleted the dsym branch August 29, 2024 04:42
@mitchellh mitchellh mentioned this pull request Aug 31, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants