Marek Kubica
Marek Kubica
Given #54 has somewhat stalled, this is a PR to upstream the dune port that we successfully use in @dune-universe. The @dune-universe ports generally try to touch as little code...
The Travis CI link is dead and so is Travis, really. Fortunately it is built by OCaml CI so we can use that as drop-in.
There was a number of packages missing from the dependencies, like a new-enough Yojson, Cohttp, Batteries and Lwt, so this PR adds them.
This is only required to get the META file for `bytes` but given the library is not used in the codebase, no need for it. This decreases the dependency cone.
It is not used to get the `bytes` library, nor is it necessary since OCaml 4.02 (the lowest version dune supports). This decreases the dependency cone by not requiring `ocamlfind`...
This will use `mariadb_config` to determine what to compile and link against. Closes #15. I've removed support for switching between connector and uhm, server, since that could be handled by...
`Yojson.Safe.t` was added in Yojson 1.6.0 and while I am going through the reverse dependencies of Yojson for the upcoming 2.0 release it seems reasonable to update that constraint.
I tried running the tests with Yojson 2 and they seem to… just work. So this constraint seems to be overly cautionous.
While building https://opam.ci.ocaml.org/github/ocaml/opam-repository/commit/aa898f09e7ffe95265651b4b2dda4d21d91baa19/variant/opam-2.1,compilers,4.14,ounit.2.1.2,revdeps,bencode.2.0 it turns out that ounit 2.1.2 was installed via one dependency and ounit2 2.2.6 via another. This fails to link since the symbols exist twice. So while...
This package is never required since jbuilder/dune require OCaml 4.02 and 4.02 has Bytes out of the box. This PR is an exploration of the explanation given by @dra27 in...