$output -match 'fail' regex too generic about foil

[dbaileyut]

Thanks!

Yeah, I gave a thumbs up to PowerShell/Crescendo#63 before I submitted this issue because I figured that was the case :)

[dbaileyut]

Huh, I'm not able to reproduce either the Office or Sysinternals issue now. I my best guess is I might have been using the built-in PackageMangement resource instead of PSDscResources/PackageManagement 1.4.7

The warning I was getting came from
Set-EnvironmentVariable.ps1

Called by (semi stack trace here):
Install-ChocolateyPath.ps1
Sysinternals/chocolateyInstall.ps1

I think the updated regex will be much more reliable either way.

Thanks again! And, kudos on a great name for the module.

[ethanbergstrom]

Hey @dbaileyut

Always happy to see a fellow DSC+ChocolateyGet user!

I think this goes back to PowerShell/Crescendo#63, where Cresendo doesn't have great any error handling, so I used something (very) simple to fill the gap.

I'll try to do some research as to what actual 'failure' output from Chocolatey looks like and write some better regex for it.

In the meantime, if it's an absolute blocker and you have an immediate need, I might suggest downgrading ChocolateyGet to v2.1.1, which doesn't use Foil and still uses exit code logic for determining failures, but should be functionally equivalent and still fixes an urgent issue (in a different way) introduced by a Chocolatey.org repository change described here: jianyunt/ChocolateyGet#24 (comment)

[ethanbergstrom]

Would you be able to provide a command example for sysinternals that yielded the "fail" output? I actually happen to use sysinternals with a custom install directory argument in my tests, and am not seeing the same behavior.

I did find a very helpful example of "uniquely error-like" text in chocolatey-community/chocolatey-packages#1608 reported to the Chocolatey core package maintainers repo, so I can use googlechrome v87.0.4280.141 to test for what valid error text looks like.

[ethanbergstrom]

Here's the new regex, I think it should work pretty well:

Foil/src/Foil.ps1

Line 109 in 577c8be

$failures = ($output -match 'Chocolatey installed \d+\/\d+ packages. \d+ packages failed\.')

Version 0.0.4 is out on PowerShell Gallery, and I think that should fix your issue. Can you try upgrading Foil to see if that fixes your issue? I wasn't able to reproduce with sysinternals, and Office 365 takes so long to install on my slower internet, I got impatient and gave up :)

[dbaileyut]

Awesome! I'll try it out in just a bit and report back on the sysinternals warning too.