search

mirage / ocaml-solo5

Freestanding OCaml runtime
Other
101 stars 30 forks source link

Travis, Cirrus CI: Update for OCaml 4.11 #83

Closed mato closed 3 years ago

mato commented 4 years ago

For Travis, test common archs and targets for the latest OCaml release, and only amd64/hvt for previous releases.

/cc @kit-ty-kate

mato commented 4 years ago

@hannesm Feel free to add OCAML_VERSION:4.11.0 for FreeBSD as part of this if one exists, I've not looked.

hannesm commented 4 years ago

@mato I pushed a commit to your branch which tests OCaml 4.10.1 and 4.11.0 on cirrus & FreeBSD. On that CI, the OCaml compiler is always built from scratch (no caching / custom images), thus it's just a matter to add an additional task with a different OCAML_VERSION

mato commented 4 years ago

@hannes Thanks, I wasn't sure if it was just a matter of asking for the newer OCaml version.

I'm not sure what's up with the Travis failures, it looks like the ocaml-ci-scripts don't yet have proper support for 4.11?

hannesm commented 4 years ago

@mato travis failure looks to me like the ocaml/opam2:alpine images in use here are not (yet) updated with 4.11. I've no idea how they're generated and published, maybe @avsm or @kit-ty-kate know that?

ocaml-ci-scripts got updated with 4.11 support in https://github.com/ocaml/ocaml-ci-scripts/commit/f966a453f1ba54a5f0d39d3288585d7f23398e35

mato commented 3 years ago

Ok, I've rebased this PR to contain only the Travis and Cirrus CI changes (OPAM metadata changes have been merged elsewhere).

Once OCaml 4.11 actually works in CI this can be merged.

hannesm commented 3 years ago

ok I force-pushed now that #89 is merged. in spite of ocaml/opam now being slim docker images (see https://github.com/ocaml/ocaml-ci-scripts/pull/347#discussion_r520044006) this may work (though there may be the ocaml/opam vs ocaml/opam2 mismatch -- let's see how travis behaves)

hannesm commented 3 years ago

:tada: this is green now, merging :)

hannesm commented 3 years ago

CI succeeded here, but failed on master --> I commented on https://github.com/ocaml/ocaml-ci-scripts/pull/349#issuecomment-727236389 about that issue (and will wait for answers there before adjusting the .travis.yml in this repository).