Bump min node version to 18.20.3

See emscripten-core/emsdk#1387 See emscripten-core/emsdk#1173
sbc100 · May 28, 2024 · e55c917 · e55c917
1 parent 182c4d8
commit e55c917
Show file tree

Hide file tree

Showing 11 changed files with 19 additions and 17 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -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"

diff --git a/site/source/docs/tools_reference/settings_reference.rst b/site/source/docs/tools_reference/settings_reference.rst
@@ -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 including 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:
 

diff --git a/src/parseTools.mjs b/src/parseTools.mjs
@@ -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}`;
 }

diff --git a/src/settings.js b/src/settings.js
@@ -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 including 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.

diff --git a/test/other/metadce/test_metadce_hello_O0.gzsize b/test/other/metadce/test_metadce_hello_O0.gzsize
@@ -1 +1 @@
-8137
+8140
diff --git a/test/other/metadce/test_metadce_hello_O0.jssize b/test/other/metadce/test_metadce_hello_O0.jssize
@@ -1 +1 @@
-22013
+22014
diff --git a/test/other/metadce/test_metadce_minimal_O0.gzsize b/test/other/metadce/test_metadce_minimal_O0.gzsize
@@ -1 +1 @@
-6721
+6725
diff --git a/test/other/metadce/test_metadce_minimal_O0.jssize b/test/other/metadce/test_metadce_minimal_O0.jssize
@@ -1 +1 @@
-18121
+18122
diff --git a/test/other/test_unoptimized_code_size.js.size b/test/other/test_unoptimized_code_size.js.size
@@ -1 +1 @@
-55520
+55521
diff --git a/test/other/test_unoptimized_code_size_strict.js.size b/test/other/test_unoptimized_code_size_strict.js.size
@@ -1 +1 @@
-54383
+54384
diff --git a/tools/shared.py b/tools/shared.py
@@ -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