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

Add a /clean option to Restore.cmd #12293

Merged
merged 1 commit into from
Jun 29, 2016
Merged

Add a /clean option to Restore.cmd #12293

merged 1 commit into from
Jun 29, 2016

Conversation

jaredpar
Copy link
Member

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.

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.
@jaredpar
Copy link
Member Author

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
Copy link
Member

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?

Copy link
Member Author

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.

Copy link
Member

Choose a reason for hiding this comment

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

no clue 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe @davkean knows

Copy link
Member

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.

@jasonmalinowski
Copy link
Member

👍 other than one small question, and that I now have to wash my eyes out after looking at cmd.

@tannergooding
Copy link
Member

test windows_eta_open_prtest please

@jaredpar jaredpar merged commit f8a6b35 into dotnet:master Jun 29, 2016
@jaredpar jaredpar deleted the nuget branch June 29, 2016 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants