rebase changes of 8.3.1

Author: bobzhang

.npmignore

@@ -38,4 +38,6 @@ package/
 .nyc_output/
 themes
 vendor/js_of_ocaml.bc
-syntax/
+syntax/
+*.log
+ninja

Changes.md

@@ -1,5 +1,13 @@
 `*` means  potential break changes
 
+# 8.3.1
+This is a minor bug fix release for 8.3.0
+- capture warnings when rebuild without enforce warn-as-error
+- #4716 internal, make ninja a submodule in dev process
+- #4722 better dataflow for cases like `let {a;b} as obj = ...`
+- no need call `caml_enter_blocking_section` for single threaded compiler
+- #4739 fix the interaction of exotic filenames like `[id]` with the build system.
+
 # 8.3
 
 - #4694, #4712 improving/customizing the underlying ninja build system, better performance  

darwin/ninja.exe

@@ -198,7 +198,6 @@
     "map.js",
     "marshal.js",
     "moreLabels.js",
-    "nativeint.js",
     "node_buffer.js",
     "node_child_process.js",
     "node_fs.js",
@@ -364,7 +363,6 @@
     "map.js",
     "marshal.js",
     "moreLabels.js",
-    "nativeint.js",
     "node_buffer.js",
     "node_child_process.js",
     "node_fs.js",
@@ -488,7 +486,6 @@
     "map.cmi",
     "marshal.cmi",
     "moreLabels.cmi",
-    "nativeint.cmi",
     "node_buffer.cmi",
     "node_child_process.cmi",
     "node_fs.cmi",
@@ -619,7 +616,6 @@
     "map.cmt",
     "marshal.cmt",
     "moreLabels.cmt",
-    "nativeint.cmt",
     "node_buffer.cmt",
     "node_child_process.cmt",
     "node_fs.cmt",
@@ -724,7 +720,6 @@
     "map.cmti",
     "marshal.cmti",
     "moreLabels.cmti",
-    "nativeint.cmti",
     "node_process.cmti",
     "obj.cmti",
     "parsing.cmti",
@@ -857,7 +852,6 @@
     "mapm.cppo.ml",
     "marshal.ml",
     "moreLabels.ml",
-    "nativeint.ml",
     "node_buffer.ml",
     "node_child_process.ml",
     "node_fs.ml",
@@ -969,7 +963,6 @@
     "mapm.cppo.mli",
     "marshal.mli",
     "moreLabels.mli",
-    "nativeint.mli",
     "node_process.mli",
     "obj.mli",
     "parsing.mli",

jscomp/artifacts.json

@@ -52,19 +52,19 @@ if (!is_windows) {
 function createOCamlTar() {
   if (process.platform === hostPlatform) {
     require("./installUtils.js").install();
-    console.log(`status in ocaml submodule:`)
+    console.log(`status in ocaml submodule:`);
     cp.execSync(`git -C ocaml status -uno`, { cwd: root, stdio: [0, 1, 2] });
     cp.execSync(
       `git  -C ocaml archive --format=tar.gz HEAD -o ../vendor/ocaml.tar.gz`,
       { cwd: root, stdio: [0, 1, 2] }
     );
+    console.log(`status in ninja submodule:`);
+    cp.execSync(`git -C ninja status -uno`, { cwd: root, stdio: [0, 1, 2] });
+    cp.execSync(
+      `git -C ninja archive --format=tar.gz HEAD -o ../vendor/ninja.tar.gz`,
+      { cwd: root, stdio: [0, 1, 2] }
+    );
   }
-  console.log(`status in ninja submodule:`)
-  cp.execSync(`git -C ninja status -uno`, { cwd: root, stdio: [0, 1, 2] });
-  cp.execSync(
-    `git -C ninja archive --format=tar.gz HEAD -o ../vendor/ninja.tar.gz`,
-    { cwd: root, stdio: [0, 1, 2] }
-  );
 }
 
 createOCamlTar();