-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Add {get,set}rlimit and getrusage to libc #23457
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
Conversation
r? @huonw (rust_highfive has picked a reviewer for you, use r? to override) |
5412b51
to
5055fbc
Compare
I have no idea about how stable these are across platforms. Are the |
from
The values and structs are included here from corresponding OS’s header files (except for OS X/iOS |
Ok, apparently this |
This looks good to me (it will be used in an upcoming patch of yours, right?). Feel free to ping once you've gone through the final update. We should always be able to fix these in the future as well, so tweaks here and there should be fine. |
5055fbc
to
2d701e6
Compare
At least a part of it, yes. I decided that if I’m binding a function in liblibc, I could as well just bind the related functions as well. Moved rusage struct to bsd43. |
In general this crate is currently a bit of a dumping ground and I'm not sure how well the organization has been maintained over time. We generally are not adding many new additions to it but instead binding them locally at (e.g. @bors: r+ |
📌 Commit 2d701e6 has been approved by |
…chton Hopefully didn’t miss or mess up anything. ~~EDIT: ah, as usual, just didn’t bother running build before pushing a submit request button. Build pending.~~
Hopefully didn’t miss or mess up anything.
EDIT: ah, as usual, just didn’t bother running build before pushing a submit request button. Build pending.