Add JSOO cmake patch

Author: phated

CMakeLists.txt

@@ -403,14 +403,36 @@ if(EMSCRIPTEN)
     # only valid with fastcomp and WASM=0
     target_link_libraries(binaryen_js "-s ELIMINATE_DUPLICATE_FUNCTIONS=1")
   endif()
-  target_link_libraries(binaryen_js "-s NO_FILESYSTEM=0")
+  # Disabling filesystem and setting web environment for js_of_ocaml
+  # so it doesn't try to detect the "node" environment
+  if(JS_OF_OCAML)
+    target_link_libraries(binaryen_js "-s NO_FILESYSTEM=1")
+    target_link_libraries(binaryen_js "-s ENVIRONMENT=web,worker")
+  else()
+    target_link_libraries(binaryen_js "-s NO_FILESYSTEM=0")
+  endif()
   target_link_libraries(binaryen_js "-s NODERAWFS=0")
   target_link_libraries(binaryen_js "-s EXPORT_NAME=Binaryen")
-  target_link_libraries(binaryen_js "-s EXPORT_ES6=1")
+  # Currently, js_of_ocaml can only process ES5 code
+  if(JS_OF_OCAML)
+    target_link_libraries(binaryen_js "-s EXPORT_ES6=0")
+  else()
+    target_link_libraries(binaryen_js "-s EXPORT_ES6=1")
+  endif()
   target_link_libraries(binaryen_js "--post-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-post.js")
-  target_link_libraries(binaryen_js "--extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-pre.js")
+  # js_of_ocaml needs a specified variable with special comment to provide the library to consumers
+  if(JS_OF_OCAML)
+    target_link_libraries(binaryen_js "--extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.jsoo-extern-pre.js")
+  else()
+    target_link_libraries(binaryen_js "--extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-pre.js")
+  endif()
   target_link_libraries(binaryen_js optimized "--closure 1")
-  target_link_libraries(binaryen_js optimized "--closure-args \"--language_in=ECMASCRIPT6 --language_out=ECMASCRIPT6\"")
+  # Currently, js_of_ocaml can only process ES5 code
+  if(JS_OF_OCAML)
+    target_link_libraries(binaryen_js optimized "--closure-args \"--language_in=ECMASCRIPT6 --language_out=ECMASCRIPT5\"")
+  else()
+    target_link_libraries(binaryen_js optimized "--closure-args \"--language_in=ECMASCRIPT6 --language_out=ECMASCRIPT6\"")
+  endif()
   target_link_libraries(binaryen_js optimized "-flto")
   target_link_libraries(binaryen_js debug "--profiling")
   target_link_libraries(binaryen_js debug "-s ASSERTIONS")

src/js/binaryen.jsoo-extern-pre.js

@@ -0,0 +1,6 @@
+// js_of_ocaml needs a special syntax for exposing variables to the compiler
+// and it is not built for nodejs so we don't need the __dirname hack
+
+//Provides: Binaryen
+var Binaryen = {};
+