Skip to content

Commit b380565

Browse files
authored
Merge pull request #14 from JustAMan/patch-emscripten
Use patched emscripten so Octopus would work in Cordova
2 parents 7e6b75d + ddf2959 commit b380565

5 files changed

+32
-26
lines changed

Dockerfile

+8-2
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,21 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
2222
automake \
2323
autoconf \
2424
m4 \
25-
gperf && \
26-
pip install ply && \
25+
gperf \
26+
wget && \
2727
rm -rf /var/lib/apt/lists/*
2828

29+
RUN pip install ply
30+
2931
RUN git clone https://github.com/emscripten-core/emsdk.git && \
3032
cd emsdk && \
3133
./emsdk install 1.39.11 && \
3234
./emsdk activate 1.39.11
3335

36+
# Patch emscripten; needed until https://github.com/emscripten-core/emscripten/pull/10846 is merged and released
37+
RUN wget https://raw.githubusercontent.com/emscripten-core/emscripten/d68250f1e6059168bd1f791921445527c7548e29/src/preamble.js -O /emsdk/upstream/emscripten/src/preamble.js && \
38+
wget https://raw.githubusercontent.com/emscripten-core/emscripten/d68250f1e6059168bd1f791921445527c7548e29/src/URIUtils.js -O /emsdk/upstream/emscripten/src/URIUtils.js
39+
3440
ENV PATH=$PATH:/emsdk:/emsdk/upstream/emscripten:/emsdk/node/12.9.1_64bit/bin
3541
WORKDIR /code
3642
CMD ["make"]

dist/js/subtitles-octopus-worker-legacy.js

+23-23
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
-184 Bytes
Binary file not shown.

dist/js/subtitles-octopus-worker.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/js/subtitles-octopus-worker.wasm

219 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)