Skip to content
This repository has been archived by the owner on Sep 20, 2018. It is now read-only.

Support for RAR5 #14

Closed
wants to merge 3 commits into from
Closed

Support for RAR5 #14

wants to merge 3 commits into from

Conversation

squid-box
Copy link

@squid-box squid-box commented May 19, 2018

Impatiuent as I am, this is a PR for #13. I'm really unsure if this is the way to approach this, but here's a (briefly tested) suggestion:

  • I added the signature and GUID for the new RAR5 format
  • The .rar extension now points to RAR5 by default, testing shows this is no problem, but I'm not very knowledgeable about this at all.
  • I kept the old signature and GUID data, but they're now under InArchiveFormat.Rar4.

I also had to update the included 7z .dll's, as the ones in the repo could not identify the RAR5 archive. The ones included in this PR are from 18.05 (currently latest released version).

I could not get the test applications in SevenZip.sln to work (before or after my changes), so there's a good chance I've missed something...

@squid-box
Copy link
Author

If you want to test it, here's two tiny RAR files, one compressed with RAR4 and one with RAR5: FakeMods.zip

squid-box added a commit to squid-box/SevenZipSharp that referenced this pull request Jul 20, 2018
@squid-box squid-box closed this Jul 20, 2018
@squid-box squid-box deleted the rar5 branch July 22, 2018 21:46
squid-box added a commit to arttarasov/SevenZipSharp that referenced this pull request Aug 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant