-
Notifications
You must be signed in to change notification settings - Fork 432
[WIP] libarchive-based dearchiving
#4576
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
base: master
Are you sure you want to change the base?
Conversation
|
Dare I ask what this is for? |
|
It supports more formats than SharpCompress. |
|
I assume the particular format you're interested in is zstd, and you want to simplify our zstd use. Including making it so that zstd compressed files in our archives won't need the .zst extension anymore and can be edited via text editor more easily. |
|
I don't even know what you're suggesting. BizHawk/src/BizHawk.Client.Common/SharpCompressDearchivalMethod.cs Lines 70 to 80 in db070b9
|
Your goal is to "be like everybody else", then? Or is there any use case/feature that this enables, or issue that it fixes? It doesn't look like you're actually using this new library anyway. I have no idea where or how you might intend to use it. |
|
I would imagine opening games inside more types of archives would be the intended use-case. That is a useful feature. |
|
Currently the dearchival method is set globally on program startup, so it would be a 1-line change to use only libarchive: BizHawk/src/BizHawk.Client.EmuHawk/Program.cs Line 181 in db070b9
|
All of those are already supported by SharpCompress. At most they are just not supported within BizHawk's usage of SharpCompress. |
tests fail for ¯\_(ツ)_/¯ reasons