fix CI

tweak

fix CI

fix CI

no warn-error for js_compiler

fix ci

Author: bobzhang

native/.gitkeep

@@ -4,17 +4,14 @@ var cp = require("child_process");
 var path = require("path");
 var fs = require("fs");
 
-var ocamlSrcDir = path.join(__dirname, "..", "ocaml");
+var ocamlSrcDir = path.join(__dirname, "..", "native");
 var ocamlVersionFilePath = path.join(ocamlSrcDir, "VERSION");
 
 /**
  * Ensures the ocaml folder exists at the root of the project, either from the submodule,
  * or by extracting the vendor/ocaml.tar.gz there
  */
 function ensureOCamlExistsSync() {
-  if (!fs.existsSync(ocamlSrcDir)) {
-    fs.mkdirSync(ocamlSrcDir);
-  }
   if (!fs.existsSync(ocamlVersionFilePath)) {
     cp.execSync(`tar xzvf ../vendor/ocaml.tar.gz`, {
       cwd: ocamlSrcDir,

scripts/buildocaml.js

@@ -67,7 +67,7 @@ function runTests() {
   var binDir = path.join(__dirname, "..", "jscomp", "bin");
   if (ounitTest) {
     cp.execSync(
-      `ocamlc.opt -I . -c js_compiler.mli js_compiler.ml`,
+      `ocamlc.opt -warn-error -a -I . -c js_compiler.mli js_compiler.ml`,
       {
         cwd: path.join(__dirname, "..", "lib", "4.06.1", "unstable"),
         stdio: [0, 1, 2],

scripts/ciTest.js

@@ -76,14 +76,8 @@ exports.vendorNinjaPath = vendorNinjaPath;
  * Note ocamldep.opt has built-in macro handling OCAML_VERSION
  */
 var getOcamldepFile = () => {
-  return path.join(
-    __dirname,
-    "..",
-    "native",
-    require("./buildocaml.js").getVersionPrefix(),
-    "bin",
-    "ocamldep.opt"
-  );
+  return "ocamldep.opt"
+  
 };
 
 /**