Skip to content

Commit d5e7061

Browse files
committed
Fix Docker container to include jsoo, which is now needed
1 parent de6b27b commit d5e7061

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

Dockerfile

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,14 @@ ARG opam_switch="/home/opam/.opam/4.12"
7171
COPY --from=compilation /home/opam/install-prefix /usr
7272
COPY --from=compilation "$opam_switch/bin"/ocaml* "$opam_switch/bin/"
7373
COPY --from=compilation "$opam_switch/lib/ocaml" "$opam_switch/lib/ocaml/"
74+
COPY --from=compilation "$opam_switch/bin/js_of_ocaml" "$opam_switch/bin/"
75+
COPY --from=compilation "$opam_switch/lib/js_of_ocaml" "$opam_switch/lib/js_of_ocaml"
76+
77+
# Fixes for ocamlfind
78+
COPY --from=compilation "$opam_switch/lib/findlib.conf" "$opam_switch/lib/"
79+
COPY --from=compilation "$opam_switch/lib/stdlib" "$opam_switch/lib/stdlib"
80+
ENV PATH="${opam_switch}/bin:${PATH}"
81+
ENV OCAMLPATH="/usr/lib"
7482

7583
ENTRYPOINT ["dumb-init","/usr/bin/learn-ocaml","--sync-dir=/sync","--repo=/repository"]
7684
CMD ["build","serve"]

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ docker-images: Dockerfile learn-ocaml.opam
6969
@docker build -t learn-ocaml-compilation --target compilation docker
7070
@docker build -t learn-ocaml --target program docker
7171
@docker build -t learn-ocaml-client --target client docker
72-
@echo "Use with 'docker run --rm -v \$$PWD/sync:/sync -v \$$PWD:/repository -p PORT:8080 learn-ocaml -- ARGS'"
72+
@echo "Use with 'docker run --rm -v learn-ocaml-sync:/sync -v \$$PWD:/repository -p PORT:8080 learn-ocaml -- ARGS'"
7373

7474
VERSION = $(shell opam show ./learn-ocaml.opam -f version)
7575

0 commit comments

Comments
 (0)