Skip to content

Conversation

@YoshiRulz
Copy link
Member

tests fail for ¯\_(ツ)_/¯ reasons

@YoshiRulz YoshiRulz added Request: Feature/Enhancement For feature requests or possible improvements Meta Relating to code organisation or to things that aren't code labels Dec 6, 2025
@Morilli
Copy link
Collaborator

Morilli commented Dec 6, 2025

Dare I ask what this is for?

@YoshiRulz
Copy link
Member Author

It supports more formats than SharpCompress.

@SuuperW
Copy link
Contributor

SuuperW commented Dec 6, 2025

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.

@YoshiRulz
Copy link
Member Author

I don't even know what you're suggesting.
My goal here is to use the de facto standard library for working with archives, and in doing so add support for .tar.gz, .tar.bz2, .tar.lz, and sure why not .tar.zst.

public IReadOnlyCollection<string> AllowedArchiveExtensions { get; } = new[]
{
".7z",
".gz",
".rar",
".tar",
// /*.tar*/".bz2", ".tb2", ".tbz", ".tbz2", ".tz2",
// /*.tar.gz,*/ ".taz", ".tgz",
// /*.tar*/".lz",
".zip",
};

@SuuperW
Copy link
Contributor

SuuperW commented Dec 6, 2025

My goal here is to use the de facto standard

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.

@RetroEdit
Copy link
Contributor

I would imagine opening games inside more types of archives would be the intended use-case. That is a useful feature.

@YoshiRulz
Copy link
Member Author

Currently the dearchival method is set globally on program startup, so it would be a 1-line change to use only libarchive:

HawkFile.DearchivalMethod = SharpCompressDearchivalMethod.Instance;

@CasualPokePlayer
Copy link
Member

and in doing so add support for .tar.gz, .tar.bz2, .tar.lz, and sure why not .tar.zst

All of those are already supported by SharpCompress. At most they are just not supported within BizHawk's usage of SharpCompress.

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

Labels

Meta Relating to code organisation or to things that aren't code Request: Feature/Enhancement For feature requests or possible improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants