making sure to copy all necessary files to dist based on known build …

…target
catdad-experiments · Nov 3, 2023 · f506f1e · f506f1e
1 parent ced8945
commit f506f1e
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 8 deletions.
diff --git a/.github/workflows/emscripten.yml b/.github/workflows/emscripten.yml
@@ -39,7 +39,7 @@ jobs:
         ./scripts/run-ci-wasm.sh
         ls -la
     - name: Dist prep
-      run: ./dist-prep.sh
+      run: ./dist-prep.sh ${{ matrix.target }}
     - name: Artifacts
       uses: actions/upload-artifact@v3
       with:

diff --git a/dist-prep.sh b/dist-prep.sh
@@ -1,6 +1,8 @@
 #!/bin/bash
 set -e
 
+build_target="$1"
+
 echoerr() { echo "$@" 1>&2; }
 
 function assertFile {
@@ -15,13 +17,33 @@ rm -rf dist/*
 
 ls -la libheif
 
-cp libheif/libheif.js dist/libheif.js
-cp libheif/COPYING dist/LICENSE
-cp libheif/libheif.wasm dist/libheif.wasm || echo wasm not found
+function copyJs() {
+  cp libheif/libheif.js dist/libheif.js
+  cp libheif/COPYING dist/LICENSE
+
+  assertFile dist/libheif.js
+  assertFile dist/LICENSE
+
+  chown $(whoami) dist/libheif.js dist/LICENSE
+}
+
+function copyWasm() {
+  copyJs
+
+  cp libheif/libheif.wasm dist/libheif.wasm
+  assertFile dist/libheif.wasm
+  chown $(whoami) dist/libheif.wasm
+}
 
-assertFile dist/libheif.js
-assertFile dist/LICENSE
+if [ "$build_target" = "js" ]
+then
+  copyJs
+elif [ "$build_target" = "wasm" ]
+then
+  copyWasm
+else
+  echo "unknown build target: $build_target"
+  exit 1
+fi
 
-chown $(whoami) dist/libheif.js dist/LICENSE
-chown $(whoami) dist/libheif.wasm || echo wasm not found
 ls -la dist