Skip to content

Commit

Permalink
opam: add a build-test step
Browse files Browse the repository at this point in the history
This should cause the CI to actually run the unit tests (!)

Also this removes a spurious dependency on `ocamlfind` and clarifies
that `ounit` is only needed for `test`. The minimum OCaml version is
bumped to the Mirage minimum of 4.04.0.

Signed-off-by: David Scott <[email protected]>
  • Loading branch information
djs55 committed Dec 3, 2017
1 parent b3d37b1 commit 6864825
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions xenstore.opam
Original file line number Diff line number Diff line change
Expand Up @@ -13,17 +13,18 @@ dev-repo: "https://github.com/mirage/ocaml-xenstore.git"
doc: "https://mirage.github.io/ocaml-xenstore"

build: [
[ "jbuilder" "subst"] {pinned}
[ "jbuilder" "build" "-p" name "-j" jobs ]
["jbuilder" "subst" "-p" name "--name" name] {pinned}
["jbuilder" "build" "-p" name "-j" jobs]
]

build-test: ["jbuilder" "runtest" "-p" name]

depends: [
"ocamlfind" {build}
"jbuilder" {build & >="1.0+beta9"}
"cstruct" {>= "1.9.0"}
"ppx_cstruct" {build}
"ppx_tools" {build}
"lwt"
"ounit" {build}
"ounit" {test}
]
available: [ ocaml-version >= "4.02.0" ]
available: [ ocaml-version >= "4.04.0" ]

0 comments on commit 6864825

Please sign in to comment.