Skip to content

Don't require a UUID in Mach-O binaries #446

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

winterqt
Copy link
Contributor

@winterqt winterqt commented Oct 14, 2021

Some Mach-O binaries do not contain the LC_UUID command, such as those built with Nix, so this PR removes the requirement for one to exist.

This fixes #443.

@winterqt winterqt changed the title Don't require UUID in Mach-O binaries Don't require a UUID in Mach-O binaries Oct 14, 2021
@alexcrichton
Copy link
Member

Nice find, thanks! Would you be up for adding a test for this as well? (it'd be a macos-specific test that would probably run the test suite with the -no_uuid linker flag)

@winterqt
Copy link
Contributor Author

Would you be up for adding a test for this as well?

Done :)

@alexcrichton
Copy link
Member

Thanks!

@alexcrichton alexcrichton merged commit 7f14f76 into rust-lang:master Oct 15, 2021
@winterqt winterqt deleted the macho-with-no-uuid branch October 15, 2021 21:12
winterqt added a commit to winterqt/rust that referenced this pull request Oct 18, 2021
rust-lang/backtrace-rs#446 allows binaries built
with Nix on macOS to be symbolized.
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 18, 2021
…ulacrum

Bump backtrace

rust-lang/backtrace-rs#446 allows binaries built with Nix on macOS to be symbolized.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

frames cannot be resolved on macOS
2 participants