Skip to content

Commit

Permalink
Disable the sexp tests on old compilers
Browse files Browse the repository at this point in the history
Due to its ppxlib constraint, ppx_import needs ppx_sexp_conv >= v.0.15.0,
which (transitively via base) requires a compiler >= 4.10.0.
  • Loading branch information
pitag-ha committed Jun 13, 2022
1 parent 343485d commit 4ee94ea
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
12 changes: 11 additions & 1 deletion ppx_import.opam
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,22 @@ dev-repo: "git+https://github.com/ocaml-ppx/ppx_import.git"
tags: [ "syntax" ]

depends: [
"ocaml" { >= "4.05.0" }
(
"ocaml" {>= "4.05.0" & < "4.10.0" }
"dune" { >= "1.11.0" }
"ppxlib" { >= "0.26.0" }
"ounit" { with-test }
"ppx_deriving" { with-test & >= "4.2.1" }
)
|
(
"ocaml" { >= "4.10.0" }
"ppx_sexp_conv" { with-test & >= "v0.13.0" }
"dune" { >= "1.11.0" }
"ppxlib" { >= "0.26.0" }
"ounit" { with-test }
"ppx_deriving" { with-test & >= "4.2.1" }
)
]

build: [["dune" "build" "-p" name "-j" jobs]
Expand Down
2 changes: 2 additions & 0 deletions src_test/ppx_deriving_sexp/dune
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
(test
(name test_ppx_deriving_sexp)
(enabled_if
(>= %{ocaml_version} "4.10.0"))
(preprocess
(staged_pps ppx_import ppx_sexp_conv)))

0 comments on commit 4ee94ea

Please sign in to comment.