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.

Sign up for GitHub

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

Jump to bottom

Added basic windows support #46

Closed
wants to merge 4 commits into from
Closed

Added basic windows support #46

wants to merge 4 commits into from

Conversation

mitsuhiko
Copy link
Owner

@mitsuhiko mitsuhiko commented May 26, 2023 •
edited
Loading

Fixes #20

It seems like the timeout support of ipc-channel is buggy which is why this is disabled for now.

@mitsuhiko mitsuhiko closed this May 26, 2023
@mitsuhiko
Copy link
Owner Author

Unfortunately ipc-channel in the current form cannot be used yet. I am running against all kinds of internal assertions.

thread 'procspawn_test_helper' panicked at 'Windows IPC channel received handles intended for pid 68076, but this is pid 65584. This likely happened because a receiver was transferred while it had outstanding data that contained a channel or shared memory in its pipe. This isn't supported in the Windows implementation.', C:\Users\armin\.cargo\registry\src\github.com-1ecc6299db9ec823\ipc-channel-0.16.0\src\platform\windows\mod.rs:163:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Windows Support
1 participant
@mitsuhiko