-
Notifications
You must be signed in to change notification settings - Fork 27
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
avoid warnings emitted by Coq #76
base: coq-master
Are you sure you want to change the base?
Conversation
Sorry to be late, but now, it seems that there are still some warnings, and moreover, it fails on:
A version problem? |
cf22d16
to
ec2d8bf
Compare
After git rebase, it seems the error has disappeared. |
Only the test-suite fails, and it seems this is only due to re-ordering in the graph. |
What about merging your other PR into this one and updating the oracles? Sorry, but I prefer not to merge a branch where the tests fail... |
I agree on the point concerning not merging a branch where tests fails. |
Some of the changes cannot be fixed by sorting results. It seems there was a symbol for the section variable A that was removed and it shifts the indices of all nodes (this is for the graph.dpd file). |
OK, but if it looks "normal" (I mean, due to coq internal modifications), you can just copy the new results in the oracles, and it will be ok then. |
I agree, I am checking the normalcy of the graph just now. |
Only modifications in the Coq source files.