search

rleonid / oml

OCaml Math Library
Apache License 2.0
119 stars 9 forks source link

opam install oml fails #151

Closed Tyruiop closed 8 years ago

Tyruiop commented 8 years ago

Hi! I'm having the following problem when running opam install oml and I can't find a way to solve it, output is here. Any idea how to solve it?

struktured commented 8 years ago

I think the oml opam package is broken because lacaml changed its api. @rleonid spent some time the last week preparing a new release to resolve the api changes. A work around is you could manually downgrade lacaml with something like opam install lacaml.7.2.6 oml.0.0.5, or you can try getting the master branch to compile (requires https://github.com/rleonid/dsfo).

Tyruiop commented 8 years ago

Worked perfectly, thanks a lot!

dsheets commented 8 years ago

If you know the lacaml version constraint, could you contribute it back to ocaml/opam-repository, please?

rleonid commented 8 years ago

@Tyruiop Once the constraints are in place (here and in opam) are there are other issues with installing oml?

Tyruiop commented 8 years ago

@rleonid as far as I can tell, the constrain for lacaml needs to be <= 7.6.2, not = 8.0.6. Also, I don't think it should require dsfo until it gets in opam

rleonid commented 8 years ago

@Tyruiop My apologies for the confusion. The constraints are on the master branch of oml, not the tagged versions in opam, their purpose is so that oml can build "right now". dsfo is also only necessary for testing, and can easily be pinned directly to the github repo.

Tyruiop commented 8 years ago

@rleonid my bad. From git, it builds fine here even with the last version of lacaml.

rleonid commented 8 years ago

Glad that we resolved it, happy to hear about any other feedback on oml.