Merge pull request #88 from ceifa/macos-build

fix macos build
ceifa · Aug 11, 2023 · 6785315 · 6785315
2 parents b4bd60d + b5091e2
commit 6785315
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -158,7 +158,7 @@ git submodule update --init # download lua submodule
 npm i # install dependencies
 ```
 
-### Windows / Linux (Docker way)
+### Windows / Linux / MacOS (Docker way)
 
 You need to install [docker](https://www.docker.com/) and ensure it is on your `PATH`.
 
@@ -170,7 +170,7 @@ npm run build # build the js code/bridge
 npm test # ensure everything it's working fine
 ```
 
-### Ubuntu / Debian
+### Ubuntu / Debian / MacOS
 
 You need to install [emscripten](https://emscripten.org/) and ensure it is on your `PATH`.
 

diff --git a/build.sh b/build.sh
@@ -12,7 +12,11 @@ else
     extension="$extension -O3 --closure 1"
 fi
 
-sed -i "s/^#define LUA_32BITS\t0$/#define LUA_32BITS\t1/" ./lua/luaconf.h
+if [[ "$OSTYPE" == "darwin"* ]]; then
+    sed -i '' "s/^#define LUA_32BITS\t0$/#define LUA_32BITS\t1/" ./lua/luaconf.h
+else
+    sed -i "s/^#define LUA_32BITS\t0$/#define LUA_32BITS\t1/" ./lua/luaconf.h
+fi
 
 emcc \
     -s WASM=1 $extension -o ./build/glue.js \
@@ -194,4 +198,8 @@ emcc \
     ]" \
     ${LUA_SRC}
 
-sed -i "s/^#define LUA_32BITS\t1$/#define LUA_32BITS\t0/" ./lua/luaconf.h
+if [[ "$OSTYPE" == "darwin"* ]]; then
+    sed -i '' "s/^#define LUA_32BITS\t1$/#define LUA_32BITS\t0/" ./lua/luaconf.h
+else
+    sed -i "s/^#define LUA_32BITS\t1$/#define LUA_32BITS\t0/" ./lua/luaconf.h
+fi