Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.Sign up
Replace use of select() by poll(). Fixes #798. #799
Since we have no explicit control over FD numbers,
This commit uses
`select()` only works with FD numbers <= 1023. Since we have no explicit control over FD numbers, `select()` can fail when there are many files open. This commit uses `poll()` instead, which doesn't suffer from this restriction. `poll()` doesn't work on Windows, but that's not a supported platform for Nixops anyway.
Thank you for this PR.
In the past several months, some major changes have taken place in
This is all accumulating in to what I hope will be a NixOps 2.0
My hope is that by adding types and more thorough automated testing,
However, because of the major changes, it has become likely that this
If you would like to see this merge, please bring it up to date with
Thank you again for the work you've done here, I am sorry to be