Clarify why we need node_modules.asar

This commit is contained in:
Asher 2020-12-18 10:59:15 -06:00
parent 121a520447
commit 3aa4ae4539
No known key found for this signature in database
GPG Key ID: D63C1EF81242354A
2 changed files with 8 additions and 6 deletions

View File

@ -42,14 +42,11 @@ vscode_yarn() {
cd lib/vscode
yarn --production --frozen-lockfile
# VS Code needs a node_modules.asar but that's just a duplicate of stuff we
# already have in node_modules.
# This is a copy of symlink_asar in ../lib.sh. Look there for details.
if [ ! -e node_modules.asar ]; then
if [ "${WINDIR-}" ]; then
# mklink takes the link name first.
mklink /J node_modules.asar node_modules
else
# ln takes the link name second.
ln -s node_modules node_modules.asar
fi
fi

View File

@ -95,8 +95,13 @@ export OS
# Defaults to release
RELEASE_PATH="${RELEASE_PATH-release}"
# Symlink node_modules.asar to node_modules. VS Code needs a node_modules.asar
# but that's just a duplicate of stuff we already have in node_modules.
# VS Code bundles some modules into an asar which is an archive format that
# works like tar. It then seems to get unpacked into node_modules.asar.
#
# I don't know why they do this but all the dependencies they bundle already
# exist in node_modules so just symlink it. We have to do this since not only VS
# Code itself but also extensions will look specifically in this directory for
# files (like the ripgrep binary or the oniguruma wasm).
symlink_asar() {
if [ ! -e node_modules.asar ]; then
if [ "${WINDIR-}" ]; then