Skip to content

Commit e910754

Browse files
authored
Update lwt dependency (#156)
1 parent 8b33464 commit e910754

File tree

8 files changed

+7
-9
lines changed

8 files changed

+7
-9
lines changed

.dockerignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,4 @@
77
!comby.opam
88
!dune
99
!push-coverage-report.sh
10+
!dune-project

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
.merlin
22
_build
33
*.install
4-
/dune-project
54
\#*
65
.\#*
76
scripts/0.*

Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ WORKDIR /home/comby
77

88
COPY Makefile /home/comby/
99
COPY comby.opam /home/comby/
10+
COPY dune-project /home/comby/
1011
COPY dune /home/comby/
1112
COPY docs /home/comby/docs
1213
COPY src /home/comby/src

comby.opam

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ build: [
1212
depends: [
1313
"ocaml" {>= "4.08.1"}
1414
"core" {>= "0.12.2"}
15-
"lwt" {>= "4.3.0"}
15+
"lwt"
1616
"mparser-comby"
1717
"ppxlib"
1818
"ppx_deriving"
@@ -32,7 +32,6 @@ depends: [
3232
pin-depends: [
3333
["bisect_ppx.git" "git+https://github.com/aantron/bisect_ppx.git"]
3434
["mparser-comby.git" "git+https://github.com/comby-tools/mparser.git"]
35-
["lwt.git" "git+https://github.com/rvantonder/lwt.git#4.3.0-with-captain"]
3635
["patdiff.git" "git+https://github.com/rvantonder/patdiff.git#0.12.1-patch-compatible-diffs"]
3736
]
3837
depexts: [

dockerfiles/alpine/base-dependencies/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,6 @@ COPY push-coverage-report.sh /home/comby/
2828
RUN sudo chown -R $(whoami) /home/comby
2929

3030
# Build and install the OCaml dependencies.
31-
RUN eval $(opam env) && opam install . --deps-only -y
31+
RUN eval $(opam env) && opam repository set-url default https://opam.ocaml.org && opam update && opam install . --deps-only -y
3232
# Delete the source files
3333
RUN sudo rm -rf /home/comby

dockerfiles/alpine/binary-release/Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ WORKDIR /home/comby
77

88
COPY Makefile /home/comby/
99
COPY comby.opam /home/comby/
10+
COPY dune-project /home/comby/
1011
COPY dune /home/comby/
1112
COPY docs /home/comby/docs
1213
COPY src /home/comby/src

dune-project

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
(lang dune 1.9)
2+
(name comby)

lib/pipeline/pipeline.ml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -377,11 +377,6 @@ let run
377377
count, inputs
378378
| _ -> failwith "Can't run interactive review for these inputs")
379379
in
380-
(* RVT modified lwt so that the toplevel Lwt scheduler is prevented from
381-
starting up at module load. Lwt interferes with hack_parallel child
382-
workers otherwise. This command runs the setup function, only after we are
383-
finished with parallel work. *)
384-
Lwt_unix.i_am_the_captain_now ();
385380
let { editor; default_is_accept } = Option.value_exn interactive_review in
386381
Interactive.run editor default_is_accept count rewrites;
387382
count

0 commit comments

Comments
 (0)