-
Notifications
You must be signed in to change notification settings - Fork 691
Call guard checking with evar handlers in higher layers #18864
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🔴 CI failure at commit 8b4c8b9 without any failure in the test-suite ✔️ Corresponding job for the base commit fc1b697 succeeded ❔ Ask me to try to extract a minimal test case that can be added to the test-suite 🏃
|
@coqbot ci minimize |
I have initiated minimization at commit 8b4c8b9 for the suggested target ci-equations_test as requested. |
Minimized File /home/runner/work/run-coq-bug-minimizer/run-coq-bug-minimizer/builds/coq/coq-failing/_build_ci/equations/test-suite/issues/issue328.v (from ci-equations_test) (full log on GitHub Actions - verbose log) We are collecting data on the user experience of the Coq Bug Minimizer. 🌟 Minimized Coq File (consider adding this file to the test-suite)(* -*- mode: coq; coq-prog-args: ("-emacs" "-q" "-w" "-deprecated-native-compiler-option" "-native-compiler" "no" "-R" "/github/workspace/builds/coq/coq-failing/_build_ci/equations/test-suite" "TestSuite" "-R" "/github/workspace/builds/coq/coq-failing/_build_ci/equations/theories" "Equations" "-Q" "/github/workspace/cwd" "Top" "-Q" "/github/workspace/builds/coq/coq-failing/_install_ci/lib/coq/user-contrib/Equations" "Equations" "-Q" "/github/workspace/builds/coq/coq-failing/_install_ci/lib/coq/user-contrib/Ltac2" "Ltac2" "-I" "/github/workspace/builds/coq/coq-failing/_build_ci/equations/src" "-top" "TestSuite.issues.issue328") -*- *)
(* File reduced by coq-bug-minimizer from original input, then from 35 lines to 17 lines, then from 30 lines to 275 lines, then from 280 lines to 23 lines, then from 36 lines to 216 lines, then from 220 lines to 28 lines, then from 33 lines to 29 lines *)
(* coqc version 8.20+alpha compiled with OCaml 4.09.0
coqtop version runner-cabngxqim-project-4504-concurrent-1:/builds/coq/coq/_build/default,(HEAD detached at 07d477a472912) (07d477a472912eb0281f963b0ac6b19b899f9d28)
Expected coqc runtime on this file: 0.091 sec *)
Require Coq.extraction.Extraction.
Declare ML Module "equations_plugin:coq-equations.plugin".
Variant equations_tag@{} : Set := the_equations_tag.
Definition fixproto := the_equations_tag.
Register fixproto as equations.fixproto.
Definition block := the_equations_tag.
Register block as equations.internal.block.
Set Implicit Arguments.
Parameter A : Type.
Inductive nonEmpty (A : Type) : Type :=
| singleton : A -> nonEmpty A
| consNE : A -> nonEmpty A -> nonEmpty A.
Equations? fromList (l : list A) : length l > 0 -> nonEmpty A :=
{
fromList nil H := _;
fromList (cons x nil) _ := singleton x;
fromList (cons x (cons y l)) _ := consNE x (fromList (cons y l) _)
}. 🛠️ Intermediate Coq File (useful for debugging if minimization did not go as far as you wanted)🛠️ 📜 Intermediate Coq File log (useful for debugging if minimization did not go as far as you wanted)📜 Build Log (contains the Coq error message) (truncated to last 8.0KiB; full 2.7MiB file on GitHub Actions Artifacts under
|
8b4c8b9
to
354c1f4
Compare
IDK what made that equations test start failing, AFAICT this PR shouldn't change what happens since equations called search_guard without evar handler. The overlay to make it work again looks more correct than the previous code so not spending much time on figuring it out. |
@coqbot run full ci |
@coqbot merge now |
@ppedrot: Please take care of the following overlays:
|
Adapt to rocq-prover/rocq#18864 (search_guard can handle evars)
Fix #18358
Fix #18680
Fix #18850
Overlays: