Skip to content

Commit 53b55f8

Browse files
committed
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.
1 parent af79521 commit 53b55f8

File tree

3 files changed

+1
-6
lines changed

3 files changed

+1
-6
lines changed

dune-project

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,10 +136,6 @@
136136
ppx_expect
137137
(ppx_minidebug
138138
(>= 2.0.3))
139-
; Cannot make it (patdiff :with-test), because `dune build -p neural_nets_lib @install`
140-
; builds the tutorials target from dune/test, despite it being inline_tests!
141-
(patdiff
142-
(>= v0.15.0))
143139
(odoc :with-doc)
144140
(md2mld :with-doc))
145141
(tags

neural_nets_lib.opam

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ depends: [
3636
"ppx_variants_conv"
3737
"ppx_expect"
3838
"ppx_minidebug" {>= "2.0.3"}
39-
"patdiff" {>= "v0.15.0"}
4039
"odoc" {with-doc}
4140
"md2mld" {with-doc}
4241
]

test/dune

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
(package neural_nets_lib)
1515
(inline_tests
1616
(deps ocannl_config))
17-
(libraries base dynlink patdiff.expect_test_patdiff ocannl)
17+
(libraries base dynlink ocannl)
1818
(modules
1919
einsum_trivia
2020
hello_world_op

0 commit comments

Comments
 (0)