Skip to content

Commit

Permalink
Use docker for node parts of build script, too
Browse files Browse the repository at this point in the history
since it seems node-pty doesn't install on macOS
  • Loading branch information
suchipi committed Sep 5, 2022
1 parent 6e70b38 commit da0529c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ fi
popd > /dev/null

# grab JS dependencies from npm
npm install
docker run --rm -it -v $PWD:/opt/yavascript -w "/opt/yavascript" node:17.4.0 npm install

# generate dist/index.js (bundles in dependencies from npm)
mkdir -p dist
rm -rf dist
npx kame bundle --resolver ./src/kame-config.js --loader ./src/kame-config.js
docker run --rm -it -v $PWD:/opt/yavascript -w "/opt/yavascript" node:17.4.0 npx kame bundle --resolver ./src/kame-config.js --loader ./src/kame-config.js

# to make the stack traces clearer, we change the filename that will get baked into the binary:
mv dist/index.js ./yavascript-internal.js
Expand Down

0 comments on commit da0529c

Please sign in to comment.