search

janestreet / toplevel_expect_test

Toplevel expectation test
MIT License
13 stars 5 forks source link

ocaml-expect calls `patdiff` #5

Closed Drup closed 6 years ago

Drup commented 7 years ago

I have no idea where patdiff comes from. diff is fine too, you should test for availability and use the one actually available.

hhugo commented 7 years ago

https://github.com/janestreet/patdiff

Drup commented 7 years ago

Oh, so ocaml-expect even requires core_extended, through patdiff.

Please use diff...

ghost commented 7 years ago

You can use -diff-command diff

hhugo commented 6 years ago

It should now automatically fallback to diff if patdiff is not available