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

the plugin and test-suite now works with Coq 8.5 #3

Merged
merged 12 commits into from
Nov 26, 2016

Conversation

aa755
Copy link
Contributor

@aa755 aa755 commented Nov 23, 2016

Most of the work here was done by Damien, who got the plugin to compile in Coq 8.5.
I noticed that except Parametricity Module everything else works in Coq8.5pl1.
I accordingly modified the test-suite to avoid using that command, and instead using Parametricity Recursive on the required items in modules.
I also updated the instructions in Readme.md.

I have used this plugin in many other applications, and it worked as expected.

Damien Rouhling and others added 12 commits January 27, 2016 14:00
except the one using Parametricity Module in features.v

The folliwng fails:

Parametricity Module (*Coq.Init.*)Logic.

with "undefined universe Coq.Logic.7".
However, when manually invoking Parametricity for each member of that
module, there was no error.

In another branch, tried to debug the problem with "Parametricity Module".
Printed the stack trace. While adding universe constraing to some local
"evar map" a function in Coq's kernel fails with that error.
It was used to figure out how to reflect inductive types in template-coq
This reverts commit f55c036.
@mlasson
Copy link
Collaborator

mlasson commented Nov 25, 2016

Great job guys !
Thank you very much for doing this.
I'll merge the pull request this weekend.

@mlasson mlasson merged commit 5eba72d into coq-community:master Nov 26, 2016
CohenCyril pushed a commit to CohenCyril/paramcoq that referenced this pull request Jul 28, 2018
Add a "qualified" version for commands Parametricity & Parametricity Recursive.
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

Successfully merging this pull request may close these issues.

2 participants