c-cube/gen

Pervasives deprecated

mgrabovsky opened this issue · 4 comments

mgrabovsky commented

Building with dune 2.5.1 and OCaml 4.09.1 on Arch Linux, I get the following errors:

File "src/gen.ml", line 671, characters 17-35:
671 | let lexico ?(cmp=Pervasives.compare) gen1 gen2 =
                       ^^^^^^^^^^^^^^^^^^
Error (alert deprecated): module Stdlib.Pervasives
Use Stdlib instead.

If you need to stay compatible with OCaml < 4.07, you can use the
stdlib-shims library: https://github.com/ocaml/stdlib-shims
File "src/gen.ml", line 851, characters 23-41:
851 | let intersection ?(cmp=Pervasives.compare) gen1 gen2 =
                             ^^^^^^^^^^^^^^^^^^
Error (alert deprecated): module Stdlib.Pervasives
Use Stdlib instead.

If you need to stay compatible with OCaml < 4.07, you can use the
stdlib-shims library: https://github.com/ocaml/stdlib-shims
[...]
c-cube commented

Are you compiling outside of opam? This should just be a warning. But I'll work on it.

mgrabovsky commented

Yes, this is without opam.

c-cube commented

it's because dune treats warnings as errors by default. I'm pushing a fix for that, but dune build --profile=release should actually already work.

  • 👍1
mgrabovsky commented

Great, thanks. I'm not that familiar with the OCaml/dune cosmos.