ocurrent / solver-service

An OCluster service for solving opam dependencies
Apache License 2.0
11 stars 7 forks source link

Replace usage of `socketpair` for IPC #25

Closed MisterDA closed 1 year ago

MisterDA commented 1 year ago

On Windows, using a socket as standard input file descriptor is daring. Even if it may be possible, Lwt will currently outright reject it. We can, instead of passing a file descriptor to the spawned process, pass the address of the UNIX domain socket as an argument so that the sub process may open a socket itself and connect to that address.

UNIX domain sockets are supported on Windows 10 and OCaml 4.14.

This PR is an other approach than #23, but doesn't work. I think it's because it tries to setup a service but it shouldn't?