-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
owl-ode, owl-ode-sundials: release 0.1.0 #14552
owl-ode, owl-ode-sundials: release 0.1.0 #14552
Conversation
048188c
to
d6ecdac
Compare
Unfortunately in debian buster libsundials-serial-dev is no longer present and has been replaced by libsundials-dev. So the build will always fail |
The 4.08.0 failure happens during the tests due to the usual mysterious error:
The one on mac is due to a too recent version of sundials |
Doesn't it make more sense to change it since the old one is not supposed to be used anymore? |
Signed-off-by: Marcello Seri <marcello.seri@gmail.com>
I've modified it, but the ocaml sundials version targets a very specific version, so I don't know if it is going to be working or not. If we see that it does not work, I'll remove the commit. I have decided to keep the old version as it will not prevent people stuck to older systems to keep their automatic setups |
Much better. I have added a workaround for the tests in docker in the next release, I am going to disable them for the moment. |
There will be distributions not providing ploplot nor sundials (or the version we need). Those will unfortunately fail |
🌤️ opam-lint warnings b7f954e
☀️ Installability check (+3)
|
Thanks! |
This pull-request concerns:
-
owl-ode.0.1.0
: Owl's ODE solvers-
owl-ode-sundials.0.1.0
: Owl's ODE solvers, interface with SundialsML🐫 Pull-request generated by opam-publish v2.0.0