Skip to content

Try to fix CI#954

Closed
filipeom wants to merge 1 commit intoOCamlPro:mainfrom
filipeom:attempt-to-fix-ci
Closed

Try to fix CI#954
filipeom wants to merge 1 commit intoOCamlPro:mainfrom
filipeom:attempt-to-fix-ci

Conversation

@filipeom
Copy link
Copy Markdown
Collaborator

@filipeom filipeom commented Apr 9, 2026

Also try to fix opam-based installations

@filipeom filipeom marked this pull request as draft April 9, 2026 16:02
@filipeom
Copy link
Copy Markdown
Collaborator Author

filipeom commented Apr 9, 2026

Ah right, might need to pin a couple of things I think

@filipeom filipeom force-pushed the attempt-to-fix-ci branch from 44b1463 to f2b2d9e Compare April 9, 2026 17:34
@filipeom
Copy link
Copy Markdown
Collaborator Author

If we dropped the constraint on cmdliner >= 2.1.0 to >= 1.3.0 it would build with opam because alt-ergo's latest available opam version conflicts with cmdliner > 2.0.0. However, it wouldn't pass a bunch of tests because cmdliner's version 2 changes the output of the CLI. I tried pinning the latest dev version alt-ergo and alt-ergo-lib because it already supports cmdliner 2, but then we have another problem: it doesn't compile with smtml 😅. I'm giving up, so I'll close this.

Let's hope we get a new version of alt-ergo soon 😅

@filipeom filipeom closed this Apr 10, 2026
@redianthus
Copy link
Copy Markdown
Member

Haha, yes sorry for not telling you, I've talked with the alt-ergo people and there should be a release soon with cmdliner > 2 allowed. :D I'm waiting for it and then the CI should be back.

@filipeom filipeom deleted the attempt-to-fix-ci branch April 10, 2026 10:09
@filipeom
Copy link
Copy Markdown
Collaborator Author

Haha, yes sorry for not telling you, I've talked with the alt-ergo people and there should be a release soon with cmdliner > 2 allowed. :D I'm waiting for it and then the CI should be back.

Ah nice! I think it needs a dolmen release as well right? https://github.com/OCamlPro/alt-ergo/blob/7398dde07d80483f72e85a926d33d2e0de728bec/alt-ergo-lib.opam#L64-L77

@redianthus
Copy link
Copy Markdown
Member

Well, I thinks this is the next branch which needs the dolmen release, whereas they're going to actually to do a minor release which will not need the dolmen release IIRC

@filipeom
Copy link
Copy Markdown
Collaborator Author

Well, I thinks this is the next branch which needs the dolmen release, whereas they're going to actually to do a minor release which will not need the dolmen release IIRC

Ah great! I was asking because smtml also needed to be upgraded if dolmen was bumped

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.

2 participants