Skip to content

Commit e1d79c0

Browse files
Include "ResolveTargetingPackAssets" in AfterTargets; Remove unused Targets
1 parent cf2daa4 commit e1d79c0

File tree

1 file changed

+4
-49
lines changed

1 file changed

+4
-49
lines changed

eng/WpfArcadeSdk/tools/SdkReferences.targets

Lines changed: 4 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -33,45 +33,18 @@
3333

3434
<PropertyGroup>
3535
<ResolveAssemblyReferencesDependsOn>
36-
ResolveMicrosoftNetCoreAppReferences;
37-
ResolveMicrosoftWindowsDesktopAppReferences;
3836
ResolveMicrosoftDotNetWpfGitHubReferences;
3937
ResolveWinFormsReferences;
4038
$(ResolveAssemblyReferencesDependsOn)
4139
</ResolveAssemblyReferencesDependsOn>
4240
</PropertyGroup>
4341

44-
<Target
45-
Name="ResolveMicrosoftNetCoreAppReferences"
46-
Returns="@(Reference)"
47-
Condition="'$(PkgMicrosoft_NETCore_App)'!='' And '@(NetCoreReference)'!='' and '$(TargetFrameworkIdentifier)' == '.NETCoreApp' And ('$(TargetFrameworkVersion)' == 'v3.0' Or '$(TargetFrameworkVersion)' == 'v3.1') and '$(DoNotLimitMicrosoftNetCoreReferences)'!='true'">
48-
<!--
49-
In your project, Add a references to assemblies from Microsoft.NETCore.App like this
50-
<ItemGroup>
51-
<NetCoreReference Include="netstandard" />
52-
<NetCoreReference Include="System.IO" />
53-
</ItemGroup>
54-
55-
It will get translated into something like this:
56-
<ItemGroup>
57-
<Reference Include="C:\Users\username\.nuget\packages\microsoft.netcore.app\3.0.0-preview5-27609-17\ref\netcoreapp3.0\netstandard.dll" />
58-
<Reference Include="C:\Users\username\.nuget\packages\microsoft.netcore.app\3.0.0-preview5-27609-17\ref\netcoreapp3.0\System.IO.dll" />
59-
</ItemGroup>
60-
-->
61-
<ItemGroup>
62-
<Reference Include="$(PkgMicrosoft_NETCore_App)\ref\$(TargetFramework)\%(NetCoreReference.Identity).dll"
63-
Condition="Exists('$(PkgMicrosoft_NETCore_App)\ref\$(TargetFramework)\%(NetCoreReference.Identity).dll')" >
64-
<NuGetPackageId>Microsoft.NetCore.App</NuGetPackageId>
65-
</Reference>
66-
</ItemGroup>
67-
</Target>
68-
6942

7043
<Target
7144
Name="LimitNetCoreAppReferences"
72-
AfterTargets="ResolveTargetingPacks"
45+
AfterTargets="ResolveTargetingPackAssets;ResolveTargetingPacks"
7346
Returns="@(Reference)"
74-
Condition="'$(PkgMicrosoft_NETCore_App)'!='' And '@(NetCoreReference)' != '' And '@(Reference)' != ''">
47+
Condition="'@(NetCoreReference)' != '' And '@(Reference)' != ''">
7548
<!--
7649
Example
7750
<NetCoreReference Include="Microsoft.CSharp" />
@@ -105,28 +78,10 @@
10578
</ItemGroup>
10679
</Target>
10780

108-
<Target Name="ResolveMicrosoftWindowsDesktopAppReferences"
109-
Returns="@(Reference)"
110-
Condition="'$(PkgMicrosoft_WindowsDesktop_App)'!='' And '@(WindowsDesktopAppReference)'!='' and '$(TargetFrameworkIdentifier)' == '.NETCoreApp' And ('$(TargetFrameworkVersion)' == 'v3.0' Or '$(TargetFrameworkVersion)' == 'v3.1') and '$(DoNotLimitWindowsDesktopAppReferences)'!='true'">
111-
<!--
112-
In your project, Add a references to assemblies from Microsoft.NETCore.App like this
113-
<ItemGroup>
114-
<WindowsDesktopAppeference Include="PresentationCore" />
115-
<WindowsDesktopAppeference Include="System.Xaml" />
116-
</ItemGroup>
117-
-->
118-
<ItemGroup>
119-
<Reference Include="$(PkgMicrosoft_WindowsDesktop_App)\ref\$(TargetFramework)\%(NetCoreReference.Identity).dll"
120-
Condition="Exists('$(PkgMicrosoft_WindowsDesktop_App)\ref\$(TargetFramework)\%(NetCoreReference.Identity).dll')">
121-
<NuGetPackageId>Microsoft.WindowsDesktop.App</NuGetPackageId>
122-
</Reference>
123-
</ItemGroup>
124-
</Target>
125-
12681
<Target Name="LimitWindowsDesktopAppReferences"
127-
AfterTargets="ResolveTargetingPacks"
82+
AfterTargets="ResolveTargetingPacks;ResolveTargetingPackAsssets"
12883
Returns="@(Reference)"
129-
Condition="'$(PkgMicrosoft_WindowsDesktop_App)'!='' And '@(WindowsDesktopReference)'!='' and '@(Reference)' != ''">
84+
Condition="'@(WindowsDesktopReference)'!='' and '@(Reference)' != ''">
13085
<!--
13186
Example
13287
<WindowsDesktopReference Include="PresentationCore" />

0 commit comments

Comments
 (0)