Skip to content

Commit 7b6fc21

Browse files
committed
Ship with remote agent package.json
Instead of the root one. This contains fewer dependencies.
1 parent 9e7e006 commit 7b6fc21

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

ci/build/build-release.sh

+9-5
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ EOF
6666

6767
bundle_vscode() {
6868
mkdir -p "$VSCODE_OUT_PATH"
69-
rsync "$VSCODE_SRC_PATH/yarn.lock" "$VSCODE_OUT_PATH"
7069
rsync "$VSCODE_SRC_PATH/out-vscode-reh-web${MINIFY:+-min}/" "$VSCODE_OUT_PATH/out"
7170

7271
rsync "$VSCODE_SRC_PATH/.build/extensions/" "$VSCODE_OUT_PATH/extensions"
@@ -126,10 +125,15 @@ bundle_vscode() {
126125
EOF
127126
) > "$VSCODE_OUT_PATH/product.json"
128127

129-
# We remove the scripts field so that later on we can run
130-
# yarn to fetch node_modules if necessary without build scripts running.
131-
# We cannot use --no-scripts because we still want dependent package scripts to run.
132-
jq 'del(.scripts)' < "$VSCODE_SRC_PATH/package.json" > "$VSCODE_OUT_PATH/package.json"
128+
# Use the package.json for the web/remote server. It does not have the right
129+
# version though so pull that from the main package.json. Also remove keytar
130+
# since the web does not rely on it and that removes the dependency on
131+
# libsecret.
132+
jq --slurp '.[0] * {version: .[1].version} | del(.dependencies.keytar)' \
133+
"$VSCODE_SRC_PATH/remote/package.json" \
134+
"$VSCODE_SRC_PATH/package.json" > "$VSCODE_OUT_PATH/package.json"
135+
136+
rsync "$VSCODE_SRC_PATH/remote/yarn.lock" "$VSCODE_OUT_PATH/yarn.lock"
133137

134138
pushd "$VSCODE_OUT_PATH"
135139
symlink_asar

0 commit comments

Comments
 (0)