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

Choco0.10.3 #219

Closed
wants to merge 2 commits into from
Closed

Choco0.10.3 #219

wants to merge 2 commits into from

Conversation

riezebosch
Copy link
Contributor

Updated the chocolatey lib to 0.10.3.

Things I had to fix:

  • Initialisation of the ChocolateyWrapper
  • Output of the PsLogger
  • Redirect of Verbose messages that (accidentally) were invoked on the Info method

Currently there are still some things up for improvement:

But at least this update fixes a handful of open compatibility issues with newer versions of chocolatey:

using static choco to register psservice exactly once
using Console to write to output from PsLogger
@ferventcoder
Copy link
Member

chocolatey/choco#1121

@@ -2,7 +2,8 @@ function Invoke-Chocolatey($chocoArgs) {
Write-BoxstarterMessage "Current runtime is $($PSVersionTable.CLRVersion)" -Verbose
$refs = @(
"$($Boxstarter.BaseDir)/boxstarter.chocolatey/chocolatey/log4net.dll",
"$($Boxstarter.BaseDir)/boxstarter.chocolatey/chocolatey/chocolatey.dll"
"$($Boxstarter.BaseDir)/boxstarter.chocolatey/chocolatey/chocolatey.dll",
"$($Boxstarter.BaseDir)/boxstarter.chocolatey/chocolatey/AlphaFS.dll"
Copy link
Member

Choose a reason for hiding this comment

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

Hopefully this won't be needed for long - chocolatey/choco#1102

@mwrock
Copy link
Member

mwrock commented Jan 8, 2017

The unfortunate consequence of using Console is that remote runs of boxstarter lose all output. However, I can see why you went down that route. I've done a fair bit of tinkering myself and something is causing powershell to crash with a stackoverflowexception if you use the powershell host.

@riezebosch
Copy link
Contributor Author

That's the reason I went for the console. It looks like the writes on PSHostUserInterface are intercepted (by chocolatey I presume) and fed back to the custom logger causing an infinite recursive loop resulting in a stack overflow. But I couldn't figure out where that interception was done and how I could work around it.

@mwrock
Copy link
Member

mwrock commented Jan 8, 2017

right. the exception occurs on the very first call to write-* by the caller of the choco library after initialization but before calling run on the choco instance. I'm torn between hunting this down in choco and just rewriting boxstarter as its design gets more grotesque every time I look at it. If I could only stop time.

@ferventcoder
Copy link
Member

Try with the fixes I implemented and see if it is still an issue. There should be an available build here - https://ci.appveyor.com/project/chocolatey/choco/build/1.0.0.1121/artifacts

@mwrock
Copy link
Member

mwrock commented Jan 9, 2017

Oh that does indeed address the stackoverflow! Nice @ferventcoder.

@mwrock
Copy link
Member

mwrock commented Jan 13, 2017

@ferventcoder any eta on when that fix will land in the nuget feed?

@ferventcoder
Copy link
Member

Jan/ early Feb is the best estimate.

@mwrock
Copy link
Member

mwrock commented Jan 13, 2017

cool. I'll keep an eye out.Closing this. Master should be good to go once that releases.

@ferventcoder
Copy link
Member

So February is what we are working with now. That date is unlikely to shift.

@wspait
Copy link
Contributor

wspait commented Feb 15, 2017

Hi Folks, is this waiting to be approved by @mwrock to be merged back into Master?

@mwrock
Copy link
Member

mwrock commented Feb 17, 2017

No master has most of what is necessary. I'm just waiting on the above referenced choco fix to land in the nuget feed and then I can prep a release.

@wspait
Copy link
Contributor

wspait commented Feb 17, 2017

@mwrock Thanks for the response! Once that choco version gets into NuGet, a new Boxstarter version is planned for release sometime thereafter?

@mwrock
Copy link
Member

mwrock commented Feb 17, 2017

correct!

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.

4 participants