strip by default for non windows platform

Author: bobzhang

scripts/install.js

@@ -357,7 +357,8 @@ function provideCompiler() {
     var releaseNinja = require("./ninjaFactory.js").libNinja({
       ocamlopt: "ocamlopt.opt",
       ext: ".exe",
-      INCL: ocamlVersion
+      INCL: ocamlVersion,
+      isWin : is_windows
     });
 
     var filePath = path.join(lib_dir, "release.ninja");

scripts/ninjaFactory.js

@@ -4,7 +4,7 @@
 //@ts-check
 /**
  * 
- * @param {{ocamlopt : string ; ext : string ; INCL : string}} config 
+ * @param {{ocamlopt : string ; ext : string ; INCL : string, isWin : boolean}} config 
  * 
  */
 function libNinja(config){
@@ -14,7 +14,7 @@ ext = ${config.ext}
 INCL = ${config.INCL}
 flags = -I $INCL -g -w -a ../jscomp/stubs/ext_basic_hash_stubs.c
 rule cc
-    command = $ocamlopt $flags $in -o $out
+    command = $ocamlopt $flags $in -o $out ${config.isWin? '' : '&& strip $out'}
     description = Making $out
 # -inline 1000 makes size too large
 # TODO: make sure it can be bootstrapped, at least is a very good

scripts/prebuilt.js

@@ -21,7 +21,8 @@ function buildCompiler() {
   var content = require('./ninjaFactory.js').libNinja({
     ocamlopt : is_windows?`ocamlopt.opt.exe`:`../native/${ocamlVersion}/bin/ocamlopt.opt`,
     ext : sys_extension,
-    INCL : ocamlVersion
+    INCL : ocamlVersion,
+    isWin : is_windows
   })
 
   fs.writeFileSync(path.join(root,'lib',prebuilt),content,'ascii')