-
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
batteries-3.7.0 #24327
batteries-3.7.0 #24327
Conversation
build and tests are passing on my computer with ocaml-4.08 and ocaml-4.09. I don't understand what the CI tests are not happy about. |
I issued the final tag v3.7.0. PS: opam-ci test details are way too long. Probably, all things that pass should be hidden/summarized. Only things that are broken and that need fixing should be clearly stated/highlighted. |
thanks for the feedback. This was implemented in ocurrent/opam-repo-ci#232 and now live. |
I don't understand why some packages using obuild cannot find Batteries, BatList, ... any longer and are now failing |
@mseri batteries switched to dune recently; this might be related. |
packages relying on obuild generally have a higher bound defined: |
I will add an upper bound for the failing ones. There is another issue that should be solved, a number of test runs are failing with
Maybe there is some missing dependency or prerequisite in the dune files and then it is a matter of concurrency when the test works or not. |
@mseri according to my tests: batteries_v3.7.0 and ocaml-4.5.0 can repeatedly be built and tested. |
I guess this only reveals itself now with 47 jobs which is more than common. EDIT: I created a fix here: ocaml-batteries-team/batteries-included#1121. |
@sim642 thanks for checking and submitting a fix. |
I'll bump the minor version number in a new tag. |
I will issue the final tag once opam CI tests are OK