Building clr shouldn't build host tests

[eerhardt]

Since #73095, we are now building the host subset when building clr. However, the host subset includes a few test directories:

runtime/eng/Subsets.props

Line 73 in f244adb

<DefaultHostSubsets Condition="'$(DotNetBuildFromSource)' != 'true'">$(DefaultHostSubsets)+host.tests</DefaultHostSubsets>

and

runtime/src/native/corehost/CMakeLists.txt

Line 34 in f244adb

add_subdirectory(test)

We should refactor our build so we don't need to build these tests when running a "normal" dev build: .\build.cmd libs+clr -rc Release.

[ghost]

Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Since #73095, we are now building the host subset when building clr. However, the host subset includes a few test directories:

runtime/eng/Subsets.props

Line 73 in f244adb

<DefaultHostSubsets Condition="'$(DotNetBuildFromSource)' != 'true'">$(DefaultHostSubsets)+host.tests</DefaultHostSubsets>

and

runtime/src/native/corehost/CMakeLists.txt

Line 34 in f244adb

add_subdirectory(test)

We should refactor our build so we don't need to build these tests when running a "normal" dev build: .\build.cmd libs+clr -rc Release.

Author:	eerhardt
Assignees:	-
Labels:	area-Host
Milestone:	-

[elinor-fung]

Should we make clr include host.native instead of host? If it is just being included to use the live host, I don't think we need host packages or tools either.

[am11]

Yes, host.native sounds like the right subset for clr and mono defaults. Microsoft.NET.HostModel.dll is loaded when dotnet publish is used with --runtime argument (with or without -p:Publish{Aot,SingleFile,ReadyToRun}). We only use live host's dotnet publish in the installer tests.

Ideally, if we are building all (or enough) subsets, then crossgen2 build (and anything else which uses dotnet publish from global sdk during the build) can be deferred after the $RuntimeFlavor+host+libs, and then build.{cmd,sh} can handover control from runtime/.dotnet/dotnet to live host in order to increase reliance on live build of corehost and HostModel.

[elinor-fung]

Switched to only include host.native in #73800.

The native tests assets are part of host.native though, so they will still be built.