search

ocurrent / opam-repo-ci

An OCurrent pipeline for testing submissions to opam-repository
Apache License 2.0
20 stars 22 forks source link

Test with ocaml-system #327

Open kit-ty-kate opened 3 months ago

kit-ty-kate commented 3 months ago

Reopening a separate issue after https://github.com/ocurrent/opam-repo-ci/pull/43#issuecomment-2181034907 was closed

this feature is still desirable

shonfeder commented 3 months ago

Thanks! It would help us with prioritization if you could give a nut-shelled account of why we need this in the CI and how it would show up in the CI results. As it is, there are like 6 years worth of discussions across different PRs and issues to try comb through, and afaict none of them give a concise statement of how this work is meant to benefit opam repo's CI, at least not one that is clear to me given my limited context. Thanks in advance!

Something that would really help with prioritization is a remark on the impact of NOT having this feature in place currently.

kit-ty-kate commented 3 months ago

ocaml-system is one of the default OCaml compiler that people will encounter in the wild. People that simply install ocaml and opam through their system package manager will get it.

The limitations of ocaml-system (the installation of OCaml being split between /usr/... and the opamroot) are well known but not tested at all anywhere. So it seems reasonable to test it. The typical issues can in build-system themselves where they look at /usr/... for packages instead of the opamroot, or if the ocaml-system package itself if the substitute for the ocaml binary (which is a script in this context) gets broken.