Skip to content

Commit b626547

Browse files
committed
Fix release building of assets
It's a little bit more complicated than though in the previous commit: - the Dockerfile will be run only when building for linux - The script is run both for linux and mac - The Dockerfile is run first I think duplicating dune subst is ok to have both that it is run first in linux and in mac modes.
1 parent 04a33c7 commit b626547

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

release/Dockerfile

+4
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,7 @@ RUN opam install -y --deps-only --with-test --with-doc .
88

99
COPY --chown=opam . .
1010

11+
RUN dune subst
12+
13+
RUN sed -i 's/"()"/"(-cclib -static -cclib -no-pie)"/g' src/compiler/bin/static-linking-flags/static_linking_flags.ml
14+

release/release.sh

-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@ archive_name=$OUTPUT/slipshow-$TARGETOS-$TARGETARCH.tar
55

66
dune subst
77

8-
sed -i 's/"()"/"(-cclib -static -cclib -no-pie)"/g' src/compiler/bin/static-linking-flags/static_linking_flags.ml
9-
108
dune build --profile release -p slipshow
119

1210
# Executables are symlinks, follow with -h.

0 commit comments

Comments
 (0)