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.

Sign up for GitHub

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

Jump to bottom

nuget.exe install should have DisableParallelProcessing on mono #5741

Closed
emgarten opened this issue Aug 10, 2017 · 0 comments · Fixed by NuGet/NuGet.Client#1634
Closed

nuget.exe install should have DisableParallelProcessing on mono #5741

emgarten opened this issue Aug 10, 2017 · 0 comments · Fixed by NuGet/NuGet.Client#1634
Assignees
@emgarten
Labels
Functionality:Install The install command in VS/nuget.exe Product:NuGet.exe NuGet.exe Type:Bug
Milestone
4.4

Comments

@emgarten
Copy link
Member

The current nuget.exe install code disables parallel too early, later it is overridden and set to false. The end result is that on mono nuget.exe install runs in parallel.

https://github.com/NuGet/NuGet.Client/blob/74f7d7557c089431e10ce55d0dce4cab8c36a54f/src/NuGet.Clients/NuGet.CommandLine/Commands/InstallCommand.cs#L56
@emgarten emgarten added this to the 4.4 milestone Aug 10, 2017
@emgarten emgarten self-assigned this Aug 10, 2017
@emgarten emgarten added the Functionality:Install The install command in VS/nuget.exe label Aug 10, 2017
@emgarten emgarten mentioned this issue Aug 11, 2017
Merged
emgarten added a commit to NuGet/NuGet.Client that referenced this issue Aug 12, 2017
@emgarten
Improvements for nuget.exe install
8614255 
* Allow updating packages when -ExcludeVersion is used
* Improved detection of already installed packages when -ExcludeVersion is used
* Adding -Framework switch to allow setting the target framework used when resolving dependencies.
* Remove NU1000 code from packages.config restore errors.
* Avoid unneeded downloads when a version is not given and the package is cached.
* Disable parallel for mono
* Display errors and return a non-zero exit when install on packages.config fails
* Remove old files during upgrades with -ExcludeVersion

Fixes NuGet/Home#5743
Fixes NuGet/Home#5737
Fixes NuGet/Home#5736
Fixes NuGet/Home#5741
Fixes NuGet/Home#5017
Fixes NuGet/Home#3957
Fixes NuGet/Home#2405
emgarten added a commit to NuGet/NuGet.Client that referenced this issue Aug 12, 2017
@emgarten
Improvements for nuget.exe install
5b6edbb 
* Allow updating packages when -ExcludeVersion is used
* Improved detection of already installed packages when -ExcludeVersion is used
* Adding -Framework switch to allow setting the target framework used when resolving dependencies.
* Remove NU1000 code from packages.config restore errors.
* Avoid unneeded downloads when a version is not given and the package is cached.
* Disable parallel for mono
* Display errors and return a non-zero exit when install on packages.config fails
* Remove old files during upgrades with -ExcludeVersion

Fixes NuGet/Home#5743
Fixes NuGet/Home#5737
Fixes NuGet/Home#5736
Fixes NuGet/Home#5741
Fixes NuGet/Home#5017
Fixes NuGet/Home#3957
Fixes NuGet/Home#2405
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emgarten emgarten

Labels
Functionality:Install The install command in VS/nuget.exe Product:NuGet.exe NuGet.exe Type:Bug
Projects
None yet
Milestone
4.4
Development

Successfully merging a pull request may close this issue.

Improvements for nuget.exe install NuGet/NuGet.Client
1 participant
@emgarten