Get rid of the patdiff dependency

Unfortunately it recursively depends on `core_unix`, which is not super portable. So it's nice that we don't need it anymore.
ahrefs · Jan 30, 2025 · 53b55f8 · 53b55f8
1 parent af79521
commit 53b55f8
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 6 deletions.
diff --git a/dune-project b/dune-project
@@ -136,10 +136,6 @@
   ppx_expect
   (ppx_minidebug
    (>= 2.0.3))
-  ; Cannot make it (patdiff :with-test), because `dune build -p neural_nets_lib @install`
-  ; builds the tutorials target from dune/test, despite it being inline_tests!
-  (patdiff
-   (>= v0.15.0))
   (odoc :with-doc)
   (md2mld :with-doc))
  (tags

diff --git a/neural_nets_lib.opam b/neural_nets_lib.opam
@@ -36,7 +36,6 @@ depends: [
   "ppx_variants_conv"
   "ppx_expect"
   "ppx_minidebug" {>= "2.0.3"}
-  "patdiff" {>= "v0.15.0"}
   "odoc" {with-doc}
   "md2mld" {with-doc}
 ]

diff --git a/test/dune b/test/dune
@@ -14,7 +14,7 @@
  (package neural_nets_lib)
  (inline_tests
   (deps ocannl_config))
- (libraries base dynlink patdiff.expect_test_patdiff ocannl)
+ (libraries base dynlink ocannl)
  (modules
   einsum_trivia
   hello_world_op