-
Notifications
You must be signed in to change notification settings - Fork 197
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.
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
dotnet: Set DOTNET_NOLOGO
when running external commands
#4515
Conversation
Just read it all and saw DOTNET_NOLOGO is what we want |
Yup - I was able to reproduce this by setting up fresh dev boxes, installing RC2 of .NET 9.0 and then running |
Per https://learn.microsoft.com/dotnet/core/tools/dotnet-environment-variables#dotnet_nologo: > Specifies whether .NET welcome and telemetry messages are displayed on the first run. Set to true to mute these messages (values `true`, `1`, or `yes` accepted) or set to false to allow them (values `false`, `0`, or `no` accepted). If not set, the default is `false` and the messages will be displayed on the first run. Since there are cases where we interpret the output of the `dotnet` CLI (for example, when calling `dotnet msbuild --getProperty:IsAspireHost`) we should set this value to ensure that this message is not printed, which would break our deserialization logic, which expects just JSON data to be printed to stdout. Fixes Azure#4513
08183dd
to
495cfdd
Compare
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash:
pwsh:
WindowsPowerShell install
MSI install
Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
Per https://learn.microsoft.com/dotnet/core/tools/dotnet-environment-variables#dotnet_nologo:
Since there are cases where we interpret the output of the
dotnet
CLI (for example, when callingdotnet msbuild --getProperty:IsAspireHost
) we should set this value to ensure that this message is not printed, which would break our deserialization logic, which expects just JSON data to be printed to stdout.Fixes #4513