diff options
| author | Gaëtan Gilbert | 2019-06-19 14:44:29 +0200 |
|---|---|---|
| committer | Gaëtan Gilbert | 2019-06-24 13:14:45 +0200 |
| commit | a3549bef1b4d4498f91e6ad35ae65b48a2fb302f (patch) | |
| tree | 98dcafc5d44bc86e39872189c59094b05a5abb9e /plugins | |
| parent | e7ae7950bb50923e005898d18158593754108725 (diff) | |
[test-suite] Fix printers test
- fix the printers themselves after discharge was moved to the kernel
- change the test to make it more robust
In addition to checking that there is no "Error|Unbound" in the
output, ensure that the "go" function at the end of base_include
is defined. If there are any errors in base_include it won't be defined.
This makes us find out that the test was silently failing in all CI
jobs except trunk+make. It failed to find the "include" file and so
failed with "could not find file include.", which we didn't detect.
To fix this:
- change automatically included paths in Coqinit.init_ocaml_path to be
based on coqlib instead of coqroot. This way top_printers.ml and
base_include can find the compiled ml objects.
- fix for dune: adapt the script to use include_dune when INSIDE_DUNE,
add dependencies to test-suite/dune.
The dependencies should be calculated automatically once Dune has
better support for debug, or we implement proper support for test
printers.
Co-authored-by: Emilio Jesus Gallego Arias <e+git@x80.org>
Diffstat (limited to 'plugins')
0 files changed, 0 insertions, 0 deletions
