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

Upgrade 7Zip to 16.04 #1184

Closed
trajano opened this issue Mar 4, 2017 · 7 comments
Closed

Upgrade 7Zip to 16.04 #1184

trajano opened this issue Mar 4, 2017 · 7 comments

Comments

@trajano
Copy link

trajano commented Mar 4, 2017

What You Are Seeing?

Error extracting files from https://mega.nz/MEGAcmdSetup.exe for the uninst.exe and `megacmdclient.exe files.

What is Expected?

I should be able to use Install-ChocolateyZipPackage with no issues.

How Did You Get This To Happen? (Steps to Reproduce)

$packageArgs = @{
    packageName   = $env:chocolateyPackageName
    unzipLocation = $env:chocolateyPackageFolder
    url = 'https://mega.nz/MEGAcmdSetup.exe'
    checksum      = '350320ca4a9d2e4ee8be4e6d9f262be91d70e03344baaa1cf5d71dde662a12453262604ee44a0d5e9d29c347f9752424e68c1c4419b2969f9a7294ffea02e5c0'
    checksumType  = 'sha512'
}
Install-ChocolateyZipPackage @packageArgs

Output Log

ERROR: Data Error : MEGAclient.exe
ERROR: Data Error : uninst.exe

To fix I think you just need to update the 7Zip to 16.04
https://github.com/chocolatey/choco/tree/master/src/chocolatey.resources/tools

For now my workaround is to install 7zip.portable first then use it to extract as per https://github.com/trajano/choco-packages/blob/master/megacmd/tools/chocolateyinstall.ps1

@ferventcoder
Copy link
Member

@trajano you making a portable and an installer version of this? I ran Package Builder on this and it generated it as an NSIS

@ferventcoder
Copy link
Member

image

@ferventcoder
Copy link
Member

And by Package Builder, I literally mean choco new --url https://mega.nz/MEGAcmdSetup.exe --build-package and within seconds I had a package.

image

@ferventcoder
Copy link
Member

Ah uninstaller is not completely silent. Fun...

@ferventcoder
Copy link
Member

Verified failure. Verified fix with latest 7z. This will be released in 0.10.4. Thanks for logging the issue!

ferventcoder added a commit that referenced this issue Mar 18, 2017
Upgrade binaries to latest version of 7zip
ferventcoder added a commit that referenced this issue Mar 18, 2017
* stable:
  (GH-1184) Upgrade to 7zip 16.04
  (maint) remove unused 7za
  (maint) update why 32-bit 7z is required
  (GH-676) Fix: Unzip specific folder feature is broken
  (maint) Additional version environment variables
@trajano
Copy link
Author

trajano commented Mar 18, 2017

What's the ETA for 0.10.4? I'll make the change to the package later.

@ferventcoder
Copy link
Member

@trajano to save on retyping, take a look at #1023 (comment)
HTH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants