Skip to content

Conversation

jjonescz
Copy link
Member

Discovered while dogfooding a locally built SDK - when dotnet exec VBCSCompiler uses wrong dotnet.exe (e.g., the globally installed one instead of the locally built one), the compiler server launch fails (which means a 20 second timeout and fallback to MSBuild).

@jjonescz jjonescz added the Area-run-file Items related to the "dotnet run <file>" effort label Sep 13, 2025
Copy link
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good, but we probably need to do this in Program.cs as an explicit part of our startup, per some of the discussions we've been having with other teams. We already do this for MSBuild-based commands, it's time to expand the scope.

cc @dsplaisted @jaredpar

@jjonescz jjonescz marked this pull request as ready for review September 13, 2025 20:13
@jjonescz jjonescz requested a review from a team September 13, 2025 20:14
@RikkiGibson
Copy link
Member

@baronfel so are you saying you are happy with this solution for now but want to change it in the future?

@baronfel
Copy link
Member

baronfel commented Sep 15, 2025

yeah this is fine for now - but the whole CLI app needs to do this at a higher-level place, in part because of the whack-a-mole nature of doing it here and in umpteen other features.

@jjonescz jjonescz merged commit eacb57b into dotnet:release/10.0.1xx Sep 16, 2025
28 checks passed
@jjonescz jjonescz deleted the sprint-host-path branch September 16, 2025 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-run-file Items related to the "dotnet run <file>" effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants