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

choco install fail, checksum missing on dependencywalker and regshot #16

Open
adricnet opened this issue Dec 7, 2016 · 8 comments
Open
Labels

Comments

@adricnet
Copy link

adricnet commented Dec 7, 2016

Hello malboxes team,

Thanks for releasing this as it looks fantastic. Unfortunately, a smoke test of the default configuration fails due to a missing checksum. This has been flagged upstream, but not acknowledged that I can see in the Choco package index page for DepWalker: http://disq.us/p/1drvmde.

Please consider either kludging this security policy in Choco or taking the very useful tool DepWalker out of the sample configs you ship so as to help new users have a good first experience as they follow your documentation and try out the tool.

Thanks,
adric

    virtualbox-iso: Failures
    virtualbox-iso: - dependencywalker (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\dependencywalker\tools\chocolateyInstall.ps1'.
    virtualbox-iso: See log for details.
==> virtualbox-iso: Unregistering and deleting virtual machine...
==> virtualbox-iso: Deleting output directory...
Build 'virtualbox-iso' errored: Script exited with non-zero exit status: 4294967295

==> Some builds didn't complete successfully and had errors:
--> virtualbox-iso: Script exited with non-zero exit status: 4294967295

==> Builds finished but no artifacts were created.
----------------------------------
packer completed with return code: 1
Packer failed. Build failed. Exiting...

Log snippet Gist here

@obilodeau
Copy link
Member

We will temporarily remove DepWalker from our tools list. Thanks for reporting!

@adricnet
Copy link
Author

adricnet commented Dec 7, 2016

Thank you for such a fast response and being so helpful!

Unfortunately my followup build (edited out depwalker in config.js) hit the same problem with regshot. I'm going to try and dig into choco docs and see what the least bad way around this is generally (as well as trying a build wo regshot or depwalker for good measure).

Cheers,
adric

@obilodeau
Copy link
Member

Ok, new idea: disable checksum checking until all outstanding upstream packages are fixed. This can be achieved via a choco.exe flag.

@adricnet
Copy link
Author

adricnet commented Dec 7, 2016

Yes, that's the general solution, though it is not without risk as I understand it. OTOH, I have a box built and running nicely without those two tools, and can go on about my testing quite pleasantly.

Cheers, thanks,
adric

@obilodeau
Copy link
Member

Ok, if regshot and DepWalker are the only two, I'll avoid installing them for now as a workaround and I'll try to find a way to contribute the checksums upstream. Thanks for your testing!

@obilodeau obilodeau changed the title Sample win10 64 box build fails on depwalker choco install fail, checksum missing choco install fail, checksum missing on dependencywalker and regshot Dec 8, 2016
obilodeau added a commit to GoSecure/chocolateypackages.AnthonyMastrean that referenced this issue Dec 8, 2016
obilodeau added a commit to GoSecure/ChocolateyPackages.MarkRobertJohnson that referenced this issue Dec 8, 2016
@obilodeau
Copy link
Member

Found source of packages and submitted two pull requests to fix the issues. Here are the PR to track:

@adricnet
Copy link
Author

adricnet commented Dec 8, 2016

Wow, thank you for finding, submitting, and linking those! Leave this one open to track against upstream?

@obilodeau
Copy link
Member

Leave this one open to track against upstream?

Yes let's do that.

dgalbraith pushed a commit to dgalbraith/chocolatey-packages that referenced this issue Sep 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants