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

(GH-501) Fix Propagation of VerbosePreference to Reduce Verbose Logging #512

Merged
merged 1 commit into from
Aug 27, 2022

Conversation

MalwareMechanic
Copy link
Contributor

Description Of Changes

Updated how VerbosePreference was being used to propagate the verbose setting for new choco.exe processes and during Boxstarter environment variable restoration.

Motivation and Context

The VerbosePreference parameter was not being set correctly for new choco.exe processes as well as when exporting Boxstarter environment variables resulting in verbose output during subprocess Chocolatey package installations.

Testing

Tested on Windows 11 (21H2 22000.593). Added a new test to check the -Verbose parameter is not being set incorrectly. Did not add a test to check for correct environment variable restoration since the test Export-BoxstarterVars in Chocolatey.tests.ps1 failed complete -- likely a scheduled task issue.

Change Types Made

  • Bug fix (non-breaking change)
  • Feature / Enhancement (non-breaking change)
  • Breaking change (fix or feature that could cause existing functionality to change)
  • PowerShell code changes.

Related Issue

Fixes #501

Change Checklist

  • Requires a change to the documentation
  • Documentation has been updated
  • Tests to cover my changes, have been added
  • All new and existing tests passed.
  • PowerShell v2 compatibility checked.

The VerbosePreference parameter was not being set
correctly for new choco.exe processes as well as
when exporting environment variables.
@mwallner
Copy link
Member

thanks for getting this fixed @MalwareMechanic !

CC @gep13 - I think we're having some issues with gitversion in the appveyor build - 3.1.0-fix-verbose-output-20220718 is too long as package version and hence breaks the build -> any suggestions on how to (permanently) get this fixed?

@Baune8D
Copy link

Baune8D commented Aug 23, 2022

I see this fixes my issue 🙂 Any possibility for getting this merged? This will make it possible for me to finally test out v3

@mwallner mwallner merged commit f3634ce into chocolatey:develop Aug 27, 2022
@MalwareMechanic
Copy link
Contributor Author

@mwallner Any thoughts on when a hotfix release can be made so this fix helps reduce the verbose output for everyone using the V3?

@Baune8D
Copy link

Baune8D commented Oct 25, 2022

@mwallner Any thoughts on when a hotfix release can be made so this fix helps reduce the verbose output for everyone using the V3?

+1 for this, im also holding out on trying v3 until this fix gets released👍

@mwallner
Copy link
Member

heya,
not going to make any promises, but I can tell you that I'm working on two (other) different defect-type-issues that will make it into a bugfix-type version somewhen soonish. in that upcoming release, this will also be included.
hth, sorry to not being able to give you a more detailed answer

gep13 added a commit that referenced this pull request Jun 6, 2023
* hotfix/3.0.1:
  GH-518: fix Call-Chocolatey usage
  (GH-532) Use 'choco install' instead of cinst (#533)
  (#524) Fix Build Handling of Long Prerelease
  (maint) eol consistency
  (maint) .gitattributes new line
  (GH-501) Fix Propagation of VerbosePreference (#512)
  (#508) Fix package iconUrl
  Fixed typo in README.md
  Update README.md
  (maint) Re-instate the Boxstarter logo
  (maint) link boxstarter.org repository
  (maint) crlf in Markdown files!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lots of verbose output noise when installing Chocolatey packages using Boxstarter 3.0.0
3 participants