-
Notifications
You must be signed in to change notification settings - Fork 137
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
Comments
We will temporarily remove DepWalker from our tools list. Thanks for reporting! |
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, |
Ok, new idea: disable checksum checking until all outstanding upstream packages are fixed. This can be achieved via a choco.exe flag. |
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, |
Ok, if |
Found source of packages and submitted two pull requests to fix the issues. Here are the PR to track: |
Wow, thank you for finding, submitting, and linking those! Leave this one open to track against upstream? |
Yes let's do that. |
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
Log snippet Gist here
The text was updated successfully, but these errors were encountered: