- Add clang to the dependencies
- Migrate Node.js installation to their new recommended method
- Add some comments for each logical step
- Remove a redundant comment
- Remove some unnecessary semicolons
- setup_lts.x already run `apt-get update` internally so it's a waste of time to run it manually afterwards.
- the wget url wasn't quoted properly. incidentally the url doesn't actually need quoting anyway, which is why it wasn't caught.
- misc
- compiling from Docker is a slow way to compile anyway (because "COPY . ruffle" and "npm install" is un-cached, and virtualization overhead), so people who want minimal compilation time shouldn't use Docker compile anyway. (switching to :dual-wasm increased build time from 8m20s to 11m10s on my ancient 2011 server, 4x xeon E7-4870)
- seems "docker cp" does not actually overwrite the firefox_unsiged.xpi file if it already exists, for reasons unclear to me, so i added "rm -rf" to the sample invocation
- git wasn't actually used in docker build process, removed git from apt-get (rustup use the git protocol internally but doesn't seem to use the actual git executable)
usage:
run this file from ruffle root dir (not the docker dir) like
docker build --tag ruffle-web-docker -f web/docker/Dockerfile .
docker cp $(docker create ruffle-web-docker:latest):/ruffle/web/packages web/docker/docker_builds/packages
renovate[bot]