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

backport ssrbool to Coq #526

Closed
affeldt-aist opened this issue Jun 8, 2020 · 11 comments
Closed

backport ssrbool to Coq #526

affeldt-aist opened this issue Jun 8, 2020 · 11 comments
Milestone

Comments

@affeldt-aist
Copy link
Member

affeldt-aist commented Jun 8, 2020

Backport to Coq new contents from ssrbool.v (as introduced by PR #513 PR #519 PR #499 ).

https://github.com/affeldt-aist/coq/tree/ssrbool_backport_MathComp1.11
already contains an ssrbool.v file with an integration of PR #513, PR #519, and PR #499
and is ready to be PRed.

@CohenCyril @ybertot

(NB: this issue has been edited, it was mentioning ssrfun.v but its contents have already made their way to Coq)

@affeldt-aist affeldt-aist changed the title backport ssrbool and ssrfun to Coq backport ssrbool to Coq Jun 8, 2020
@affeldt-aist affeldt-aist added this to the 1.12.0 milestone Jun 8, 2020
@chdoc
Copy link
Member

chdoc commented Jun 17, 2020

Note that PR #499, which is hopefully going to be merged soon, also includes additions to ssrbool.v. So you may want to hold off on this for a few more days.

@chdoc
Copy link
Member

chdoc commented Jun 18, 2020

PR #499 has been merged.

@affeldt-aist
Copy link
Member Author

PR #499 has been merged.

Thank you for letting me know. I have updated the tentative backport accordingly.

@chdoc
Copy link
Member

chdoc commented Jun 25, 2020

Maybe the contra lemmas should be integrated with the other contra lemmas already present in that file, i.e., somewhere around this line. Possibly the whole set of lemmas could be wrapped in a section with the appropriate Implicit Types ... command.

@pi8027
Copy link
Member

pi8027 commented Sep 10, 2020

I hope this will be done before the feature freeze of Coq 8.13, which is 15 Nov. (see coq/coq#12334)

@CohenCyril
Copy link
Member

Wasn't this fixed in coq/coq#12898?

@pi8027
Copy link
Member

pi8027 commented Sep 10, 2020

I was not aware of that, but it does not seem to include #552.

@CohenCyril
Copy link
Member

I was not aware of that, but it does not seem to include #552.

I think it doesn't need to, it was a problem specific to ssrbool.v on mathcomp side, @chdoc looked at this more closely than I did, am I right @chdoc?

@chdoc
Copy link
Member

chdoc commented Sep 10, 2020

Yes, the problem was/is that recent versions of ssr.ssrbool.v declare a reserved notation for [rel _ _ : _ | _] (rel with explicit typing) with a particular printing format. Prior to the fix, mathcomp was redeclaring this notation in ssreflect.ssrbool.v. However, one needs to give level information to please old versions of Coq that don't yet provide the reserved notation. This invalidates the preexisting format. Moreover, one cannot provide an equivalent format string at this point, because only parsing notations don't take format strings. Thus, mathcomp needs to repeat the Reserved Notation command for the time being, as this is the only way to declare only parsing notations with a particular format 😁

@pi8027
Copy link
Member

pi8027 commented Sep 10, 2020

I see. So shall we close here?

@affeldt-aist
Copy link
Member Author

I think we can close. I forgot to do so when the backport was merged (coq/coq@40140bf, coq/coq@bfd384e, coq/coq@3405ab8).

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

4 participants