janestreet / jane-street-tests

Apache License 2.0
5 stars 2 forks source link

Error trying to build this repo: Package `ppx_expect.evaluator' not found #1

Open taylorjg opened 8 years ago

taylorjg commented 8 years ago

Hi,

I just cloned this repo and tried to build it but I get the following error:

$ make
make -C test-inline-tests/with-oasis-method1 test
ocamlfind: Package `ppx_expect.evaluator' not found
W: Field 'pkg_ppx_expect_evaluator' is not set: Command ''/Users/jonathantaylor/.opam/system/bin/ocamlfind' query -format %d ppx_expect.evaluator > '/var/folders/y9/bys8qbvx0bb1nnmbv__l_1t40000gn/T/oasis-efb328.txt'' terminated with error code 2
E: Cannot find findlib package ppx_expect.evaluator
E: Failure("1 configuration error")
make[1]: *** [test] Error 1
make: *** [tests] Error 2

Here are some more details of my setup:

$ ocaml --version
The OCaml toplevel, version 4.03.0
$ ocamlc --version
4.03.0
$ ocamlopt --version
4.03.0
$ 
$ ocamlfind list | grep ppx_expect
ppx_expect          (version: 113.33.01+4.03)
ppx_expect.collector (version: 113.33.01+4.03)
ppx_expect.common   (version: 113.33.01+4.03)
ppx_expect.config   (version: 113.33.01+4.03)
ppx_expect.matcher  (version: 113.33.01+4.03)
ppx_expect.payload  (version: 113.33.01+4.03)
ppx_expect.payload.for_ppx_deriving (version: n/a)
$ 
$ oasis version
0.4.6
$ 
$ sw_vers
ProductName:    Mac OS X
ProductVersion: 10.9.5
BuildVersion:   13F1808

Regards, Jonathan Taylor

ghost commented 8 years ago

I think this problem was fixed on 4.02.3 but not in the 4.03 branches of our packages. We are planning to do a proper stable release for 4.03 soon, which will fix this as well

wolfram8 commented 7 years ago

@diml Any update on this? It seems that there is already a version 114 of ppx_expect that fixes this issue and works on 4.03 and 4.03, but it doesn't seem to be on opam yet.

ghost commented 7 years ago

The 114 series got delayed and now we prefer to wait for the next release which will be easier to maintain moving forward. We are planning to do a release beginning of February, sorry for the wait. You should be able to sort this out by fiddling with the META files, but I don't remember exactly how. If you can make it work, we'll be happy to merge a patch to fix this issue now.