Skip to content

Clean up AOT publish process #73416

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
LakshanF merged 29 commits into from
Aug 15, 2022
Merged

Clean up AOT publish process #73416

Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
b9d18e1
Clean up AOT publish process
LakshanF Aug 4, 2022
62a302a
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 5, 2022
b5f37f0
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 5, 2022
3b86b66
Merge branch 'main' of https://github.com/dotnet/runtime into CleanPu…
LakshanF Aug 5, 2022
e992c9b
FB and working around running native aot tests
LakshanF Aug 7, 2022
dcf710c
update crossgen2 reference
LakshanF Aug 8, 2022
da529fd
moving package version change out of a target as per FB
LakshanF Aug 8, 2022
3cb58e6
Fix nativeaot test take 2
LakshanF Aug 9, 2022
5696f2d
disabling a failing test
LakshanF Aug 10, 2022
941d558
Clean up AOT publish process
LakshanF Aug 4, 2022
363fd28
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 5, 2022
48b0f74
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 5, 2022
c1fb84e
FB and working around running native aot tests
LakshanF Aug 7, 2022
eb13dc8
update crossgen2 reference
LakshanF Aug 8, 2022
294c463
moving package version change out of a target as per FB
LakshanF Aug 8, 2022
2a07fcc
Fix nativeaot test take 2
LakshanF Aug 9, 2022
8038c58
disabling a failing test
LakshanF Aug 10, 2022
45c21cb
Remove TrimMode workaround
LakshanF Aug 10, 2022
7867291
Merge branch 'CleanPublishAot' of https://github.com/LakshanF/runtime…
LakshanF Aug 11, 2022
5907416
work around a test issue
LakshanF Aug 11, 2022
b12795b
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 11, 2022
80cedc4
Update src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILComp…
LakshanF Aug 11, 2022
a52fcdd
Merge branch 'CleanPublishAot' of https://github.com/LakshanF/runtime…
LakshanF Aug 11, 2022
80faf94
update the version check for package
LakshanF Aug 12, 2022
141c61f
Cross target support, requires changes from SDK
LakshanF Aug 14, 2022
de74e2b
Change ResolvedTargetILCompilerPack to match SDK changes
LakshanF Aug 15, 2022
3d48f00
Merge branch 'main' into CleanPublishAot
LakshanF Aug 15, 2022
62ce9bd
FB
LakshanF Aug 15, 2022
ff574f8
Trigger Build to fix interop break
LakshanF Aug 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 27 additions & 16 deletions src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILCompiler.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">

<PropertyGroup>

<!-- Set the publishAot property to true if not set-->
<PublishAot Condition="'$(PublishAot)' == ''">true</PublishAot>
</PropertyGroup>


<PropertyGroup Condition="'$(RuntimeIdentifier)' != ''">

<!-- Define the name of the runtime specific compiler package to import -->
Expand All @@ -26,7 +33,6 @@

<IlcHostArch Condition="'$(IlcHostArch)' == ''">$(OSHostArch)</IlcHostArch>
<IlcHostPackageName>runtime.$(OSIdentifier)-$(IlcHostArch).Microsoft.DotNet.ILCompiler</IlcHostPackageName>
<IlcCalledViaPackage>true</IlcCalledViaPackage>

</PropertyGroup>

Expand All @@ -39,29 +45,34 @@

</PropertyGroup>

<!-- Locate the runtime package according to the current target runtime -->
<Target Name="ImportRuntimeIlcPackageTarget" Condition="'$(BuildingFrameworkLibrary)' != 'true' AND $(IlcCalledViaPackage) == 'true'" DependsOnTargets="$(ImportRuntimeIlcPackageTargetDependsOn)" BeforeTargets="Publish">
<!-- This targets file is imported by the SDK when the AotRuntimePackageLoadedViaSDK property is set. SDK resolves runtime package paths differently-->
<Error Condition="'@(ResolvedILCompilerPack)' == '' AND '$(AotRuntimePackageLoadedViaSDK)' == 'true'" Text="The ResolvedILCompilerPack ItemGroup is required for target ImportRuntimeIlcPackageTarget" />
<!-- NativeAOT runtime pack assemblies need to be defined to avoid the default CoreCLR implementations being set as compiler inputs -->
<Error Condition="'@(PackageDefinitions)' == '' AND '$(AotRuntimePackageLoadedViaSDK)' != 'true'" Text="The PackageDefinitions ItemGroup is required for target ImportRuntimeIlcPackageTarget" />
<!-- Target ProcessFrameworkReferences sets the details to download the runtime package but requires setting the version to the explicit package if one exists -->
<Target Name="SetILCompilerVersionForPackageReference" Condition="'$(AotRuntimePackageLoadedViaSDK)' != 'true' and '$(PublishAot)' == 'true'" BeforeTargets="ProcessFrameworkReferences">
<ItemGroup>
<OldKnownIlPack Include="@(KnownILCompilerPack)"/>
<ILCompilerRefernece Include="@(PackageReference)" Condition="'%(Identity)'=='Microsoft.DotNet.ILCompiler'"/>
</ItemGroup>
<ItemGroup>
<KnownILCompilerPack Remove="@(OldKnownIlPack)" />
<KnownILCompilerPack Include="Microsoft.DotNet.ILCompiler"
TargetFramework="@(OldKnownIlPack->'%(TargetFramework)')"
ILCompilerPackNamePattern="@(OldKnownIlPack->'%(ILCompilerPackNamePattern)')"
ILCompilerPackVersion="@(ILCompilerRefernece->'%(Version)')"
ILCompilerRuntimeIdentifiers="@(OldKnownIlPack->'%(ILCompilerRuntimeIdentifiers)')"/>
</ItemGroup>
</Target>

<!-- Locate the runtime package according to the current target runtime -->
<Target Name="ImportRuntimeIlcPackageTarget" Condition="'$(BuildingFrameworkLibrary)' != 'true' and '$(PublishAot)' == 'true'" DependsOnTargets="$(ImportRuntimeIlcPackageTargetDependsOn)" BeforeTargets="Publish">
<Error Condition="'@(ResolvedILCompilerPack)' == ''" Text="The ResolvedILCompilerPack ItemGroup is required for target ImportRuntimeIlcPackageTarget" />

<!-- This targets file is imported by the SDK when the AotRuntimePackageLoadedViaSDK property is set. Use the SDK runtime package resolve property to set down stream properties -->
<PropertyGroup Condition="'$(AotRuntimePackageLoadedViaSDK)' == 'true'">
<PropertyGroup>
<RuntimePackagePath>@(ResolvedILCompilerPack->'%(PackageDirectory)')</RuntimePackagePath>
<IlcHostPackagePath>@(ResolvedILCompilerPack->'%(PackageDirectory)')</IlcHostPackagePath>
<IlcPath>>@(ResolvedILCompilerPack->'%(PackageDirectory)')</IlcPath>
</PropertyGroup>

<!-- Use the non-SDK runtime package resolve property to set down stream properties if there is an explicit reference in the project -->
<PropertyGroup Condition="'$(AotRuntimePackageLoadedViaSDK)' != 'true'">
<RuntimePackagePath Condition="'%(PackageDefinitions.Name)' == '$(RuntimeIlcPackageName)'">%(PackageDefinitions.ResolvedPath)</RuntimePackagePath>
<IlcHostPackagePath Condition="'%(PackageDefinitions.Name)' == '$(IlcHostPackageName)'">%(PackageDefinitions.ResolvedPath)</IlcHostPackagePath>
</PropertyGroup>

</Target>

<Import Project="$(MSBuildThisFileDirectory)\Microsoft.NETCore.Native.targets" />
<Import Project="$(MSBuildThisFileDirectory)\Microsoft.NETCore.Native.targets" Condition="'$(PublishAot)' == 'true'"/>

</Project>