search

msgpack / msgpack-ocaml

MessagePack implementation for OCaml / msgpack.org[OCaml]
45 stars 19 forks source link

Set the minimal OCaml version required #24

Closed kit-ty-kate closed 7 years ago

kit-ty-kate commented 7 years ago

This f.i.x.e.s. the travis build problem in https://github.com/ocaml/opam-repository/pull/9927 You are making use of @@, "%revapply" (since 4.01) and List.iteri (since 4.00). If you prefer to stay retro-compatible with older versions please ask, I've got the patch ready.

mzp commented 7 years ago

Thanks!

If you prefer to stay retro-compatible with older versions please ask, I've got the patch ready.

Great. But I don't feel the need of supporting older verison.

mzp commented 7 years ago

Wrote Changelog https://github.com/msgpack/msgpack-ocaml/commit/fc48b5492f59d74a54a164618706cdab43e8f6a7 and updated opam file at https://github.com/ocaml/opam-repository/pull/9927.

kit-ty-kate commented 7 years ago

Thanks !