Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(decompress): Handle split RAR archives #4994

Merged
merged 9 commits into from
Jun 24, 2022
Merged

fix(decompress): Handle split RAR archives #4994

merged 9 commits into from
Jun 24, 2022

Conversation

issaclin32
Copy link
Member

@issaclin32 issaclin32 commented Jun 18, 2022

  • This is a follow-up of fix(depends): Avoid digits in archive file extension #4915.

  • Splitted RAR archives is rare for normal apps, but it is still possible for games or something with a large file size.

  • File name for splitted RAR archives can be: something.part1.rar, something.part01.rar, something.part001.rar, ..., depending on total number of splits.

  • For TestCases.zip: I tried to compress empty into 3 RAR archives (30 Bytes each), but it didn't work. WinRAR rather creates a single file. The minimum size for RAR split archive is 925 Bytes according to my tests. (not sure why though)

lib/decompress.ps1 Outdated Show resolved Hide resolved
Co-authored-by: Rashil Gandhi <46838874+rashil2000@users.noreply.github.com>
rashil2000
rashil2000 previously approved these changes Jun 23, 2022
niheaven
niheaven previously approved these changes Jun 24, 2022
@niheaven niheaven changed the title fix(decompress): Handle splitted RAR archives fix(decompress): Handle split RAR archives Jun 24, 2022
@niheaven niheaven merged commit 1b5ee6f into develop Jun 24, 2022
@niheaven niheaven deleted the issaclin32-rar branch June 24, 2022 14:15
slaughtering pushed a commit to slaughtering/scoop that referenced this pull request Jul 7, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants