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.

Sign up for GitHub

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

Jump to bottom

abstract tactic doesn't work when the top-level proof is sort-polymorphic #19072

Closed
jesboat opened this issue May 22, 2024 · 0 comments · Fixed by #19073
Closed

abstract tactic doesn't work when the top-level proof is sort-polymorphic #19072

jesboat opened this issue May 22, 2024 · 0 comments · Fixed by #19073
Labels
kind: bug An error, flaw, fault or unintended behaviour. part: sort polymorphism The sorts subsystem of the universe system.
Milestone
8.20+rc1

Comments

@jesboat
Copy link

jesboat commented May 22, 2024 •
edited by SkySkimmer

Description of the problem

Polymorphic Definition demo@{srt| |} : True.
Proof.
  abstract exact I.
  (* Quality srt is unbound. *)

Version of Coq where this bug occurs

8.19.0

Last version of Coq where the bug did not occur

No response
@jesboat jesboat added kind: bug An error, flaw, fault or unintended behaviour. needs: triage The validity of this issue needs to be checked, or the issue itself updated. labels May 22, 2024
@SkySkimmer SkySkimmer added part: sort polymorphism The sorts subsystem of the universe system. and removed needs: triage The validity of this issue needs to be checked, or the issue itself updated. labels May 22, 2024
SkySkimmer added a commit to SkySkimmer/coq that referenced this issue May 22, 2024
@SkySkimmer
UState: don't check for unbound qualities as it doesn't make sense
7d74a5b 
due to collapsing qualities any quality we see must be named

Fix coq#19072

(we still have a check in the functions used by rewrite rules, not
sure if it's correct there but probably doesn't matter)
@SkySkimmer SkySkimmer mentioned this issue May 22, 2024
Merged
SkySkimmer added a commit to SkySkimmer/coq that referenced this issue May 27, 2024
@SkySkimmer
Univ declarations can have extensible qualities (internally)
90e31e2 
Not supported in syntax as due to collapsing qualities any quality we
see must be named, but side effects use default_univ_decl which must
allow the named qualities from the surrounding proof.

Fix coq#19072
@coqbot-app coqbot-app bot closed this as completed in 435896b Jun 4, 2024
@coqbot-app coqbot-app bot added this to the 8.20+rc1 milestone Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind: bug An error, flaw, fault or unintended behaviour. part: sort polymorphism The sorts subsystem of the universe system.
Projects
None yet
Milestone
8.20+rc1
Development

Successfully merging a pull request may close this issue.

Univ declarations can have extensible qualities (internally) SkySkimmer/coq
2 participants
@jesboat @SkySkimmer