-
Notifications
You must be signed in to change notification settings - Fork 168
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
release infotheo 0.9.0 #3342
release infotheo 0.9.0 #3342
Conversation
@affeldt-aist it looks like the tests don't work on OCaml < 5, which is what we use in the Coq Platform (coq-native is not available on OCaml 5). Maybe disable the tests? |
Given the error, it does not have anything to do with OCaml < 5. It is an actual bug in Extract Inductive ConstructiveCauchyReals.CReal => "float" ["assert false"]. |
My bad. I forgot to run the tests locally. The above Extract command is not supposed to be executed. |
@affeldt-aist so how do you want to handle this? We could remove the tests from the opam package and add the |
Removing the tests + avoid-version flag seems good to me, I'll release a new version soon (certainly this week), sorry for the trouble :-( |
- fix opam file for 0.9.0 and 0.7.7
We released a new version of InfoTheo (0.9.1) (which relies on MathComp-Analysis 1.9.0 which has been released in-between) where the tests ought to go through. |
The compilation fails immediately. Could it be that the package is incompatible with MathComp Analysis 1.9.0, which was released yesterday?
|
Yes, this is at least one problem (I pushed a fix but I feel bad for triggering again a full CI). |
Oh! Thanks for your patience! |
This release essentially rely on MathComp-Analysis instead of the Coq standard library.