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

Add nightly feature #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add nightly feature #2

wants to merge 1 commit into from

Conversation

voidc
Copy link

@voidc voidc commented Aug 2, 2021

This PR adds a nightly feature which uses the recently added PidFd type from the standard library.

@joshtriplett
Copy link
Owner

joshtriplett commented Aug 2, 2021

Thank you very much for taking the time to implement this, and show how it could work with nightly Rust.

I would prefer to avoid maintaining parallel support for nightly Rust, though; I would rather wait for PidFd support in std to be stable. At that point, though, I will definitely switch to it, and drop a substantial amount of code from this crate. (I may use the nightly portion of your PR at that point.)

I'm also hoping that OwnedFd and BorrowedFd become stable around the same time, so that I can stop using RawFd. I may also bump the version to 1.0 at that time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants