Skip to content

Bump the recommended OCaml version to 4.14.2 (master branch)#251

Merged
DmxLarchey merged 3 commits into
uds-psl:masterfrom
meithecatte:push-nknnwootrlxu
Jun 17, 2026
Merged

Bump the recommended OCaml version to 4.14.2 (master branch)#251
DmxLarchey merged 3 commits into
uds-psl:masterfrom
meithecatte:push-nknnwootrlxu

Conversation

@meithecatte

Copy link
Copy Markdown
Contributor

#250 ported to the master branch, as requested.

The 4.14.2 patch release incorporates ocaml/ocaml#12577, which makes the
toolchain compile without errors on new versions of GCC. This is
already necessary to be able to set up the library on bleeding-edge
systems, such as Arch Linux, and the requirement will spread as the GCC
update trickles down across distributions.

The CI is already set up to use the latest available OCaml release from
the 4.14 branch.
@mrhaandi

Copy link
Copy Markdown
Collaborator

Probably not related to the current PR, but there is a compile error:

  ROCQ compile FOL/TRAKHTENBROT/enumerable.v
  File "./FOL/TRAKHTENBROT/enumerable.v", line 133, characters 70-72:
  Error: Error: No such hypothesis: Hm

now get erased when "destruct" as invoked on them
@DmxLarchey

Copy link
Copy Markdown
Collaborator

@mrhaandi

The compile error was due to a change in how destruct behaves on Section variables. Before, it was not removed but now it is.

There are other compile errors

ROCQ compile HOU/concon/conservativity_consequences.v
File "./HOU/firstorder.v", line 999, characters 70-99:
Warning: In term, tolerating this expression at a higher level than expected
by the notation continuing on the right (which is not left-associative). This
tolerance will be eventually removed. Insert parentheses or try to lower the
level at which the top symbol of this expression is parsed.
[level-tolerance,deprecated-since-9.2,deprecated,parsing,default]
File "./HOU/firstorder.v", line 1028, characters 15-31:
Error: decr_lambda_free depends on the section variable free but free in the
current context does not refer to the section variable of the same name.

possibly more annoying ...

@DmxLarchey

Copy link
Copy Markdown
Collaborator

@mrhaandi

Btw there are tons of warnings about rewrite being renamed as rw ... I do not know why such "cleanup" ideas pop up but they mess with this code base on a very large scale.

@meithecatte

Copy link
Copy Markdown
Contributor Author

FWIW, the rewrite -> rw thing seems to be specifically about ssreflect's rewrite tactic – cf. rocq-prover/rocq#21478

@DmxLarchey

Copy link
Copy Markdown
Collaborator

Oh yes indeed, this to not apply to Ltac rewrite, thank god ;-)

@meithecatte

Copy link
Copy Markdown
Contributor Author

I've pushed a fix for the error, the branch now compiles with Rocq-dev on my side.

@DmxLarchey

Copy link
Copy Markdown
Collaborator

@mrhaandi I am ok for a merge. Do you agree?

@mrhaandi

Copy link
Copy Markdown
Collaborator

@mrhaandi I am ok for a merge. Do you agree?

I agree.

@DmxLarchey DmxLarchey merged commit c290e6a into uds-psl:master Jun 17, 2026
1 check passed
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.

3 participants