Add -s STRICT=1 flag emscripten build

mcbarton · mcbarton · commit ba47df0f3af6 · 2025-05-12T20:54:59.000+01:00
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -441,6 +441,8 @@ if(EMSCRIPTEN)
         PUBLIC "SHELL: --preload-file ${XEUS_CPP_DATA_DIR}@/share/xeus-cpp"
         PUBLIC "SHELL: --preload-file ${XEUS_CPP_CONF_DIR}@/etc/xeus-cpp"
         PUBLIC "SHELL: --post-js ${CMAKE_CURRENT_SOURCE_DIR}/wasm_patches/post.js"
+        PUBLIC "SHELL: -s STRICT=1"
+        PUBLIC "SHELL: --no-entry"
     )
     # TODO: Uncomment the above line regarding preloading clang's resource dir
     # once has been supported through cppinterop.
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -69,6 +69,7 @@ if(EMSCRIPTEN)
         PUBLIC "SHELL: --preload-file ${SYSROOT_PATH}/include@/include"
         PUBLIC "SHELL: --preload-file ../${XEUS_CPP_DATA_DIR}@/share/xeus-cpp"
         PUBLIC "SHELL: --preload-file ../${XEUS_CPP_CONF_DIR}@/etc/xeus-cpp"
+        PUBLIC "SHELL: -s STRICT=1"
     )
 
     target_include_directories(test_xeus_cpp PRIVATE ${XEUS_CPP_INCLUDE_DIR})

Original file line number	Diff line number	Diff line change
`@@ -441,6 +441,8 @@ if(EMSCRIPTEN)`
`441`	`441`	`PUBLIC "SHELL: --preload-file ${XEUS_CPP_DATA_DIR}@/share/xeus-cpp"`
`442`	`442`	`PUBLIC "SHELL: --preload-file ${XEUS_CPP_CONF_DIR}@/etc/xeus-cpp"`
`443`	`443`	`PUBLIC "SHELL: --post-js ${CMAKE_CURRENT_SOURCE_DIR}/wasm_patches/post.js"`
	`444`	`+ PUBLIC "SHELL: -s STRICT=1"`
	`445`	`+ PUBLIC "SHELL: --no-entry"`
`444`	`446`	`)`
`445`	`447`	`# TODO: Uncomment the above line regarding preloading clang's resource dir`
`446`	`448`	`# once has been supported through cppinterop.`
Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ if(EMSCRIPTEN)`
`69`	`69`	`PUBLIC "SHELL: --preload-file ${SYSROOT_PATH}/include@/include"`
`70`	`70`	`PUBLIC "SHELL: --preload-file ../${XEUS_CPP_DATA_DIR}@/share/xeus-cpp"`
`71`	`71`	`PUBLIC "SHELL: --preload-file ../${XEUS_CPP_CONF_DIR}@/etc/xeus-cpp"`
	`72`	`+ PUBLIC "SHELL: -s STRICT=1"`
`72`	`73`	`)`
`73`	`74`
`74`	`75`	`target_include_directories(test_xeus_cpp PRIVATE ${XEUS_CPP_INCLUDE_DIR})`