Skip to content

Commit

Permalink
ldp_curl is not compatible with cohttp 6
Browse files Browse the repository at this point in the history
Fails with
```
=== ERROR while compiling ldp_curl.0.3.0 =====================================#
 context              2.4.0~alpha1~dev | linux/x86_64 | ocaml-base-compiler.4.14.2 | file:///home/opam/opam-repository
 path                 ~/.opam/4.14/.opam-switch/build/ldp_curl.0.3.0
 command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p ldp_curl -j 39 --promote-install-files=false @install
 exit-code            1
 env-file             ~/.opam/log/ldp_curl-7-93e790.env
 output-file          ~/.opam/log/ldp_curl-7-93e790.out
 (cd _build/default && /home/opam/.opam/4.14/bin/ocamlopt.opt -g -w -40 -w -6-7-9-10-27-32-33-34-35-36-50-52 -no-strict-sequence -g -I curl/.ldp_curl.objs/byte -I curl/.ldp_curl.objs/native -I /home/opam/.opam/4.14/lib/angstrom -I /home/opam/.opam/4.14/lib/base64 -I /home/opam/.opam/4.14/lib/bigstringaf -I /home/opam/.opam/4.14/lib/bytes -I /home/opam/.opam/4.14/lib/cohttp -I /home/opam/.opam/4.14/lib/cohttp-lwt -I /home/opam/.opam/4.14/lib/curl -I /home/opam/.opam/4.14/lib/dune-build-info -I /home/opam/.opam/4.14/lib/fmt -I /home/opam/.opam/4.14/lib/gen -I /home/opam/.opam/4.14/lib/http -I /home/opam/.opam/4.14/lib/http/__private__/http_bytebuffer -I /home/opam/.opam/4.14/lib/iri -I /home/opam/.opam/4.14/lib/ldp -I /home/opam/.opam/4.14/lib/logs -I /home/opam/.opam/4.14/lib/lwt -I /home/opam/.opam/4.14/lib/lwt/unix -I /home/opam/.opam/4.14/lib/menhirLib -I /home/opam/.opam/4.14/lib/ocaml/threads -I /home/opam/.opam/4.14/lib/ocf -I /home/opam/.opam/4.14/lib/ocplib-endian -I /home/opam/.opam/4.14/lib/ocplib-endian/bigstring -I /home/opam/.opam/4.14/lib/pcre -I /home/opam/.opam/4.14/lib/ppx_sexp_conv/runtime-lib -I /home/opam/.opam/4.14/lib/ptime -I /home/opam/.opam/4.14/lib/rdf -I /home/opam/.opam/4.14/lib/re -I /home/opam/.opam/4.14/lib/re/str -I /home/opam/.opam/4.14/lib/sedlex -I /home/opam/.opam/4.14/lib/seq -I /home/opam/.opam/4.14/lib/sexplib0 -I /home/opam/.opam/4.14/lib/stringext -I /home/opam/.opam/4.14/lib/uri -I /home/opam/.opam/4.14/lib/uri-sexp -I /home/opam/.opam/4.14/lib/uucp -I /home/opam/.opam/4.14/lib/uuidm -I /home/opam/.opam/4.14/lib/uunf -I /home/opam/.opam/4.14/lib/uutf -I /home/opam/.opam/4.14/lib/xmlm -I /home/opam/.opam/4.14/lib/yojson -intf-suffix .ml -no-alias-deps -o curl/.ldp_curl.objs/native/ldp_curl.cmx -c -impl curl/ldp_curl.pp.ml)
 File "curl/ldp_curl.ml", line 61, characters 22-42:
 61 |     module Response = Cohttp.Response.Make (IO)
                            ^^^^^^^^^^^^^^^^^^^^
 Alert deprecated: module Cohttp.Response.Make
 This functor is not part of the public API.
 File "curl/ldp_curl.ml", line 75, characters 23-40:
 75 |                 | _ -> Response.has_body res
                             ^^^^^^^^^^^^^^^^^
 Error: Unbound value Response.has_body
 (cd _build/default && /home/opam/.opam/4.14/bin/ocamlc.opt -g -w -40 -w -6-7-9-10-27-32-33-34-35-36-50-52 -no-strict-sequence -g -bin-annot -I curl/.ldp_curl.objs/byte -I /home/opam/.opam/4.14/lib/angstrom -I /home/opam/.opam/4.14/lib/base64 -I /home/opam/.opam/4.14/lib/bigstringaf -I /home/opam/.opam/4.14/lib/bytes -I /home/opam/.opam/4.14/lib/cohttp -I /home/opam/.opam/4.14/lib/cohttp-lwt -I /home/opam/.opam/4.14/lib/curl -I /home/opam/.opam/4.14/lib/dune-build-info -I /home/opam/.opam/4.14/lib/fmt -I /home/opam/.opam/4.14/lib/gen -I /home/opam/.opam/4.14/lib/http -I /home/opam/.opam/4.14/lib/iri -I /home/opam/.opam/4.14/lib/ldp -I /home/opam/.opam/4.14/lib/logs -I /home/opam/.opam/4.14/lib/lwt -I /home/opam/.opam/4.14/lib/lwt/unix -I /home/opam/.opam/4.14/lib/menhirLib -I /home/opam/.opam/4.14/lib/ocaml/threads -I /home/opam/.opam/4.14/lib/ocf -I /home/opam/.opam/4.14/lib/ocplib-endian -I /home/opam/.opam/4.14/lib/ocplib-endian/bigstring -I /home/opam/.opam/4.14/lib/pcre -I /home/opam/.opam/4.14/lib/ppx_sexp_conv/runtime-lib -I /home/opam/.opam/4.14/lib/ptime -I /home/opam/.opam/4.14/lib/rdf -I /home/opam/.opam/4.14/lib/re -I /home/opam/.opam/4.14/lib/re/str -I /home/opam/.opam/4.14/lib/sedlex -I /home/opam/.opam/4.14/lib/seq -I /home/opam/.opam/4.14/lib/sexplib0 -I /home/opam/.opam/4.14/lib/stringext -I /home/opam/.opam/4.14/lib/uri -I /home/opam/.opam/4.14/lib/uri-sexp -I /home/opam/.opam/4.14/lib/uucp -I /home/opam/.opam/4.14/lib/uuidm -I /home/opam/.opam/4.14/lib/uunf -I /home/opam/.opam/4.14/lib/uutf -I /home/opam/.opam/4.14/lib/xmlm -I /home/opam/.opam/4.14/lib/yojson -intf-suffix .ml -no-alias-deps -o curl/.ldp_curl.objs/byte/ldp_curl.cmo -c -impl curl/ldp_curl.pp.ml)
 File "curl/ldp_curl.ml", line 61, characters 22-42:
 61 |     module Response = Cohttp.Response.Make (IO)
                            ^^^^^^^^^^^^^^^^^^^^
 Alert deprecated: module Cohttp.Response.Make
 This functor is not part of the public API.
 File "curl/ldp_curl.ml", line 75, characters 23-40:
 75 |                 | _ -> Response.has_body res
                             ^^^^^^^^^^^^^^^^^
 Error: Unbound value Response.has_body
```

Signed-off-by: Marcello Seri <[email protected]>
  • Loading branch information
mseri committed Nov 26, 2024
1 parent 99bc90f commit 8226874
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion packages/ldp_curl/ldp_curl.0.2.0/opam
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ bug-reports: "https://framagit.org/zoggy/ocaml-ldp/issues"
depends: [
"dune" {>= "2.9"}
"ldp" {= version}
"cohttp-lwt" {< "6.0~~"}
"ocaml" {>= "4.14.0"}
"ocurl" {>= "0.9.1"}
"odoc" {with-doc}
Expand Down Expand Up @@ -36,4 +37,4 @@ url {
"md5=7f82ccbdcb07a75f6d2bb7ca226d02df"
"sha512=42f8e53d6634de4b19ac5249fcd8e3d2bde13de7a3963e6821eae9e7b46ff914458d3bfde201352fd18b6de14bb276705561d8fd09a09ddd16f001d0f25cf209"
]
}
}
3 changes: 2 additions & 1 deletion packages/ldp_curl/ldp_curl.0.3.0/opam
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ bug-reports: "https://framagit.org/zoggy/ocaml-ldp/issues"
depends: [
"dune" {>= "2.9"}
"ldp" {= version}
"cohttp-lwt" {< "6.0~~"}
"ocaml" {>= "4.14.0"}
"ocurl" {>= "0.9.1"}
"odoc" {with-doc}
Expand Down Expand Up @@ -36,4 +37,4 @@ url {
"md5=8903912748afccb622e88b52d1d89260"
"sha512=4924c40cf5f80d7b46e93de848bc52cbe7fa0b864082f93c07c253cd0d81395160339e80533d6b5bd4779c93afd1e37754323e1ca4a5a0b5371c840ee1bf2f67"
]
}
}

0 comments on commit 8226874

Please sign in to comment.