From 5e0fb1d0334b2d2811cfec202ab3366a75ae15ee Mon Sep 17 00:00:00 2001
From: Ismo Vuorinen <ismo@ivuorinen.net>
Date: Wed, 19 Nov 2025 00:37:27 +0200
Subject: [PATCH] fix: conditionally include iOS/macOS targets only on macOS

- Add MSBuild condition to only include net8.0-ios and net8.0-maccatalyst on macOS
- On Linux/Windows, only net8.0 is included as a target framework
- Fixes GitHub dependency submission failing on Ubuntu runners
- Remove wasm-tools installation from test workflow (no longer needed)

This fixes the root cause of NETSDK1147 errors on non-macOS platforms.
---
 .github/workflows/test.yml     | 3 ---
 src/HihaArvio/HihaArvio.csproj | 5 ++++-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 985c89f..f834829 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -24,9 +24,6 @@ jobs:
       with:
         dotnet-version: '8.0.x'
 
-    - name: Install WebAssembly Tools workload
-      run: dotnet workload install wasm-tools-net8
-
     - name: Restore dependencies
       run: dotnet restore tests/HihaArvio.Tests/HihaArvio.Tests.csproj /p:TargetFrameworks=net8.0
 
diff --git a/src/HihaArvio/HihaArvio.csproj b/src/HihaArvio/HihaArvio.csproj
index b3b2e24..0a17e15 100644
--- a/src/HihaArvio/HihaArvio.csproj
+++ b/src/HihaArvio/HihaArvio.csproj
@@ -3,7 +3,10 @@
 	<PropertyGroup>
 		<!-- Per spec: iOS (primary), Web (Blazor), macOS (tertiary) -->
 		<!-- net8.0 included for unit testing without platform dependencies -->
-		<TargetFrameworks>net8.0;net8.0-ios;net8.0-maccatalyst</TargetFrameworks>
+		<!-- Include iOS/macOS targets only on macOS (where they can be built) -->
+		<TargetFrameworks Condition="$([MSBuild]::IsOSPlatform('OSX'))">net8.0;net8.0-ios;net8.0-maccatalyst</TargetFrameworks>
+		<!-- On Linux/Windows, only include net8.0 -->
+		<TargetFrameworks Condition="!$([MSBuild]::IsOSPlatform('OSX'))">net8.0</TargetFrameworks>
 
 		<!-- Note for MacCatalyst:
 		The default runtime is maccatalyst-x64, except in Release config, in which case the default is maccatalyst-x64;maccatalyst-arm64.