Skip to content

compatibility with coq 8.12#241

Merged
CohenCyril merged 2 commits intomasterfrom
compatibility_coq_8_12
Jul 27, 2020
Merged

compatibility with coq 8.12#241
CohenCyril merged 2 commits intomasterfrom
compatibility_coq_8_12

Conversation

@affeldt-aist
Copy link
Member

This is to fix a compatibility problem with Coq 8.12 observed by Karl.

@affeldt-aist affeldt-aist added this to the 0.3.2 milestone Jul 26, 2020
near=> c; move: k'Ok kOg; apply: filter_app2; near=> x => lek'c2k.
rewrite -(@ler_pmul2l _ c2%:num) // mulrA => /(le_trans lek'c2k) /le_trans; apply.
by rewrite ler_pmul //; near: c; apply: locally_pinfty_ge.
Unshelve. end_near. Grab Existential Variables. all: end_near. Qed.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@affeldt-aist I wonder if this still compiles with Coq 8.10 and 8.11

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It went through. But we'd better get rid of it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I meant: does removing this still make Coq 8.10 pass?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opam Outdated
]
depends: [
"coq" { ((>= "8.7" & < "8.12~") | = "dev") }
"coq" { ((>= "8.7" & < "8.13~") | = "dev") }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK Coq < 8.10 is not supported anymore

Copy link
Member

@CohenCyril CohenCyril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's rebase and merge

@affeldt-aist affeldt-aist force-pushed the compatibility_coq_8_12 branch from f22b44d to 64f2f5e Compare July 27, 2020 12:45
@CohenCyril CohenCyril merged commit b8595a1 into master Jul 27, 2020
@affeldt-aist affeldt-aist deleted the compatibility_coq_8_12 branch July 29, 2020 10:15
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