Skip to content

Conversation

@ericstj
Copy link
Member

@ericstj ericstj commented Aug 20, 2024

This introduced a dependency on a newer runtime from build tools.

Partial revert from #106620.

The culprit here is MicrosoftDotNetILCompilerVersion - that seems to be bringing in ilc which runs as shared framework app against the toolchain runtime. Introduced in #100845 @elinor-fung @jkoritzinsky
The alternative would be to disable that functionality here.

…ld 20240819.2

This introduced a dependency on a newer runtime from build tools
@ericstj ericstj requested review from carlossanlop and lewing August 20, 2024 22:28
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 20, 2024
@ericstj ericstj added Servicing-approved Approved for servicing release area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Aug 20, 2024
@ericstj
Copy link
Member Author

ericstj commented Aug 20, 2024

Marking approved -- need this to unbreak the build.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@carlossanlop
Copy link
Contributor

I will fast merge as this is a revert.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure Servicing-approved Approved for servicing release

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants