-
Notifications
You must be signed in to change notification settings - Fork 654
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
tests should protect concurrent access to signal handlers #579
Conversation
@homu retry |
@asomers I switched the PR branch to |
@Susurrus why don't we just wait until after you've merged new-ci/master? Then I'll rebase. I don't want to hold up that work. |
@asomers Sure. |
Adds a mutex to protect access to SIGUSR2 signal handlers by the AIO tests. Fixes nix-rust#578
Well, it didn't fix the mips errors, but I still think that this is the right thing to do. Does anybody object to adding the dependency on lazy_static ? |
Nope, that should be fine. Doesn't really drag in anything too much extra. |
bors r+ |
Build succeeded |
Adds a mutex to protect access to SIGUSR2 signal handlers by the AIO
tests.
Fixes #578