search

chocolatey / cChoco

Community resource to manage Chocolatey
Apache License 2.0
154 stars 99 forks source link

Turn off download progress #83

Closed ferventcoder closed 5 years ago

ferventcoder commented 7 years ago

In 0.10.4, Chocolatey started to show download progress for packages as well as any downloaded resources. When something else is using Chocolatey and capturing output for logging purposes, it can overwhelm the log.

This is the case in PowerShell DSC. You receive a cryptic error similar to The WS-Management service cannot process the request. The computed response packet size (3388031) exceeds the maximum envelope size that is allowed (512000).

When 0.10.4+ is detected, add --no-progress (https://github.com/chocolatey/choco/issues/917) automatically when managing packages with cChoco.

jrdnr commented 6 years ago

@ferventcoder Should this be Up for grabs or are you working on it? I have working code to patch this that I'm using internally, but its a little fragile because I just pasted the same check into the install/uninstall/ and update functions. The right way to fix this might be to refactor the install/uninstall/update functions to have more consistent code and naming across them, and actually break running choco $params into its own function so that anytime there is a a change to the core choco application output it only has to be updated one place.

ferventcoder commented 6 years ago

@jrdnr up for grabs.

ferventcoder commented 6 years ago

Go for the simple changes first and we can look for the more invasive changes later.

jessejacob commented 6 years ago

Just wanted to say thanks y'all for implementing this. HUGE reduction in Packer log sizes where I was using lots of choco operations!

jrdnr commented 6 years ago

@ferventcoder Any chance of getting this PR merged?

TTimo commented 5 years ago

I would like to see this addressed as well.

jrdnr commented 5 years ago

@TTimo The PR to fix was submitted almost a year ago, but I haven't been able to get a response from the maintainers.

FinickyCode commented 5 years ago

A possible workaround while waiting for this PR to be merged is to disable progress globally.

choco feature disable -n=showDownloadProgress
pauby commented 5 years ago

The PR is working it's way through. We have been very slow to pick this up. Changes have been requested and would hope to get this ready for the next release.