From 0bb8f7269a1b70fbcd49c240559292bca3c7ce9e Mon Sep 17 00:00:00 2001
From: Roman Volosatovs <rvolosatovs@riseup.net>
Date: Wed, 18 Sep 2024 15:27:02 +0200
Subject: [PATCH] ci: use `rustup` on Windows

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
---
 .github/workflows/west.yml | 28 +++++++++++++++++++++++-----
 .gitignore                 |  1 +
 build_dev.go               |  3 +--
 3 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/west.yml b/.github/workflows/west.yml
index fa6c6c9..e72165c 100644
--- a/.github/workflows/west.yml
+++ b/.github/workflows/west.yml
@@ -27,11 +27,11 @@ jobs:
           - aarch64-unknown-linux-musl
           - aarch64-linux-android
           - riscv64gc-unknown-linux-gnu
-          - x86_64-pc-windows-gnu
           - x86_64-unknown-linux-musl
-          # TODO: figure out what's different in Mac libraries built this way
+          # TODO: figure out what's different in Mac/Windows libraries built this way
           #- aarch64-apple-darwin
           #- x86_64-apple-darwin
+          #- x86_64-pc-windows-gnu
 
     name: west-${{ matrix.target }}
     runs-on: ubuntu-latest
@@ -44,10 +44,14 @@ jobs:
         with:
           package: west-${{ matrix.target }}
 
-  build-ffi-darwin:
+  build-ffi-rustup:
     strategy:
       matrix:
         config:
+          - os: windows-latest
+            lib: x86_64-windows
+            target: x86_64-pc-windows-msvc
+
           - os: macos-13
             lib: x86_64-darwin
             target: x86_64-apple-darwin
@@ -65,6 +69,9 @@ jobs:
       - run: cargo build -p west-sys --release
       - run: mkdir -p artifact/lib
       - run: mv target/release/libwest_sys.a artifact/lib/libwest_sys.a
+        if: matrix.config.os != 'windows-latest'
+      - run: mv target/release/west_sys.lib artifact/lib/west_sys.lib
+        if: matrix.config.os == 'windows-latest'
       - uses: actions/upload-artifact@v4
         with:
           name: west-${{ matrix.config.target }}
@@ -96,7 +103,7 @@ jobs:
 
           - os: windows-latest
             lib: x86_64-windows
-            target: x86_64-pc-windows-gnu
+            target: x86_64-pc-windows-msvc
 
           - os: macos-13
             lib: x86_64-darwin
@@ -109,7 +116,7 @@ jobs:
     name: test-release (${{ matrix.config.os }})
     needs:
       - build-ffi
-      - build-ffi-darwin
+      - build-ffi-rustup
       - build-wasm
     runs-on: ${{ matrix.config.os }}
     steps:
@@ -122,6 +129,9 @@ jobs:
         with:
           name: west-${{ matrix.config.target }}
       - run: mv lib/libwest_sys.a "lib/${{ matrix.config.lib }}/libwest.a"
+        if: matrix.config.os != 'windows-latest'
+      - run: mv lib/west_sys.lib "lib/${{ matrix.config.lib }}/west.lib"
+        if: matrix.config.os == 'windows-latest'
       - uses: actions/setup-go@v5
         with:
           go-version-file: 'go.mod'
@@ -158,6 +168,14 @@ jobs:
       - run: cargo binstall -y wasm-tools@1.217
       - run: cargo test --workspace --all-targets
       - run: go generate -tags=dev ./...
+        if: matrix.os != 'windows-latest'
+      - run: |
+          cargo build -p west-sys
+          cargo build -p west-passthrough --target wasm32-unknown-unknown
+          go generate -tags=dev ./examples/go/...
+          go generate -tags=dev ./tests/go/...
+          wasm-tools component new target/wasm32-unknown-unknown/debug/west_passthrough.wasm -o lib/passthrough.wasm
+        if: matrix.os == 'windows-latest'
       - run: go test -tags=dev ./...
         env:
           GOGC: 1
diff --git a/.gitignore b/.gitignore
index 684e484..0f5250b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 /target
 *.wasm
 *.a
+*.lib
 west_bindings_test.go
 /go/internal/tests/*/bindings/*
diff --git a/build_dev.go b/build_dev.go
index c71c840..44032ae 100644
--- a/build_dev.go
+++ b/build_dev.go
@@ -6,6 +6,5 @@
 
 package west
 
-// #cgo       LDFLAGS: -L${SRCDIR}/target/debug -lwest_sys
-// #cgo linux LDFLAGS: -lm
+// #cgo LDFLAGS: -L${SRCDIR}/target/debug -lwest_sys
 import "C"