Merge pull request #299 from hannesm/no-mmap

since cstruct-unix requires 4.08+, no need for mmap -- use Unix.map_file directly
mirage · Apr 1, 2022 · 7cfde7f · 7cfde7f
2 parents 407f1c1 + 2bb4afb
commit 7cfde7f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/cstruct-unix.opam b/cstruct-unix.opam
@@ -18,7 +18,6 @@ depends: [
   "ocaml" {>= "4.06.0"}
   "dune" {>= "2.0.0"}
   "base-unix"
-  "mmap" {>= "1.2.0"}
   "cstruct" {=version}
 ]
 synopsis: "Access C-like structures directly from OCaml"

diff --git a/unix/dune b/unix/dune
@@ -2,4 +2,4 @@
  (name cstruct_unix)
  (wrapped false)
  (public_name cstruct-unix)
- (libraries cstruct mmap unix))
+ (libraries cstruct unix))
diff --git a/unix/unix_cstruct.ml b/unix/unix_cstruct.ml
@@ -15,5 +15,5 @@
  *)
 
 let of_fd fd =
-  let buffer = Bigarray.(array1_of_genarray (Mmap.V1.map_file fd char c_layout false [|-1|])) in
+  let buffer = Bigarray.(array1_of_genarray (Unix.map_file fd char c_layout false [|-1|])) in
   Cstruct.of_bigarray buffer