Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump min node version to 18.20.3 #22010

Merged
merged 1 commit into from
May 29, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@ orbs:
executors:
linux-node:
docker:
- image: circleci/node:stretch
- image: cimg/node:18.20.3
linux-python:
docker:
- image: cimg/python:3.10.7
bionic:
docker:
- image: emscripten/emscripten-ci
- image: emscripten/emscripten-ci:focal
environment:
LANG: "C.UTF-8"
EMCC_CORES: "4"
Expand Down Expand Up @@ -41,7 +41,7 @@ commands:
name: download chrome
command: |
# TODO: Make these part of the base image
apt-get install libu2f-udev libvulkan1
apt-get install -q -y libu2f-udev libvulkan1 xdg-utils
# wget -O ~/chrome.deb https://dl.google.com/linux/direct/google-chrome-beta_current_amd64.deb
# If that download link breaks, temporarily use this URL instead:
# wget -O ~/chrome.deb https://storage.googleapis.com/webassembly/chrome/google-chrome-stable_current_amd64.deb
Expand Down Expand Up @@ -170,7 +170,7 @@ commands:
command: |
./emcc --clear-cache
- pip-install
- run: apt-get install ninja-build
- run: apt-get install -q -y ninja-build
- run:
name: embuilder build ALL
command: |
Expand Down Expand Up @@ -791,7 +791,7 @@ jobs:
environment:
EMTEST_SKIP_NODE_CANARY: "1"
steps:
- run: apt-get install ninja-build scons
- run: apt-get install -q -y ninja-build scons
- run-tests-linux:
# some native-dependent tests fail because of the lack of native
# headers on emsdk-bundled clang
Expand Down
1 change: 1 addition & 0 deletions requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ sphinxcontrib-htmlhelp<=2.0.0
sphinxcontrib-serializinghtml<=1.1.5
sphinxcontrib-qthelp<=1.0.3
alabaster<=0.7.12
pygments==2.17.2
# See https://github.com/readthedocs/readthedocs.org/issues/9038
jinja2<3.1

Expand Down
7 changes: 4 additions & 3 deletions site/source/docs/tools_reference/settings_reference.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2933,11 +2933,12 @@ MIN_NODE_VERSION

Specifies minimum node version to target for the generated code. This is
distinct from the minimum version required run the emscripten compiler.
This version aligns with the current Ubuuntu TLS 20.04 (Focal).
Version is encoded in MMmmVV, e.g. 181401 denotes Node 18.14.01.
This version aligns with the current Node LTS release, and the version
of node that is included with emsdk.
Version is encoded in MMmmVV, e.g. 181401 denotes Node 18.14.1.
Minimum supported value is 101900, which was released 2020-02-05.

Default value: 160000
Default value: 182003

.. _support_errno:

Expand Down
4 changes: 2 additions & 2 deletions src/parseTools.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -1059,8 +1059,8 @@ function getEntryFunction() {
}

function formattedMinNodeVersion() {
var major = MIN_NODE_VERSION / 10000;
var minor = (MIN_NODE_VERSION / 100) % 100;
var major = (MIN_NODE_VERSION / 10000) | 0;
var minor = ((MIN_NODE_VERSION / 100) | 0) % 100;
var rev = MIN_NODE_VERSION % 100;
return `v${major}.${minor}.${rev}`;
}
Expand Down
7 changes: 4 additions & 3 deletions src/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -1905,10 +1905,11 @@ var MIN_CHROME_VERSION = 85;

// Specifies minimum node version to target for the generated code. This is
// distinct from the minimum version required run the emscripten compiler.
// This version aligns with the current Ubuuntu TLS 20.04 (Focal).
// Version is encoded in MMmmVV, e.g. 181401 denotes Node 18.14.01.
// This version aligns with the current Node LTS release, and the version
// of node that is included with emsdk.
// Version is encoded in MMmmVV, e.g. 181401 denotes Node 18.14.1.
// Minimum supported value is 101900, which was released 2020-02-05.
var MIN_NODE_VERSION = 160000;
var MIN_NODE_VERSION = 182003;

// Whether we support setting errno from JS library code.
// In MINIMAL_RUNTIME builds, this option defaults to 0.
Expand Down
2 changes: 1 addition & 1 deletion test/other/metadce/test_metadce_hello_O0.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8137
8140
2 changes: 1 addition & 1 deletion test/other/metadce/test_metadce_hello_O0.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
22013
22014
2 changes: 1 addition & 1 deletion test/other/metadce/test_metadce_minimal_O0.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
6721
6725
2 changes: 1 addition & 1 deletion test/other/metadce/test_metadce_minimal_O0.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
18121
18122
2 changes: 1 addition & 1 deletion test/other/test_unoptimized_code_size.js.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
55520
55521
2 changes: 1 addition & 1 deletion test/other/test_unoptimized_code_size_strict.js.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
54383
54384
4 changes: 2 additions & 2 deletions test/test_sanity.py
Original file line number Diff line number Diff line change
Expand Up @@ -284,8 +284,8 @@ def test_node(self):
for version, succeed in [('v0.8.0', False),
('v4.1.0', False),
('v10.18.0', False),
('v16.20.0', True),
('v16.20.1-pre', True),
('v19.20.3', True),
('v19.20.4-pre', True),
('cheez', False)]:
print(version, succeed)
delete_file(SANITY_FILE)
Expand Down
2 changes: 1 addition & 1 deletion tools/shared.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
# (settings.MIN_NODE_VERSION).
# This version currently matches the node version that we ship with emsdk
# which means that we can say for sure that this version is well supported.
MINIMUM_NODE_VERSION = (16, 20, 0)
MINIMUM_NODE_VERSION = (18, 20, 3)
EXPECTED_LLVM_VERSION = 19

# These get set by setup_temp_dirs
Expand Down
Loading