Skip to content

Do not link with tls.lwt on windows#267

Merged
samoht merged 2 commits into
mirage:masterfrom
samoht:windows
Aug 6, 2018
Merged

Do not link with tls.lwt on windows#267
samoht merged 2 commits into
mirage:masterfrom
samoht:windows

Conversation

@samoht
Copy link
Copy Markdown
Member

@samoht samoht commented Jul 16, 2018

Fix #264

@samoht
Copy link
Copy Markdown
Member Author

samoht commented Jul 16, 2018

note: I haven't tested the PR at all yet

@dinosaure
Copy link
Copy Markdown
Member

async_ssl.0.11.0 does not compile.

@samoht samoht merged commit f8dbdf2 into mirage:master Aug 6, 2018
@samoht samoht deleted the windows branch August 6, 2018 09:49
samoht added a commit to samoht/opam-repository that referenced this pull request Sep 3, 2018
CHANGES:

* Correct depopt for conduit-lwt-unix (mirage/ocaml-conduit#260, @dra27)
* async: provide all `Async_ssl` options at config (mirage/ocaml-conduit#263, @vbmithr)
* async: add a V2 module for a new versioned API (mirage/ocaml-conduit#265, @rgrinberg)
* lwt-unix: do not link with tls.lwt on windows (mirage/ocaml-conduit#267, @samoht)
samoht added a commit to samoht/opam-repository that referenced this pull request Sep 3, 2018
CHANGES:

* Correct depopt for conduit-lwt-unix (mirage/ocaml-conduit#260, @dra27)
* async: provide all `Async_ssl` options at config (mirage/ocaml-conduit#263, @vbmithr)
* async: add a V2 module for a new versioned API (mirage/ocaml-conduit#265, @rgrinberg)
* lwt-unix: do not link with tls.lwt on windows (mirage/ocaml-conduit#267, @samoht)
samoht added a commit to samoht/opam-repository that referenced this pull request Sep 4, 2018
CHANGES:

* Correct depopt for conduit-lwt-unix (mirage/ocaml-conduit#260, @dra27)
* async: provide all `Async_ssl` options at config (mirage/ocaml-conduit#263, @vbmithr)
* async: add a V2 module for a new versioned API (mirage/ocaml-conduit#265, @rgrinberg)
* lwt-unix: do not link with tls.lwt on windows (mirage/ocaml-conduit#267, @samoht)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants