Skip to content

Commit

Permalink
Exclude System.* reference assemblies in ILCompiler.Build.Tasks (#86423)
Browse files Browse the repository at this point in the history
* Exclude System.* reference assemblies in ILCompiler.Build.Tasks

We expect the implementation of these assemblies to come as part of msbuild.

Fixes #83695

* Do not include any System.* assemblies as part of the task

* Delete custom resolver
  • Loading branch information
jkotas committed May 18, 2023
1 parent b25f2bb commit 29d485c
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 103 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

namespace Build.Tasks
{
public class ComputeManagedAssembliesToCompileToNative : DesktopCompatibleTask
public class ComputeManagedAssembliesToCompileToNative : Task
{
[Required]
public ITaskItem[] Assemblies
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,14 @@
using System.Text;

using Microsoft.Build.Framework;
using Microsoft.Build.Utilities;

namespace Build.Tasks
{
/// <summary>
/// Dumps native Win32 resources in the given assembly into a specified *.res file.
/// </summary>
public class DumpNativeResources : DesktopCompatibleTask
public class DumpNativeResources : Task
{
/// <summary>
/// File name of the assembly with Win32 resources to be dumped.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,11 @@
<OutputPath>$(RuntimeBinDir)/ilc-published/netstandard</OutputPath>
<Configurations>Debug;Release;Checked</Configurations>
<PlatformTarget>AnyCPU</PlatformTarget>
<CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Microsoft.Build.Framework" Version="$(MicrosoftBuildFrameworkVersion)" PrivateAssets="all" ExcludeAssets="runtime" />
<PackageReference Include="Microsoft.Build.Utilities.Core" Version="$(MicrosoftBuildUtilitiesCoreVersion)" PrivateAssets="all" ExcludeAssets="runtime" />
<PackageReference Include="Microsoft.Build.Framework" Version="$(MicrosoftBuildFrameworkVersion)" />
<PackageReference Include="Microsoft.Build.Utilities.Core" Version="$(MicrosoftBuildUtilitiesCoreVersion)" />
<PackageReference Include="System.Reflection.Metadata" Version="$(SystemReflectionMetadataVersion)" />
</ItemGroup>

Expand Down

0 comments on commit 29d485c

Please sign in to comment.