未验证 提交 d4b3d21d 编写于 作者: A Anmol Sethi

Require minimum node 12

上级 40778b15
...@@ -41,6 +41,8 @@ platform or architecture. ...@@ -41,6 +41,8 @@ platform or architecture.
**note:** Installing via `npm` requires certain dependencies for the native module builds. **note:** Installing via `npm` requires certain dependencies for the native module builds.
See [./doc/npm.md](./doc/npm.md) for installing these dependencies. See [./doc/npm.md](./doc/npm.md) for installing these dependencies.
You also need at least node v12 installed. See [#1633](https://github.com/cdr/code-server/issues/1633).
```bash ```bash
npm install -g code-server npm install -g code-server
code-server code-server
......
...@@ -2,6 +2,16 @@ ...@@ -2,6 +2,16 @@
set -eu set -eu
main() { main() {
# Grabs the major version of node from $npm_config_user_agent which looks like
# yarn/1.21.1 npm/? node/v14.2.0 darwin x64
major_node_version=$(echo "$npm_config_user_agent" | sed -n 's/.*node\/v\([^.]*\).*/\1/p')
if [ "$major_node_version" -lt 12 ]; then
echo "code-server currently requires at least node v12"
echo "We have detected that you are on node v$major_node_version"
echo "See https://github.com/cdr/code-server/issues/1633"
exit 1
fi
case "${npm_config_user_agent-}" in npm*) case "${npm_config_user_agent-}" in npm*)
# We are running under npm. # We are running under npm.
if [ "${npm_config_unsafe_perm-}" != "true" ]; then if [ "${npm_config_unsafe_perm-}" != "true" ]; then
......
...@@ -11,7 +11,7 @@ main() { ...@@ -11,7 +11,7 @@ main() {
download_artifact npm-package ./release download_artifact npm-package ./release
# https://github.com/actions/upload-artifact/issues/38 # https://github.com/actions/upload-artifact/issues/38
chmod +x $(grep -rl '^#!/.\+' release) chmod +x $(grep -rl '^#!/.*' release)
yarn publish --non-interactive release yarn publish --non-interactive release
} }
......
...@@ -5,7 +5,7 @@ main() { ...@@ -5,7 +5,7 @@ main() {
cd "$(dirname "$0")/../.." cd "$(dirname "$0")/../.."
# https://github.com/actions/upload-artifact/issues/38 # https://github.com/actions/upload-artifact/issues/38
chmod +x $(grep -rl '^#!/.\+' release) chmod +x $(grep -rl '^#!/.*' release)
yarn release:static yarn release:static
yarn test:static-release yarn test:static-release
......
...@@ -9,7 +9,7 @@ Please refer to [VS Code's prerequisites](https://github.com/Microsoft/vscode/wi ...@@ -9,7 +9,7 @@ Please refer to [VS Code's prerequisites](https://github.com/Microsoft/vscode/wi
Differences: Differences:
- We are not constrained by node version - We require at least node v12 but later versions should work
- We use [fnpm](https://github.com/goreleaser/nfpm) to build .deb and .rpm packages - We use [fnpm](https://github.com/goreleaser/nfpm) to build .deb and .rpm packages
## Development Workflow ## Development Workflow
......
...@@ -3,6 +3,8 @@ ...@@ -3,6 +3,8 @@
If you're installing the npm module you'll need certain dependencies to build If you're installing the npm module you'll need certain dependencies to build
the native modules used by VS Code. the native modules used by VS Code.
You also need at least node v12 installed. See [#1633](https://github.com/cdr/code-server/issues/1633).
## Ubuntu, Debian ## Ubuntu, Debian
```bash ```bash
......
...@@ -92,5 +92,8 @@ ...@@ -92,5 +92,8 @@
"coder", "coder",
"vscode-remote", "vscode-remote",
"browser-ide" "browser-ide"
] ],
"engines": {
"node": ">= 12"
}
} }
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册