-
Notifications
You must be signed in to change notification settings - Fork 89
Add support for WinIO to process. #177
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
|
/cc @bgamari (I don't know AndreasK github handle). |
AndreasPK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will take a closer look soon.
3320a25 to
d73130a
Compare
|
Arg... looks like |
d73130a to
3731a2f
Compare
4645ca5 to
76b157e
Compare
|
Looks like I have to drop the test using the new manager explicitly. I will push changes for |
bgamari
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good to me
4711f0e to
9dac4b3
Compare
9dac4b3 to
54a004c
Compare
54a004c to
b790744
Compare
|
This should be the final commit. Everything passes with it. |
bgamari
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Thanks @Mistuke!
This adds support for the new Asynchronous I/O manager in GHC for Windows[1] to process.
This is required because process uses internal GHC APIs.
Do not merge yet, however goal for this is for GHC 8.12. It is ready for review.
[1] https://gitlab.haskell.org/ghc/ghc/-/merge_requests/1224