Thomas Leonard

That was more difficult than expected, but it looks like it's working now: https://ci.appveyor.com/project/talex5-ci/0install-yp593/build/artifacts I don't have a Windows machine to test on - do they work for you?

The AppVeyor console output says: ``` Downloaded C:\cygwin64\packages/http%3a%2f%2fcygwin.mirror.constant.com%2f/noarch/release/mingw64-x86_64-curl/mingw64-x86_64-curl-7.59.0-1.tar.xz ``` So I guess you want http://cygwin.mirror.constant.com/noarch/release/mingw64-x86_64-curl/mingw64-x86_64-curl-7.59.0-1.tar.xz

Unfortunately, I don't have access to a Windows machine any longer. But presumably just asking cygwin to install curl would pull in all the required dlls?

Guess so. Try changing `__details_cb` in `packagekit.py`. Maybe it takes a single dict argument now or something?

As far as I can see, nothing uses `group` or `detail` anyway. If it's working for you, please go ahead and submit a pull request.

> Do you mean I do Eio.Switch.run inside Mirage_crypto_rng_eio.run? I was thinking of something like this: ```ocaml let run ?(sleep = Duration.of_sec 1) env fn = if !running then (...

You need to know when the user's main function returns so that the rng fiber can be stopped too. `Fiber.any` will do this, but a switch won't. There's shouldn't be...

It would be better to mark `Uri.pp` as deprecated; then the compiler can find all the uses for you. Different callers will need different parts redacted, so having a global...

It says: > lru.0.3.0: Requires ocaml >= 4.03.0 & < 5.0

> I'm not particularly a fan of passing first-class modules around, and don't think there's a good reason here. Indeed. I removed it in my branch (see https://github.com/mirage/ocaml-dns/pull/312#pullrequestreview-1088918946). > (but...