Skip to content

fix: module name for dylib sym files#158

Merged
bobbyg603 merged 5 commits intomainfrom
dylib-dsym-fix
May 17, 2025
Merged

fix: module name for dylib sym files#158
bobbyg603 merged 5 commits intomainfrom
dylib-dsym-fix

Conversation

@bobbyg603
Copy link
Copy Markdown
Member

@bobbyg603 bobbyg603 commented May 16, 2025

Description

The minidump-stackwalk executable from rust-minidump looks for liba.dylib/GUID/liba.dylib.sym and we were storing it as liba/GUID/liba.dylib.sym.

Checklist

  • Tested manually
  • Unit tests pass with no errors or warnings
  • Documentation updated (if applicable)
  • Reviewed by at least 1 other contributor

@bobbyg603 bobbyg603 requested a review from daveplunkett May 16, 2025 23:52
Comment thread .github/workflows/test.yaml Fixed
Comment thread .github/workflows/test.yaml Fixed
@bobbyg603 bobbyg603 merged commit 7e9bdb9 into main May 17, 2025
7 checks passed
@bobbyg603 bobbyg603 deleted the dylib-dsym-fix branch May 17, 2025 00:17
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.

3 participants