-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Add a /clean option to Restore.cmd #12293
Conversation
When present this option will clean out all of the NuGet caches before doing a restore. This is necessary to remove our hard coded dependencies on NuGet.exe in our official builds. Once they are removed we can switch to dowloading NuGet.exe on demand vs. checking it into the repo.
CC @jasonmalinowski, @tannergooding, @dotnet/roslyn-infrastructure |
set NuGetAdditionalCommandLineArgs=-verbosity quiet -configfile "%~dp0nuget.config" -Project2ProjectTimeOut 1200 | ||
set RoslynRoot=%~dp0 | ||
set NuGetExe="%RoslynRoot%\NuGet.exe" | ||
set NuGetAdditionalCommandLineArgs=-verbosity quiet -configfile "%RoslynRoot%\nuget.config" -Project2ProjectTimeOut 1200 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not your change, but any idea why we have a timeout for the P2P resolution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no idea. Maybe @agocke or @tannergooding know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no clue 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe @davkean knows
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because the default caused them to timeout on us.
👍 other than one small question, and that I now have to wash my eyes out after looking at cmd. |
test windows_eta_open_prtest please |
When present this option will clean out all of the NuGet caches before doing a restore.
This is necessary to remove our hard coded dependencies on NuGet.exe in our official builds. Once they are removed we can switch to dowloading NuGet.exe on demand vs. checking it into the repo.