Skip to content

restoring compilation in Rocq 9.4alpha - #256

Merged
nmvdw merged 11 commits into
UniMath:masterfrom
rmatthes:restorecompilationforRocq94alpha
Sep 3, 2026
Merged

restoring compilation in Rocq 9.4alpha#256
nmvdw merged 11 commits into
UniMath:masterfrom
rmatthes:restorecompilationforRocq94alpha

Conversation

@rmatthes

@rmatthes rmatthes commented Sep 2, 2026

Copy link
Copy Markdown
Member

the essential problem is that "is" is now a keyword, the solution in sync with UniMath PR2083

but there is a type-checking error in Initiality/SyntacticCategory.v as well, which is not dealt with in the first commit, however, it is solved in 2720547 , but the solution is partly ugly.

…in sync with UniMath PR2083

but there is a type-checking error in Initiality/SyntacticCategory.v as well, which is not dealt with in the present commit
@rmatthes rmatthes closed this Sep 2, 2026
@rmatthes rmatthes reopened this Sep 2, 2026
@rmatthes rmatthes changed the title towards restoring compilation in Rocq 9.4alpha restoring compilation in Rocq 9.4alpha Sep 3, 2026
@rmatthes
rmatthes requested a review from nmvdw September 3, 2026 10:41
@nmvdw
nmvdw merged commit 08e9709 into UniMath:master Sep 3, 2026
2 checks passed
@nmvdw

nmvdw commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Thanks for the fix! The solution in SyntacticCategory.v is fine, I think. Perhaps some arguments can be replaced by _ to make this step more readable, but I do not know whether the unifier will be able to deduce these arguments.

@rmatthes

rmatthes commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Thank you @nmvdw for having looked through it. Ideally, the author of this formalization (@peterlefanulumsdaine) could try to figure out what went wrong before this fix, maybe a coercion that is no longer applied by the system. However, there is currently strictly no need to make an effort towards reestablishing the previous elegant proof scripts. Only one place in my fix is ugly. It can easily wait until 9.4alpha has become an official release and then the working environment.

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