Skip to content

Commit

Permalink
Merge pull request #206 from yfyf/remove-ppxlib-dependency
Browse files Browse the repository at this point in the history
Patch ppx_protocol_conv_jsonm to not have a dependency on ppxlib
  • Loading branch information
knuton authored Nov 21, 2024
2 parents 4facb4a + dadc2d7 commit a05e86f
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 0 deletions.
4 changes: 4 additions & 0 deletions pkgs/ocaml-modules/ppx_protocol_conv_jsonm/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ buildDunePackage rec {

useDune2 = true;

patches = [
./remove-runtime-dependency-from-ppxlib.patch
];

propagatedBuildInputs = [
ppx_protocol_conv
ezjsonm
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
diff --git a/drivers/jsonm/dune b/drivers/jsonm/dune
index fb0bc81..554241f 100644
--- a/drivers/jsonm/dune
+++ b/drivers/jsonm/dune
@@ -1,6 +1,6 @@
(library
(name protocol_conv_jsonm)
(public_name ppx_protocol_conv_jsonm)
- (libraries ppx_protocol_conv ppx_protocol_conv.runtime ppx_protocol_conv.driver ezjsonm)
+ (libraries ppx_protocol_conv.runtime ppx_protocol_conv.driver ezjsonm)
(synopsis "jsonm (de)serialization driver for ppx_protocol_conv based on ezjsonm")
)

0 comments on commit a05e86f

Please sign in to comment.