timbertson / opam2nix

Generate nix expressions from opam packages
MIT License
93 stars 28 forks source link

Can't build opam2nix tests #89

Closed talex5 closed 1 year ago

talex5 commented 1 year ago

I wanted to run the tests, but dune runtest fails:

$ dune runtest
File "test_src/test_vars.ml", line 12, characters 55-68:
12 |        assert_equal ~printer:print_var (Some (S expected)) (Vars.path_var ~env ~prefix ~scope name)
                                                                 ^^^^^^^^^^^^^
Error: Unbound value Vars.path_var

(I noticed this because I was trying to update it to use dune 2, and there dune build builds the tests by default)

Vars.path_var is not exported, and the function with that name doesn't match the signature, so I'm not sure what to do about it.

timbertson commented 1 year ago

Sorry for the glacial response, I'm not doing much work in ocaml/opam at the moment. Fixed in dee96eb1c706edc61e38df35240e0f707cfb800e