Skip to content
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

Don't use mmap on macOS #302

Merged
merged 11 commits into from
Mar 10, 2020
Merged

Conversation

Aaron1011
Copy link
Member

@Aaron1011 Aaron1011 commented Mar 9, 2020

This was previously disabled in rust-lang/rust#45866, but was lost in #298.

@alexcrichton
Copy link
Member

I'm not sure why, but tests look like they're failing?

@Aaron1011
Copy link
Member Author

@alexcrichton: I've finally gotten the tests to pass. It was necessary to create a separate (non-workspace) crate for the new test, since deleting the dSYM ended up influencing the output of other tests run from the same directory.

This should be ready to merge.

@alexcrichton alexcrichton merged commit 594ffa6 into rust-lang:master Mar 10, 2020
@Aaron1011 Aaron1011 deleted the fix/osx-mmap branch March 10, 2020 21:02
# 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.

2 participants