Skip to content

Commit

Permalink
Merge pull request #14 from JustAMan/patch-emscripten
Browse files Browse the repository at this point in the history
Use patched emscripten so Octopus would work in Cordova
  • Loading branch information
JustAMan authored Apr 8, 2020
2 parents 7e6b75d + ddf2959 commit b380565
Show file tree
Hide file tree
Showing 5 changed files with 32 additions and 26 deletions.
10 changes: 8 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,21 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
automake \
autoconf \
m4 \
gperf && \
pip install ply && \
gperf \
wget && \
rm -rf /var/lib/apt/lists/*

RUN pip install ply

RUN git clone https://github.com/emscripten-core/emsdk.git && \
cd emsdk && \
./emsdk install 1.39.11 && \
./emsdk activate 1.39.11

# Patch emscripten; needed until https://github.com/emscripten-core/emscripten/pull/10846 is merged and released
RUN wget https://raw.githubusercontent.com/emscripten-core/emscripten/d68250f1e6059168bd1f791921445527c7548e29/src/preamble.js -O /emsdk/upstream/emscripten/src/preamble.js && \
wget https://raw.githubusercontent.com/emscripten-core/emscripten/d68250f1e6059168bd1f791921445527c7548e29/src/URIUtils.js -O /emsdk/upstream/emscripten/src/URIUtils.js

ENV PATH=$PATH:/emsdk:/emsdk/upstream/emscripten:/emsdk/node/12.9.1_64bit/bin
WORKDIR /code
CMD ["make"]
46 changes: 23 additions & 23 deletions dist/js/subtitles-octopus-worker-legacy.js

Large diffs are not rendered by default.

Binary file modified dist/js/subtitles-octopus-worker-legacy.js.mem
Binary file not shown.
2 changes: 1 addition & 1 deletion dist/js/subtitles-octopus-worker.js

Large diffs are not rendered by default.

Binary file modified dist/js/subtitles-octopus-worker.wasm
Binary file not shown.

0 comments on commit b380565

Please sign in to comment.