math-classes icon indicating copy to clipboard operation
math-classes copied to clipboard

Fix for a new rapply tactic

Open JasonGross opened this issue 6 years ago • 2 comments

This backwards compatible change makes math-classes work with coq/coq#10760 by replacing all instances of rapply which were relying on typeclass resolution happening before refine (instead of simultaneously with it) to instead invoke Tactics.rapply (which is the same tactic in Coq <= 8.10, and which will be the tactic rather than the uconstr-taking tactic notation in Coq >= 8.11).

See also https://github.com/coq/coq/pull/10760#issuecomment-532399518

For reference, this changes 9 of the 34 invocations of rapply.

JasonGross avatar Sep 17 '19 21:09 JasonGross

Note that while this patch does no harm, it is not clear yet whether coq/coq#10760 will be merged as is, and thus whether it will be necessary.

Zimmi48 avatar Sep 18 '19 07:09 Zimmi48

I'm fine with the changes, but propose to wait until the discussion in coq/coq#10760 stabilizes.

spitters avatar Sep 18 '19 08:09 spitters