Skip to content

Commit

Permalink
Detect linux better for the perf library.
Browse files Browse the repository at this point in the history
This fixes coq#18257 following the recommendations given in
ocaml/dune#4895. Starting with OCaml 5.2, testing "linux"
will be enough thanks to ocaml/ocaml#12405.
  • Loading branch information
Rodolphe Lepigre committed Nov 6, 2023
1 parent d52b787 commit 4c31ea5
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion perf/dune
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,11 @@
(public_name coq-core.perf)
(wrapped false)
(foreign_stubs (language c) (names perf))
(enabled_if (= %{system} linux)))
; Just "linux" is not enough, see https://github.com/ocaml/dune/issues/4895.
(enabled_if
(or
(= %{system} "linux")
(= %{system} "linux_eabi")
(= %{system} "linux_eabihf")
(= %{system} "linux_elf")
(= %{system} "elf"))))

0 comments on commit 4c31ea5

Please sign in to comment.