Refactor mupdf-sys build script (#136)

* Refactor mupdf-sys build script

* Only use sys-libs if requested

* Don't use cfg for msvc check

* Don't use "Small" with msbuild

* Disable system-fonts feature on wasm

* Fix wrong msbuild library search path

* Update msbuild search path

* Set OutDir on windows and create build dir

* Copy files to build directory

* Remove semicolon

* Fix linking issue and reapply NAN patch

* Use include_bytes to allow running tests in WASM

* Satisfy Rust 1.87 clippy

* Specify runner emscripten for CI

* Also include_bytes! data from serde tests

* Update actions/checkout to v4 to use node 20

* Use node 22

* Link to stdc++ when using tesseract

* Change c++ link flags

Author: ginnyTheCat

.github/workflows/CI.yml

@@ -17,7 +17,7 @@ jobs:
     name: Check
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: "recursive"
           fetch-depth: 500
@@ -33,30 +33,40 @@ jobs:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-2019, windows-2022, windows-2025]
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: "recursive"
           fetch-depth: 500
       - uses: dtolnay/rust-toolchain@stable
+
       - run: sudo apt-get -y install libfontconfig1-dev
         if: matrix.os == 'ubuntu-latest'
-      - name: Install LLVM
-        if: matrix.os == 'windows-2019'
-        run: choco install -y llvm
-      - name: Setup msbuild
-        if: matrix.os == 'windows-2019'
-        uses: microsoft/setup-msbuild@v1.3.1
-      - run: cargo test --features serde
-        timeout-minutes: 20
-        if: matrix.os == 'windows-2019'
-        env:
-          LIBCLANG_PATH: "C:\\Program Files\\LLVM\\bin"
+
       - run: cargo test --features serde
-        if: matrix.os != 'windows-2019'
+
       - name: Test package mupdf-sys
         if: matrix.os == 'ubuntu-latest'
         run: cargo package --manifest-path mupdf-sys/Cargo.toml
 
+  test-wasm:
+    name: Test Suite (Emscripten)
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          submodules: "recursive"
+          fetch-depth: 500
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          targets: wasm32-unknown-emscripten
+      - uses: mymindstorm/setup-emsdk@v14
+
+      - run: cargo test --target wasm32-unknown-emscripten --config "target.wasm32-unknown-emscripten.runner = 'node'" --features serde
+
   test-msys:
     name: Test Suite (MSYS2)
     runs-on: windows-2019
@@ -76,7 +86,7 @@ jobs:
           msystem: ${{matrix.sys}}
           install: mingw-w64-${{matrix.env}}-rust mingw-w64-${{matrix.env}}-clang base base-devel unzip git
 
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: "recursive"
           fetch-depth: 500
@@ -88,7 +98,7 @@ jobs:
     name: Address Sanitizer
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: "recursive"
           fetch-depth: 500
@@ -107,7 +117,7 @@ jobs:
     name: Valgrind
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: "recursive"
           fetch-depth: 500
@@ -122,7 +132,7 @@ jobs:
     name: Rustfmt
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
         with:
           components: rustfmt

Cargo.toml

@@ -58,16 +58,15 @@ libarchive = ["mupdf-sys/libarchive"]
 serde = ["dep:serde"]
 
 [dependencies]
-mupdf-sys = { version = "0.5.0", path = "mupdf-sys" }
+mupdf-sys = { version = "0.5.0", path = "mupdf-sys", features = ["zerocopy"] }
 once_cell = "1.3.1"
 num_enum = "0.7.0"
 bitflags = "2.0.2"
 serde = { version = "1.0.201", features = ["derive"], optional = true }
 zerocopy = { version = "0.8.17", features = ["derive"] }
 
-[dependencies.font-kit]
-version = "0.14.1"
-optional = true
+[target.'cfg(not(target_arch = "wasm32"))'.dependencies]
+font-kit = { version = "0.14.1", optional = true }
 
 [workspace]
 members = [

mupdf-sys/Cargo.toml

@@ -97,11 +97,14 @@ tesseract = []
 zxingcpp = []
 libarchive = []
 
+zerocopy = ["dep:zerocopy"]
+
 [build-dependencies]
 bindgen = { version = "0.71", default-features = false, features = ["runtime"] }
 cc = "1.0.50"
 pkg-config = "0.3"
 regex = "1.11"
 
 [dependencies]
-zerocopy = { version = "0.8.17", features = ["derive"] }
+zerocopy = { version = "0.8.17", features = ["derive"], optional = true }
+