Skip to content

Commit 19b816e

Browse files
authored
Misc: remove test deps on cohttp-lwt-unix (#1902)
1 parent 31317ca commit 19b816e

File tree

8 files changed

+7
-6
lines changed

8 files changed

+7
-6
lines changed

.github/workflows/build-wasm_of_ocaml.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ jobs:
124124
working-directory: ./wasm_of_ocaml
125125
run: |
126126
opam install .
127-
opam install num cohttp-lwt-unix ppx_expect cstruct uucp
127+
opam install num ppx_expect cstruct uucp
128128
129129
- name: Run tests
130130
if: ${{ matrix.separate_compilation }}

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ jobs:
165165
- name: build doc
166166
if: ${{ !matrix.skip-doc && github.event_name == 'push' && github.ref_name == 'master'}}
167167
run: |
168-
opam install odoc lwt_log cohttp-lwt-unix yojson ocp-indent graphics higlo
168+
opam install odoc lwt_log yojson ocp-indent graphics higlo
169169
opam exec -- make doc
170170
171171
- name: synchronize doc

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ We actively welcome pull requests.
2323
### Install dependencies
2424
```
2525
opam install --deps-only -t js_of_ocaml js_of_ocaml-lwt js_of_ocaml-compiler js_of_ocaml-toplevel js_of_ocaml-ppx js_of_ocaml-ppx_deriving_json js_of_ocaml-tyxml
26-
opam install odoc lwt_log cohttp-lwt-unix yojson ocp-indent graphics higlo
26+
opam install odoc lwt_log yojson ocp-indent graphics higlo
2727
```
2828

2929
### Running the tests

dune-project

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@
9393
(ocaml (>= 4.08))
9494
(js_of_ocaml-compiler (= :version))
9595
(ocamlfind (>= 1.5.1))
96-
(cohttp-lwt-unix (and (>= 6.0.0) :with-test))
96+
;;(cohttp-lwt-unix (and (>= 6.0.0) :with-test))
9797
(graphics :with-test)
9898
(num :with-test)
9999
(ppx_expect (and (>= v0.14.2) :with-test))

js_of_ocaml-toplevel.opam

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ depends: [
1616
"ocaml" {>= "4.08"}
1717
"js_of_ocaml-compiler" {= version}
1818
"ocamlfind" {>= "1.5.1"}
19-
"cohttp-lwt-unix" {>= "6.0.0" & with-test}
2019
"graphics" {with-test}
2120
"num" {with-test}
2221
"ppx_expect" {>= "v0.14.2" & with-test}

manual/contribute.wiki

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ We actively welcome pull requests.
2323
=== Install dependencies
2424
{{{
2525
opam install --deps-only js_of_ocaml js_of_ocaml-lwt js_of_ocaml-compiler js_of_ocaml-toplevel js_of_ocaml-ppx js_of_ocaml-ppx_deriving_json js_of_ocaml-tyxml
26-
opam install lwt reactiveData tyxml ppxlib cohttp-lwt-unix menhir ppx_expect yojson sexplib
26+
opam install lwt reactiveData tyxml ppxlib menhir ppx_expect yojson sexplib
2727
}}}
2828

2929
=== Running the tests

tools/ci_setup.ml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,7 @@ let () =
335335
pin_packages ();
336336
install_others others;
337337
sync_exec (fun () -> clone "ocaml-uri" "https://github.com/mirage/ocaml-uri") [ () ];
338+
sync_exec (fun () -> exec_async "opam install uri --deps-only") [ () ];
338339
sync_exec
339340
(fun nm ->
340341
let branch = if is_forked nm then Some "wasm-v0.18" else None in

toplevel/examples/server/dune

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
(executables
22
(names server)
3+
(enabled_if false)
34
(libraries findlib cohttp-lwt-unix))

0 commit comments

Comments
 (0)