Skip to content

Use a more recent stable version in this CI test - #34

Closed
mbarbin wants to merge 3 commits into
ocaml-dune:mainfrom
mbarbin:upgrade-stable-test-version
Closed

mbarbin wants to merge 3 commits into
ocaml-dune:mainfrom
mbarbin:upgrade-stable-test-version

Conversation

@mbarbin

@mbarbin mbarbin commented Mar 28, 2026

Copy link
Copy Markdown
Contributor

Use a more recent version in the CI test, with compatibility with the dune trace command.

The purpose of this stable version is to exercise the installation when a stable
release is selected. The test can be satisfactory with pretty much any version.

The issue with too old versions is that now we rely on `dune trace` in the test,
which is a fairly recent command. This test is currently failing on the main
branch, this commit bumps the stable version to one that should support the
trace command, with the goal of fixing the CI.

Signed-off-by: Mathieu Barbin <mathieu.barbin@gmail.com>
@mbarbin

mbarbin commented Mar 28, 2026

Copy link
Copy Markdown
Contributor Author

I don't understand what is causing the addition of the selection of the ocaml.5.5.0 in the CI job for the stable version, but this causes a dune pkg lock error.

I think the CI got broken by a background change to the ocaml-compiler packages available in opam, requiring a change in this CI job to force it to choose a lower ocaml version.

@shym I will leave this for your consideration as I am not very familiar with all the tested combinations in this file. I was attempting to fix the CI while working on rebasing the other PR #25 . Thank you for your help.

@shym

shym commented May 28, 2026

Copy link
Copy Markdown
Collaborator

Superseded by the latest commit in main, as I took some time to figure out I had to revert a work-around first. Thanks for the suggestion to do that though!

@shym shym closed this May 28, 2026
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