Skip to content

Commit 30452cf

Browse files
authored
Fix Notifications library for C++ apps (#3423)
* Remove System.ValueTuple It was seemingly added for no reason with the ToastContentBuilder code: 9c38f71 Nothing there uses Tuple, so not sure why it was added... * Revert "Remove System.ValueTuple" This reverts commit 5ad4fa8. * Exclude ValueTuple from C++
1 parent 5bf4265 commit 30452cf

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Microsoft.Toolkit.Uwp.Notifications/Microsoft.Toolkit.Uwp.Notifications.csproj

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,8 @@
5252
<None Include="Microsoft.Toolkit.Uwp.Notifications.targets" Pack="true" PackagePath="build\native" />
5353
</ItemGroup>
5454

55-
<ItemGroup>
55+
<!--Native (C++) doesn't need System.ValueTuple (plus it's incompatible with this package)-->
56+
<ItemGroup Condition=" '$(TargetFramework)' != 'native' ">
5657
<PackageReference Include="System.ValueTuple" Version="4.5.0" />
5758
</ItemGroup>
5859

0 commit comments

Comments
 (0)