coverage: remove dependency on bisect_ppx

This infrastructure has been unused for many years (I think it only ever
worked for some months during 2019), and it's making ocaml 5.4 difficult
to adopt, so drop it.

I'm leaving the instrumentation in the dune metadata because it's
disabled by default, and the document about coverage, because it's a
historical document.

Signed-off-by: Pau Ruiz Safont <pau.safont@vates.tech>

Author: psafont

Makefile

@@ -21,11 +21,6 @@ build:
 check:
 	dune build @check -j $(JOBS)
 
-coverage:
-	dune runtest --instrument-with bisect_ppx --force --profile=$(RELEASE) -j $(JOBS)
-	bisect-ppx-report html
-	bisect-ppx-report summary --per-file
-
 clean:
 	dune clean
 

dune-project

@@ -858,7 +858,6 @@
   (alcotest :with-test)
   astring
   base-unix
-  (bisect_ppx :with-test)
   (clock
    (and
     (= :version)

ocaml/libs/xapi-stdext/lib/xapi-fd-test/dune

@@ -2,7 +2,5 @@
 (library
 	(name xapi_fd_test)
 	(libraries clock (re_export xapi-stdext-unix.fdcaps) unix qcheck-core logs fmt (re_export mtime) mtime.clock.os rresult threads.posix)
-
-	; off by default, enable with --instrument-with bisect_ppx
 	(instrumentation (backend bisect_ppx))
 )

ocaml/libs/xapi-stdext/lib/xapi-fdcaps/dune

@@ -5,7 +5,5 @@
 	(name xapi_fdcaps)
 	(libraries fmt unix threads.posix)
 	(flags (:standard -principal))
-
-	; off by default, enable with --instrument-with bisect_ppx
 	(instrumentation (backend bisect_ppx))
 )

ocaml/xapi-idl/lib/coverage/disabled.ml

@@ -12,7 +12,6 @@ depends: [
   "alcotest" {with-test}
   "astring"
   "base-unix"
-  "bisect_ppx" {with-test}
   "clock" {= version & with-test}
   "fd-send-recv" {>= "2.0.0"}
   "fmt"