Skip to content

Always use live host #71725

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
agocke merged 21 commits into from
Jul 18, 2022
Merged

Always use live host #71725

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
eef9cf9
Always use live host
agocke Jul 6, 2022
7617e9f
Respond to PR comments
agocke Jul 7, 2022
fd4336b
Move host subset to the end
agocke Jul 7, 2022
3a1bbef
Copy over host DLLs to runtime artifacts
agocke Jul 8, 2022
5bf7ff4
Update eng/liveBuilds.targets
agocke Jul 11, 2022
723f2c2
Merge remote-tracking branch 'upstream/main' into live-host
agocke Jul 12, 2022
78e9ed6
Skip host files on mobile and remove duplicates
agocke Jul 12, 2022
c094df7
Don't include symbols in RuntimeFiles
agocke Jul 12, 2022
1c791ef
Port over comment
agocke Jul 12, 2022
cb2396b
Merge remote-tracking branch 'upstream/main' into live-host
agocke Jul 12, 2022
54a0f07
Only set up test host if the host partition was actually built
agocke Jul 13, 2022
679c016
Mark files as native
agocke Jul 13, 2022
b26c45f
Build host in libraries build
agocke Jul 15, 2022
6dfc65d
Merge remote-tracking branch 'upstream/main' into live-host
agocke Jul 15, 2022
11511ab
Use different CMake no cache command for older cmake
agocke Jul 15, 2022
a8eb2f7
Remove obsolete packages
agocke Jul 16, 2022
f6519af
Remove references to host pkgprojs
agocke Jul 16, 2022
914fead
Use a container for libraries test run jobs now that they need to bui…
agocke Jul 16, 2022
0600bbd
Merge remote-tracking branch 'upstream/main' into live-host
agocke Jul 16, 2022
9f70db0
Typo
agocke Jul 16, 2022
bf1415b
Add crossBuild parameter to libraries run-test-job
agocke Jul 17, 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
Next Next commit
Always use live host 
Eliminates package dependency on the host during build and test.
  • Loading branch information
@agocke
agocke committed Jul 6, 2022
commit eef9cf9616359083c5bd2130e552ea1dffd27eba
2 changes: 1 addition & 1 deletion eng/Subsets.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@
</PropertyGroup>

<PropertyGroup>
<DefaultCoreClrSubsets>clr.native+linuxdac+clr.corelib+clr.tools+clr.nativecorelib+clr.packages+clr.nativeaotlibs+clr.crossarchtools</DefaultCoreClrSubsets>
<DefaultCoreClrSubsets>clr.native+linuxdac+clr.corelib+clr.tools+clr.nativecorelib+clr.packages+clr.nativeaotlibs+clr.crossarchtools+host</DefaultCoreClrSubsets>
<!-- Even on platforms that do not support the CoreCLR runtime, we still want to build ilasm/ildasm. -->
<DefaultCoreClrSubsets Condition="'$(PrimaryRuntimeFlavor)' != 'CoreCLR'">clr.iltools+clr.packages</DefaultCoreClrSubsets>

Expand Down
8 changes: 0 additions & 8 deletions eng/Version.Details.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -214,14 +214,6 @@
<Uri>https://github.com/dotnet/runtime</Uri>
<Sha>74961633cef5f1ae4313ff9b16d1c22b22e96e6d</Sha>
</Dependency>
<Dependency Name="Microsoft.NETCore.DotNetHost" Version="7.0.0-preview.6.22354.1">
<Uri>https://github.com/dotnet/runtime</Uri>
<Sha>74961633cef5f1ae4313ff9b16d1c22b22e96e6d</Sha>
</Dependency>
<Dependency Name="Microsoft.NETCore.DotNetHostPolicy" Version="7.0.0-preview.6.22354.1">
<Uri>https://github.com/dotnet/runtime</Uri>
<Sha>74961633cef5f1ae4313ff9b16d1c22b22e96e6d</Sha>
</Dependency>
<Dependency Name="runtime.native.System.IO.Ports" Version="7.0.0-preview.6.22354.1">
<Uri>https://github.com/dotnet/runtime</Uri>
<Sha>74961633cef5f1ae4313ff9b16d1c22b22e96e6d</Sha>
Expand Down
2 changes: 0 additions & 2 deletions eng/Versions.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,6 @@
<NuGetBuildTasksPackVersion>6.0.0-preview.1.102</NuGetBuildTasksPackVersion>
<!-- Installer dependencies -->
<MicrosoftNETCoreAppRuntimewinx64Version>7.0.0-preview.6.22354.1</MicrosoftNETCoreAppRuntimewinx64Version>
<MicrosoftNETCoreDotNetHostVersion>7.0.0-preview.6.22354.1</MicrosoftNETCoreDotNetHostVersion>
<MicrosoftNETCoreDotNetHostPolicyVersion>7.0.0-preview.6.22354.1</MicrosoftNETCoreDotNetHostPolicyVersion>
<MicrosoftExtensionsDependencyModelVersion>3.1.0</MicrosoftExtensionsDependencyModelVersion>
<!-- CoreClr dependencies -->
<MicrosoftNETCoreILAsmVersion>7.0.0-preview.6.22354.1</MicrosoftNETCoreILAsmVersion>
Expand Down
20 changes: 3 additions & 17 deletions src/libraries/externals.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
<BinPlaceNative>true</BinPlaceNative>
<BinPlaceRuntime>false</BinPlaceRuntime>
<CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
<UseLiveBuiltDotNetHost Condition="'$(DotNetBuildFromSource)' == 'true' or '$(TargetArchitecture)' == 's390x' or '$(TargetArchitecture)' == 'ppc64le' or '$(TargetArchitecture)' == 'armv6' or '$(TargetsLinuxBionic)' == 'true'">true</UseLiveBuiltDotNetHost>
</PropertyGroup>

<PropertyGroup>
Expand All @@ -21,19 +20,12 @@
</PropertyGroup>

<Import Project="$(RepositoryEngineeringDir)coredistools.targets" Condition="$(CopyCoreDisToolsToCoreRoot)" />

<ItemGroup Condition="'$(DotNetBuildFromSource)' != 'true'">
<PackageReference Include="Microsoft.DiaSymReader.Native"
Version="$(MicrosoftDiaSymReaderNativeVersion)" />
</ItemGroup>

<ItemGroup Condition="'$(TargetsMobile)' != 'true' and '$(UseLiveBuiltDotNetHost)' != 'true'">
<PackageReference Include="Microsoft.NETCore.DotNetHost"
Version="$(MicrosoftNETCoreDotNetHostVersion)" />
<PackageReference Include="Microsoft.NETCore.DotNetHostPolicy"
Version="$(MicrosoftNETCoreDotNetHostPolicyVersion)" />
</ItemGroup>

<!-- Setup the testing shared framework host -->
<Target Name="SetupTestingHost"
AfterTargets="AfterResolveReferences"
Expand All @@ -48,13 +40,7 @@
<ReferenceCopyLocalPaths Remove="@(ReferenceCopyLocalPaths)" Condition="'%(Filename)' == 'apphost'" />
</ItemGroup>

<ItemGroup Condition="'$(UseLiveBuiltDotNetHost)' != 'true'">
<HostFxFile Include="@(ReferenceCopyLocalPaths)" Condition="'%(ReferenceCopyLocalPaths.Filename)' == 'hostfxr' or
'%(ReferenceCopyLocalPaths.Filename)' == 'libhostfxr'" />
<DotnetExe Include="@(ReferenceCopyLocalPaths)" Condition="'%(ReferenceCopyLocalPaths.Filename)' == 'dotnet'" />
</ItemGroup>

<ItemGroup Condition="'$(UseLiveBuiltDotNetHost)' == 'true'">
<ItemGroup>
<CoreHostFiles Include="$(DotNetHostBinDir)*" />
<HostFxFile Include="@(CoreHostFiles)" Condition="'%(CoreHostFiles.Filename)' == 'hostfxr' or
'%(CoreHostFiles.Filename)' == 'libhostfxr'" />
Expand All @@ -78,7 +64,7 @@
SkipUnchangedFiles="true"
UseHardlinksIfPossible="$(UseHardlink)" />

<Copy Condition="$(CopyCoreDisToolsToCoreRoot)"
<Copy Condition="$(CopyCoreDisToolsToCoreRoot)"
SourceFiles="$(CoreDisToolsLibrary)"
DestinationFolder="$(NetCoreAppCurrentTestHostPath)shared\Microsoft.NETCore.App\$(ProductVersion)"
SkipUnchangedFiles="true"
Expand Down