Skip to content

Commit

Permalink
Make num dependency optional
Browse files Browse the repository at this point in the history
sexplib pulls in a dependency on `num` which for most users is
unnecessary. Those users that need `sexplib.num` to convert to and from
`num` values have a dependency on `num` already, those who don't have no
need for `num` in their dependency cone.
  • Loading branch information
Leonidas-from-XIV committed Jun 3, 2024
1 parent c6acfdc commit 888d208
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 2 additions & 1 deletion num/lib/dune
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,5 @@
(name sexplib_num)
(public_name sexplib.num)
(libraries num sexplib)
(preprocess no_preprocessing))
(preprocess no_preprocessing)
(optional))
2 changes: 2 additions & 0 deletions sexplib.opam
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ depends: [
"parsexp"
"sexplib0"
"dune" {>= "3.11.0"}
]
depopts: [
"num"
]
available: arch != "arm32" & arch != "x86_32"
Expand Down

0 comments on commit 888d208

Please sign in to comment.