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

Fix CI #337

Merged
merged 51 commits into from
Sep 3, 2024
Merged

Fix CI #337

merged 51 commits into from
Sep 3, 2024

Conversation

maxRN
Copy link
Contributor

@maxRN maxRN commented Aug 21, 2024

This PR fixes the pipeline to enable running tests again on Ubuntu and MacOS on OCaml verions 4.14.x and 5.0.x.

Windows builds require some more special attention. There were some errors during compilation of the ssl dependency. Logs can be found here: https://github.com/maxRN/dream/actions/runs/10508356669/job/29112078780

The Mirage tests are also failing, because of some dependency issues which I didn't have time to investigate further. Logs for those can be found here: https://github.com/maxRN/dream/actions/runs/10521553562/job/29152441822 (the other, non-mirage test failures can be ignored - I was just testing something in this run)

@maxRN maxRN marked this pull request as ready for review August 23, 2024 07:42
@aantron
Copy link
Owner

aantron commented Sep 3, 2024

The issue with package ssl on Windows is caused somewhere upstream, by either ssl, setup-ocaml, or elsewhere. I've opened savonet/ocaml-ssl#155 and savonet/ocaml-ssl#156 about that. For now, I'll try pinning ssl to an older version or disabling the Windows build, both regrettable options.

@aantron
Copy link
Owner

aantron commented Sep 3, 2024

Mirage build looks like it needs robur-coop/ocaml-letsencrypt#34, which is awaiting release of ocaml/opam-repository#26454. Probably better to just ping and wait for that.

@aantron aantron changed the title ci: run tests + quickstart *nix systems on 4.14.x and 5.0.x Fix CI Sep 3, 2024
@aantron aantron merged commit d3cb68b into aantron:master Sep 3, 2024
8 checks passed
@aantron
Copy link
Owner

aantron commented Sep 3, 2024

Amazing. Thank you!

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