-
-
Notifications
You must be signed in to change notification settings - Fork 272
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
7-Zip.Compression #722
base: master
Are you sure you want to change the base?
7-Zip.Compression #722
Conversation
Build 3.0.1.1438 (with this change) Build 3.0.1.1430 (recent build without this change) So the change increased the size |
I saw this, too! But I needed a break! ;) |
Short explanation to the size (later more to it): Seems the compression method PPMd saves a lot of space with small files (library.zip), but waste a lot by big archives (portable.zip)... I need to do some new tests/comparison and play with the parameters of 7-Zip... ;) |
While it's possible to do a bit better, I expect there's little margin to improve the portable archive because most of its contents (e.g., executables, images, and the library.zip archive) are already compressed. |
Yes, but if I look at your past work, it looks like you are a perfectionist... I be one, too! And sometimes I like a bit modding and tweaking... ;) ...so why not try to find out, whats really the perfect parameters for this archives? ;) |
Btw.: I tested some of my different library.zip test files with the portable version... And yes, it's true, the portable version doesn't work with a e.g. with Deflate64 compressed archive! What I have now done (and it's easy to see in the PR, because I needed to change just some lines for it) is the following: And btw.: I guess I found the reason why the EXE headers of the NSIS installers don't get compressed, ATM. The path to the packer seems to be incomplete! But this must/should be changed in the NSI file and the PRs of this file should get merged serial and I have already 3 PRs open for it, so I decided, to don't write one for it, yet. |
I'm closing this because the current PR makes the files bigger. If you want to try again later, that's fine |
Andrew, I wasn't just finished, yet... :-/ |
Reopened |
No description provided.