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 support for anonymous pipes #82

Closed
notgull opened this issue Feb 1, 2023 · 3 comments
Closed

Add support for anonymous pipes #82

notgull opened this issue Feb 1, 2023 · 3 comments

Comments

@notgull
Copy link
Member

notgull commented Feb 1, 2023

It would be useful if this crate supported Windows anonymous pipes.

I probably can't make any progress on this until #76 is complete, as this would require the backend to not be wepoll.

@kchibisov
Copy link

Given that your patch merged, how far are we from anon pipes? Asking since rust time bombed mio 0.6.x (it has future incompatible code) and we must find an alternative.

@notgull
Copy link
Member Author

notgull commented Mar 11, 2023

We still need to support direct OVERLAPPED operations in our use case. However, looking at your use case, it looks like you use [mio-anonymous-pipes], which "cheats" by putting the pipe onto a thread pool. Therefore, you might be able to get away with just wrapping the pipe in an Unblock and using it like that.

@notgull
Copy link
Member Author

notgull commented Aug 10, 2023

Not as much of a priority anymore as we've worked around this limitation in alacritty. This is mostly a duplicate of #97 now, as we would need overlapped operation support to truly support anonymous pipes in polling.

@notgull notgull closed this as not planned Won't fix, can't repro, duplicate, stale Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants