Skip to content

[ci] Test Coq 8.18#9348

Closed
ejgallego wants to merge 1 commit intoocaml:mainfrom
ejgallego:coq_818
Closed

[ci] Test Coq 8.18#9348
ejgallego wants to merge 1 commit intoocaml:mainfrom
ejgallego:coq_818

Conversation

@ejgallego
Copy link
Copy Markdown
Collaborator

This is an experiment to see if we should test in CI a lower and an upper bound for Coq.

Note that starting with Coq 8.17, Coq has been split into coq-core and coq-stdlib, with coq-stdlib being the heavy package.

Unfortunately we still need to install coq-stdlib for our tests, as we need to check that stdlib / installed theories are working properly, but we could at some point just have a very light coq-prelude package that would take 2 seconds to build and would work for us too.

This is an experiment to see if we should test in CI a lower and an
upper bound for Coq.

Note that starting with Coq 8.17, Coq has been split into `coq-core`
and `coq-stdlib`, with `coq-stdlib` being the heavy package.

Unfortunately we still need to install `coq-stdlib` for our tests, as
we need to check that stdlib / installed theories are working
properly, but we could at some point just have a very light
`coq-prelude` package that would take 2 seconds to build and would
work for us too.

Signed-off-by: Emilio Jesus Gallego Arias <e+git@x80.org>
@ejgallego ejgallego added the coq label Dec 1, 2023
@Alizter
Copy link
Copy Markdown
Collaborator

Alizter commented Dec 1, 2023

Good opportunity to make the output of the tests a bit more version independent.

@Alizter
Copy link
Copy Markdown
Collaborator

Alizter commented Dec 1, 2023

Regarding this:

+  *** Warning: ltac_plugin.cmxs already found in /home/runner/work/dune/dune/_opam/lib/coq-core/plugins/ltac (discarding /home/runner/work/dune/dune/_opam/lib/coq/../coq-core/plugins/ltac/ltac_plugin.cmxs)
+
         coqdep thy2/.thy2.theory.d
+  *** Warning: ltac_plugin.cmxs already found in /home/runner/work/dune/dune/_opam/lib/coq-core/plugins/ltac (discarding /home/runner/work/dune/dune/_opam/lib/coq/../coq-core/plugins/ltac/ltac_plugin.cmxs)
+

It appears that we have a bug in dune since we must be passing both:

-I /home/runner/work/dune/dune/_opam/lib/coq/../coq-core/plugins/ltac/
-I /home/runner/work/dune/dune/_opam/lib/coq-core/plugins/ltac

to coqdep causing it to discard the first. Arguably the fact that coqdep doesn't recognize these as equivalent is it's own issue, but we could probably stop feeding it duplicate arguments in dune.

@ejgallego ejgallego closed this Jul 17, 2025
@ejgallego ejgallego deleted the coq_818 branch July 17, 2025 19:42
@ejgallego
Copy link
Copy Markdown
Collaborator Author

Superseeded by #12035

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants