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

Show file tree
Hide file tree
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
Prev Previous commit
Next Next commit
FB and working around running native aot tests
  • Loading branch information
@LakshanF
LakshanF committed Aug 7, 2022
commit e992c9bb86f6fbc9abb4bed4c5cce6cede733d93
2 changes: 1 addition & 1 deletion eng/testing/tests.singlefile.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
<SelfContained>true</SelfContained>
</PropertyGroup>

<Import Project="$(CoreCLRBuildIntegrationDir)Microsoft.DotNet.ILCompiler.targets" Condition="'$(TestNativeAot)' == 'true'" />
<Import Project="$(CoreCLRBuildIntegrationDir)Microsoft.DotNet.ILCompiler.SingleEntry.targets" Condition="'$(TestNativeAot)' == 'true'" />

<ItemGroup Condition="'$(TestNativeAot)' == 'true'">
<RdXmlFile Include="$(MSBuildThisFileDirectory)default.rd.xml" />
Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/nativeaot/BuildIntegration/BuildFrameworkNativeObjects.proj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
<DebugSymbols Condition="'$(DebugSymbols)' == ''">true</DebugSymbols>
</PropertyGroup>

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

<Target Name="BuildAllFrameworkLibraries"
Expand Down
27 changes: 12 additions & 15 deletions ...ation/Microsoft.DotNet.ILCompiler.targets → ...oft.DotNet.ILCompiler.SingleEntry.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,14 +1,11 @@
<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 -->
<OSIdentifier Condition="$(RuntimeIdentifier.StartsWith('win'))">win</OSIdentifier>
<OSIdentifier Condition="$(RuntimeIdentifier.StartsWith('osx'))">osx</OSIdentifier>
Expand All @@ -33,36 +30,36 @@

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


<IlcCalledViaPackage>true</IlcCalledViaPackage>
</PropertyGroup>

<PropertyGroup>

<!-- If the NativeAOT toolchain is being consumed via package, runtime-specific properties must be set before compilation can proceed -->
<ImportRuntimeIlcPackageTargetDependsOn>RunResolvePackageDependencies</ImportRuntimeIlcPackageTargetDependsOn>
<IlcSetupPropertiesDependsOn>ImportRuntimeIlcPackageTarget</IlcSetupPropertiesDependsOn>
<IlcDynamicBuildPropertyDependencies>SetupProperties</IlcDynamicBuildPropertyDependencies>

</PropertyGroup>

<!-- 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)"/>
<ILCompilerReference Include="@(PackageReference)" Condition="'%(Identity)'=='Microsoft.DotNet.ILCompiler'"/>
<OldKnownILCompilerPack Include="@(KnownILCompilerPack)"/>
<ILCompilerRefernece Include="@(PackageReference)" Condition="'%(Identity)'=='Microsoft.DotNet.ILCompiler'"/>
</ItemGroup>
<ItemGroup>
<KnownILCompilerPack Remove="@(OldKnownIlPack)" />
<KnownILCompilerPack Remove="@(OldKnownILCompilerPack )" Condition="@(ILCompilerReference->'%(Version)')!=''"/>
<KnownILCompilerPack Include="Microsoft.DotNet.ILCompiler"
TargetFramework="@(OldKnownIlPack->'%(TargetFramework)')"
ILCompilerPackNamePattern="@(OldKnownIlPack->'%(ILCompilerPackNamePattern)')"
ILCompilerPackVersion="@(ILCompilerReference->'%(Version)')"
ILCompilerRuntimeIdentifiers="@(OldKnownIlPack->'%(ILCompilerRuntimeIdentifiers)')"/>
TargetFramework="@(OldKnownILCompilerPack ->'%(TargetFramework)')"
ILCompilerPackNamePattern="@(OldKnownILCompilerPack ->'%(ILCompilerPackNamePattern)')"
ILCompilerPackVersion="@(ILCompilerRefernece->'%(Version)')"
ILCompilerRuntimeIdentifiers="@(OldKnownILCompilerPack ->'%(ILCompilerRuntimeIdentifiers)')"
Condition="@(ILCompilerReference->'%(Version)')!=''"/>
</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">
<Target Name="ImportRuntimeIlcPackageTarget" Condition="'$(BuildingFrameworkLibrary)' != 'true' and '$(PublishAot)' == 'true' and $(IlcCalledViaPackage) == 'true'" DependsOnTargets="$(ImportRuntimeIlcPackageTargetDependsOn)" BeforeTargets="Publish">
<Error Condition="'@(ResolvedILCompilerPack)' == ''" Text="The ResolvedILCompilerPack ItemGroup is required for target ImportRuntimeIlcPackageTarget" />

<PropertyGroup>
Expand All @@ -73,6 +70,6 @@

</Target>

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

</Project>
3 changes: 1 addition & 2 deletions src/coreclr/nativeaot/BuildIntegration/Microsoft.DotNet.ILCompiler.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,9 @@ Copyright (c) .NET Foundation. All rights reserved.
***********************************************************************************************
-->
<Project>

<PropertyGroup>
<!-- N.B. The ILCompilerTargetsPath is used as a sentinel to indicate a version of this file has already been imported. It will also be the path
used to import the targets later in the SDK. -->
<ILCompilerTargetsPath>$(MSBuildThisFileDirectory)Microsoft.DotNet.ILCompiler.targets</ILCompilerTargetsPath>
<ILCompilerTargetsPath>$(MSBuildThisFileDirectory)Microsoft.DotNet.ILCompiler.SingleEntry.targets</ILCompilerTargetsPath>
</PropertyGroup>
</Project>