search

ocaml-toml / To.ml

OCaml library for TOML
https://ocaml-toml.github.io/To.ml
Other
82 stars 20 forks source link

menhir is required, yet isn't a library #35

Closed c-cube closed 9 years ago

c-cube commented 9 years ago

In _oasis, mehhir is added as a dependency ("XMetaRequires"), but menhir is a binary. It should only be required for compiling, not when linking the library. Subsequently I can't use the library (version 2.2.0).

mackwic commented 9 years ago

thanks for the bug report.

So basically, the fix would be to remove "menhir" in this line in the _oasis_ file ?

c-cube commented 9 years ago

I think so. Why do you need menhirLib to build, btw?

sagotch commented 9 years ago

As far as I know, you only need the menhir binary in order to compile toml.

sagotch commented 9 years ago

@c-cube I just released 2.2.1, so this issue should really be fixed now.

Thanks again for reporting and for you fix :+1: