Skip to content

Commit c0d535d

Browse files
committed
Use custom property instead of TargetPlatformIdentifier to avoid SDK conflicts
1 parent 3a181ac commit c0d535d

File tree

5 files changed

+11
-6
lines changed

5 files changed

+11
-6
lines changed

MultiTarget/MultiTargetIdentifiers.props

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,11 @@
4343

4444
<IsUwp Condition="'$(IsUwp)' == '' AND $(UwpTargetFramework.Contains('$(TargetFramework);')) AND '$(UwpTargetFramework)' != '' AND '$(MultiTargetsUwp)' == 'true'">true</IsUwp>
4545
<IsWinAppSdk Condition="'$(IsWinAppSdk)' == '' AND $(WinAppSdkTargetFramework.Contains('$(TargetFramework);')) AND '$(WinAppSdkTargetFramework)' != '' AND '$(MultiTargetsWasdk)' == 'true'">true</IsWinAppSdk>
46-
<TargetPlatformIdentifier>$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)'))</TargetPlatformIdentifier>
47-
<IsWindowsTfm Condition="$(TargetPlatformIdentifier) == 'windows'">true</IsWindowsTfm>
46+
47+
<MultiTargetPlatformIdentifier Condition="'$(MultiTargetPlatformIdentifier)' == ''">$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)'))</MultiTargetPlatformIdentifier>
48+
49+
<!-- TODO: Investigate. Should be named IsWindows, but ran into unexpected issues in VS. Property name may be taken and internally used by an SDK. -->
50+
<IsWindowsTfm Condition="$(MultiTargetPlatformIdentifier) == 'windows'">true</IsWindowsTfm>
4851

4952
<IsDroid Condition="'$(IsDroid)' == '' AND $(AndroidLibTargetFramework.Contains('$(TargetFramework);')) AND '$(AndroidLibTargetFramework)' != '' AND '$(MultiTargetsDroid)' == 'true'">true</IsDroid>
5053
<IsMacOS Condition="'$(IsMacOS)' == '' AND $(MacOSLibTargetFramework.Contains('$(TargetFramework);')) AND '$(MacOSLibTargetFramework)' != '' AND '$(MultiTargetsMacOS)' == 'true'">true</IsMacOS>
@@ -62,6 +65,6 @@
6265
<WinUIMajorVersion Condition="'$(HasWinUI)' == 'true' AND '$(IsUwp)' == 'true' AND $(WinUIMajorVersion) == ''">2</WinUIMajorVersion>
6366
<WinUIMajorVersion Condition="'$(HasWinUI)' == 'true' AND '$(IsWinAppSdk)' == 'true' AND $(WinUIMajorVersion) == ''">3</WinUIMajorVersion>
6467

65-
<UseUwp Condition="'$(HasWinUI)' == 'true' AND '$(IsUwp)' == 'true' AND '$(IsWindowsTfm)' == 'true'">true</UseUwp>
68+
<UseUwp Condition="'$(HasWinUI)' == 'true' AND '$(IsUwp)' == 'true' AND '$(MultiTargetPlatformIdentifier)' == 'windows'">true</UseUwp>
6669
</PropertyGroup>
6770
</Project>

MultiTarget/NoWarn.props

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<Project>
22
<!-- Disable warnings for 'This call site is reachable on all platforms.' caused by underlying platform. -->
3-
<PropertyGroup Condition="'$(IsUwp)' == 'true' AND '$(IsWindowsTfm)' == 'true'">
3+
<PropertyGroup Condition="'$(IsUwp)' == 'true' AND '$(MultiTargetPlatformIdentifier)' == 'windows'">
44
<NoWarn>$(NoWarn);CA1416;NETSDK1219;</NoWarn>
55
</PropertyGroup>
66

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
<Project>
22
<ItemGroup>
33
<PackageReference Condition="'$(HasWinUI)' == 'true'" Include="Microsoft.UI.Xaml" Version="2.8.7-prerelease.241119001" />
4-
<PackageReference Condition="'$(IsWindowsTfm)' == 'true'" Include="Microsoft.Windows.CsWinRT" Version="2.1.6" PrivateAssets="all" />
4+
<PackageReference Condition="'$(MultiTargetPlatformIdentifier)' == 'windows'" Include="Microsoft.Windows.CsWinRT" Version="2.1.6" PrivateAssets="all" />
55

66
<!-- Temp workaround until next Windows SDK servicing update -->
7-
<PackageReference Condition="'$(HasWinUI)' == 'true' AND '$(IsWindowsTfm)' == 'true'" Include="Microsoft.Windows.SDK.CPP" Version="10.0.26100.2454" PrivateAssets="all" />
7+
<PackageReference Condition="'$(HasWinUI)' == 'true' AND '$(MultiTargetPlatformIdentifier)' == 'windows'" Include="Microsoft.Windows.SDK.CPP" Version="10.0.26100.2454" PrivateAssets="all" />
88
</ItemGroup>
99
</Project>

ProjectHeads/Head.Uwp.props

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
<AppDesignerFolder>Properties</AppDesignerFolder>
1414
<DefaultLanguage>en-US</DefaultLanguage>
1515
<TargetPlatformIdentifier>UAP</TargetPlatformIdentifier>
16+
<MultiTargetPlatformIdentifier>$(TargetPlatformIdentifier)</MultiTargetPlatformIdentifier>
1617
<MinimumVisualStudioVersion>14</MinimumVisualStudioVersion>
1718
<FileAlignment>512</FileAlignment>
1819
<ProjectTypeGuids>{A5A43C5B-DE2A-4C0C-9213-0A381AF9435A};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>

ProjectHeads/Head.WinAppSdk.props

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
<Platforms>x86;x64;arm64</Platforms>
66
<PublishProfile>win-$(Platform).pubxml</PublishProfile>
77
<UseWinUI>true</UseWinUI>
8+
<MultiTargetPlatformIdentifier>windows</MultiTargetPlatformIdentifier>
89
<EnablePreviewMsixTooling>true</EnablePreviewMsixTooling>
910
<EnableMsixTooling>true</EnableMsixTooling>
1011
<WindowsAppContainer>true</WindowsAppContainer>

0 commit comments

Comments
 (0)