From 685eb0d70ef7876e3d1984d047b24977eac0c2ae Mon Sep 17 00:00:00 2001
From: Matt Johnson-Pint <matt.johnson-pint@sentry.io>
Date: Sun, 23 Oct 2022 20:07:23 -0700
Subject: [PATCH] Build Windows and Tizen targets for `Sentry.Maui` (#2005)

---
 .github/actions/environment/action.yml | 10 +++++++++-
 CHANGELOG.md                           |  1 +
 Directory.Build.props                  |  1 +
 Directory.Build.targets                |  6 ++++++
 src/Sentry.Maui/Sentry.Maui.csproj     | 14 ++++++++++++--
 5 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/.github/actions/environment/action.yml b/.github/actions/environment/action.yml
index 27067f8656..d3a2afb7c5 100644
--- a/.github/actions/environment/action.yml
+++ b/.github/actions/environment/action.yml
@@ -43,6 +43,14 @@ runs:
       run: >
         dotnet workload install \
           maui-android \
-          ${{ runner.os == 'macOS' && 'maui-ios maui-maccatalyst' || '' }} \
+          ${{ runner.os == 'macOS' && 'maui-ios maui-maccatalyst maui-windows maui-tizen' || '' }} \
           ${{ runner.os == 'Windows' && 'maui-windows' || '' }} \
           --temp-dir "${{ runner.temp }}"
+
+    # We build Sentry.Maui for every supported MAUI target so we can access platform-specific features.
+    # That includes Tizen. We don't need the entire Tizen SDK, but we do need the base Tizen workload.
+    # Though we could install it on all runners, we really only need it when building and packing for NuGet, which we do on macOS only.
+    - name: Install Tizen Workload
+      if: runner.os == 'macOS'
+      shell: bash
+      run: "curl -sSL https://raw.githubusercontent.com/Samsung/Tizen.NET/main/workload/scripts/workload-install.sh | sudo bash"
diff --git a/CHANGELOG.md b/CHANGELOG.md
index f527c1f1fc..8534ca6db3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -9,6 +9,7 @@
 - Allow custom processors to be added as a scoped dependency ([#1979](https://github.com/getsentry/sentry-dotnet/pull/1979))
 - Support DI for custom transaction processors ([#1993](https://github.com/getsentry/sentry-dotnet/pull/1993))
 - Mark Transaction as aborted when unhandled exception occurs ([#1996](https://github.com/getsentry/sentry-dotnet/pull/1996))
+- Build Windows and Tizen targets for `Sentry.Maui` ([#2005](https://github.com/getsentry/sentry-dotnet/pull/2005))
 
 ### Fixes
 
diff --git a/Directory.Build.props b/Directory.Build.props
index 0f2e353be5..8ae6b78ee9 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -20,6 +20,7 @@
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'ios'">10.0</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'maccatalyst'">13.1</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'android'">21.0</SupportedOSPlatformVersion>
+    <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'tizen'">6.5</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'windows'">10.0.17763.0</SupportedOSPlatformVersion>
     <TargetPlatformMinVersion Condition="'$(TargetPlatformIdentifier)' == 'windows'">10.0.17763.0</TargetPlatformMinVersion>
   </PropertyGroup>
diff --git a/Directory.Build.targets b/Directory.Build.targets
index e05f722fb9..8db7436372 100644
--- a/Directory.Build.targets
+++ b/Directory.Build.targets
@@ -9,6 +9,7 @@
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'ios'">10.0</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'maccatalyst'">13.1</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'android'">21.0</SupportedOSPlatformVersion>
+    <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'tizen'">6.5</SupportedOSPlatformVersion>
     <SupportedOSPlatformVersion Condition="'$(TargetPlatformIdentifier)' == 'windows'">10.0.17763.0</SupportedOSPlatformVersion>
     <TargetPlatformMinVersion Condition="'$(TargetPlatformIdentifier)' == 'windows'">10.0.17763.0</TargetPlatformMinVersion>
   </PropertyGroup>
@@ -28,6 +29,11 @@
     <Compile Remove="**\Windows\**\*.cs" />
   </ItemGroup>
 
+  <ItemGroup Condition="'$(TargetPlatformIdentifier)' != 'tizen'">
+    <Compile Remove="**\*.Tizen.cs" />
+    <Compile Remove="**\Tizen\**\*.cs" />
+  </ItemGroup>
+
   <!-- Workaround the fact that http doesnt exist in earlier frameworks -->
   <ItemGroup>
     <Using Remove="System.Net.Http" />
diff --git a/src/Sentry.Maui/Sentry.Maui.csproj b/src/Sentry.Maui/Sentry.Maui.csproj
index 3692cc7fef..7c0e6d2032 100644
--- a/src/Sentry.Maui/Sentry.Maui.csproj
+++ b/src/Sentry.Maui/Sentry.Maui.csproj
@@ -3,11 +3,21 @@
   <PropertyGroup>
     <Description>Official MAUI integration for Sentry - Open-source error tracking that helps developers monitor and fix crashes in real time.</Description>
     <!--
-      Build for net6.0 to cover all managed platforms,
-      but also target platforms where we intend to bundle native SDKs.
+      Target net6.0 so we can run unit tests on platform-neutral code.
+      Target other platforms so we can include platform-specific code, and bundle native SDKs.
     -->
     <TargetFrameworks>net6.0;net6.0-android</TargetFrameworks>
     <TargetFrameworks Condition="$([MSBuild]::IsOSPlatform('OSX'))">$(TargetFrameworks);net6.0-ios;net6.0-maccatalyst</TargetFrameworks>
+    <TargetFrameworks Condition="$([MSBuild]::IsOSPlatform('Windows')) Or $([MSBuild]::IsOSPlatform('OSX'))">$(TargetFrameworks);net6.0-windows10.0.19041.0</TargetFrameworks>
+
+    <!-- Target Tizen only if the Tizen SDK workload pack is installed. -->
+    <TargetFrameworks Condition="Exists('$(MSBuildBinPath)\..\..\packs\Samsung.Tizen.Sdk')">$(TargetFrameworks);net6.0-tizen</TargetFrameworks>
+
+    <!--
+      This flag allows us to target Windows-specific code when building on OSX, so we can build and pack all platforms on a single machine.
+      See https://learn.microsoft.com/dotnet/core/tools/sdk-errors/netsdk1100
+    -->
+    <EnableWindowsTargeting Condition="$([MSBuild]::IsOSPlatform('OSX'))">true</EnableWindowsTargeting>
 
     <!-- We'll need to package this only on OSX, so we get the iOS native support. -->
     <IsPackable Condition="!$([MSBuild]::IsOSPlatform('OSX'))">false</IsPackable>