Skip to content
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

!temporary! Increase dotnet-install logging #27972

Merged
merged 1 commit into from
Nov 19, 2020

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented Nov 19, 2020

  • changes may help figure out dotnet/core-eng#11424
  • will likely go away in the next Arcade update PR
    • if not, we should revert this change after dotnet/core-eng#11424 is done for realz

FYI @MattGal I didn't find a way to add similar logging in tools.sh. Tried

  • adding chmod +x "$install_script" in GetDotNetInstallScript
  • changing commands to bash "verbose=true \"$install_script\" ..." in InstallDotNet

but that didn't get the say_verbose or eval $invocation bits in the script going.

- changes may help figure out dotnet/core-eng#11424
- will likely go away in the next Arcade update PR
  - if not, we should revert this change after dotnet/core-eng#11424 is done for realz

FYI @MattGal I didn't find a way to add similar logging in tools.sh. Tried
- adding `chmod +x "$install_script"` in `GetDotNetInstallScript`
- changing commands to `bash "verbose=true \"$install_script\" ..."` in `InstallDotNet`
but that didn't get the `say_verbose` or `eval $invocation` bits in the script going.
@dougbu dougbu requested review from captainsafia and a team November 19, 2020 00:35
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Nov 19, 2020
@dougbu
Copy link
Member Author

dougbu commented Nov 19, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@dougbu dougbu merged commit 8b254e7 into master Nov 19, 2020
@dougbu dougbu deleted the dougbu/verbose.verbosity.of.the.install.scripts branch November 19, 2020 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants