Skip to content
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

Failing test from elpi 1.18.1 to coq-elpi 1.19.3 #559

Closed
SnarkBoojum opened this issue Dec 6, 2023 · 2 comments
Closed

Failing test from elpi 1.18.1 to coq-elpi 1.19.3 #559

SnarkBoojum opened this issue Dec 6, 2023 · 2 comments

Comments

@SnarkBoojum
Copy link

While checking if the newer elpi worked with the next Debian packages (transition to coq 8.18 and MC 2 is still stuck with mathcomp-abel and mathcomp-analysis), I got the following error on coq-elpi:

test link order
--- tests/test_link_order_import3.ref   2023-10-12 14:06:10.000000000 +0000
+++ tests/test_link_order_import3.txt   2023-12-06 10:41:04.054791445 +0000
@@ -116,6 +116,8 @@
 std.split-at _ _ _ _ :- std.fatal-error split-at run out of list items.
 std.fold [] A0 _ A0.
 std.fold [A0 | A1] A2 A3 A4 :- A3 A0 A2 A5 , std.fold A1 A5 A3 A4.
+std.fold-right [] A0 _ A0.
+std.fold-right [A0 | A1] A2 A3 A4 :- std.fold-right A1 A2 A3 A5 , A3 A0 A5 A4.
 std.fold2 [] [_ | _] _ _ _ :-
  std.fatal-error fold2 on lists of different length.
 std.fold2 [_ | _] [] _ _ _ :-
@gares
Copy link
Contributor

gares commented Dec 6, 2023

That is fine, the list of builtins also come from elpi.
I guess you have a more recent elpi than what is currently required. Maybe you are on elpi 8.18? I plan to make it the requirement for coq-elpi 2.0, to be released soon.

@SnarkBoojum
Copy link
Author

I was checking elpi’s latest: 8.18... I hadn’t checked the opam file for the depend versions - sorry for the noise

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

No branches or pull requests

2 participants