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.

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

[new release] tls, tls-mirage, tls-lwt, tls-eio and tls-async (0.17.0) #23430

Merged
merged 3 commits into from
Mar 2, 2023

Conversation

hannesm
Copy link
Member

@hannesm hannesm commented Mar 1, 2023

CHANGES:

* tls-async: remove ocaml < 5.0.0 constraint (mirleft/ocaml-tls#471 mirleft/ocaml-tls#474 @hannesm)
* remove dependency on ppx, especially ppx_cstruct and ppx_sexp_conv across
  the libraries (mirleft/ocaml-tls#473 @hannesm, discussion in mirleft/ocaml-tls#472)
@hannesm
Copy link
Member Author

hannesm commented Mar 1, 2023

@hannesm
Copy link
Member Author

hannesm commented Mar 2, 2023

CI failures are unrelated, looks fine to merge to me

@kit-ty-kate
Copy link
Member

tls-eio uses the network and thus can’t be tested on macOS. Could you return the fix upstream?

We’ll merge this asap

@hannesm
Copy link
Member Author

hannesm commented Mar 2, 2023

tls-eio uses the network and thus can’t be tested on macOS. Could you return the fix upstream?

sure, mirleft/ocaml-tls@fdd2f0e

@kit-ty-kate
Copy link
Member

Thanks!

@kit-ty-kate kit-ty-kate merged commit 1694df0 into ocaml:master Mar 2, 2023
@hannesm hannesm deleted the release-tls-v0.17.0 branch March 2, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants