Skip to content

[CI] Reenable jasmin#21743

Open
eponier wants to merge 2 commits intorocq-prover:masterfrom
eponier:reenable-jasmin-ci
Open

[CI] Reenable jasmin#21743
eponier wants to merge 2 commits intorocq-prover:masterfrom
eponier:reenable-jasmin-ci

Conversation

@eponier
Copy link
Contributor

@eponier eponier commented Mar 11, 2026

Jasmin was removed from CI in #20589 following changes in mathcomp (math-comp/math-comp#1256) that made the extraction to OCaml fail (see jasmin-lang/jasmin#1093 for the discussion).

We spent some time to make Jasmin (jasmin-lang/jasmin#1372) and its dependency coqword (jasmin-lang/coqword#37) avoid referencing the ring structures of mathcomp in the parts of Jasmin that are extracted (which were the reason of the failed extraction). Now that the changes are merged, extraction works again (we tested it with mathcomp 2.5, hopefully it works also with master), and I hope Jasmin can be enabled back in the CI.

@eponier eponier requested a review from a team as a code owner March 11, 2026 15:03
@coqbot-app
Copy link
Contributor

coqbot-app bot commented Mar 11, 2026

I am not triggering a CI run on this PR because the CI configuration has been modified. CI can be triggered manually by an authorized contributor.

@coqbot-app coqbot-app bot added the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Mar 11, 2026
@SkySkimmer
Copy link
Contributor

@coqbot run full ci

@SkySkimmer SkySkimmer self-assigned this Mar 11, 2026
@coqbot-app coqbot-app bot removed the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Mar 11, 2026
@SkySkimmer SkySkimmer added this to the 9.3+rc1 milestone Mar 11, 2026
@SkySkimmer SkySkimmer added the kind: infrastructure CI, build tools, development tools. label Mar 11, 2026
@SkySkimmer
Copy link
Contributor

Doesn't seem to work

File "./lang/ident.v", line 29, characters 21-23:
Error: Syntax error: [def_body] expected after [ident_decl] (in [gallina]).

@SkySkimmer
Copy link
Contributor

Seems like a conflict with #21611 removing of as an anonymous binder syntax outside constructors. Definition id_name & t : string := "". should work and be backwards compatible IIUC

@SkySkimmer SkySkimmer added the needs: overlay This is breaking external developments we track in CI. label Mar 12, 2026
@vbgl
Copy link
Contributor

vbgl commented Mar 13, 2026

Thanks for the hint. I’ll apply the patch. And we’ll discover the next issue…

File "./lang/extraction.v", line 61, characters 0-364:
Error:
Anomaly
"File "kernel/environ.ml", line 929, characters 11-17: Assertion failed."

@SkySkimmer
Copy link
Contributor

some module issue in extraction
cc @ppedrot

eponier and others added 2 commits March 13, 2026 13:18
now that the extraction problems are fixed
@vbgl vbgl force-pushed the reenable-jasmin-ci branch from b21311b to 5dcd833 Compare March 13, 2026 12:31
@coqbot-app
Copy link
Contributor

coqbot-app bot commented Mar 13, 2026

I am not triggering a CI run on this PR because the CI configuration has been modified. CI can be triggered manually by an authorized contributor.

@coqbot-app coqbot-app bot added the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Mar 13, 2026
@vbgl
Copy link
Contributor

vbgl commented Mar 13, 2026

I’ve open an issue regarding the extraction issue: #21754

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: infrastructure CI, build tools, development tools. needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. needs: overlay This is breaking external developments we track in CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants