Skip to content

Experiment: Try to make crunch64 not mandatory - #552

Draft
AngheloAlf wants to merge 2 commits into
ethteck:mainfrom
AngheloAlf:optional_dependencies
Draft

Experiment: Try to make crunch64 not mandatory#552
AngheloAlf wants to merge 2 commits into
ethteck:mainfrom
AngheloAlf:optional_dependencies

Conversation

@AngheloAlf

@AngheloAlf AngheloAlf commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

WIP for issue #331

The current state is an experiment. Looking for testing and feedback

With this branch you could avoid installing crunch64 and use everything else without much issue.
For this you would need to install [mips-min] instead of [mips] dependency group.

To test this branch, either:

  • Checkout with git and do pip install .[mips-min] on a fresh venv.
  • Directly with pip pip uninstall splat64 -y && pip install git+https://github.com/AngheloAlf/splat.git@optional-dependencies[mips-min] on a fresh venv.

A fresh venv is needed to ensure you don't have crunch64 from a previous installation.

If splat detects you want to use either mio0 or yay0 then it suggest you to install splat64[compression] to get them, since they aren't available here, I think. I haven't tried this tho, so would be nice if somebody tries it out first. splat64[compression] implies both splat64[compression,mips-min]

The usual splat64[mips] group still installs crunch64 to avoid breaking current setups.

@AngheloAlf
AngheloAlf marked this pull request as draft August 6, 2026 12:58
@ethteck

ethteck commented Aug 6, 2026

Copy link
Copy Markdown
Owner

I like this approach a lot. I think the only thing I'm not in love with is specifically the name mips-compression, because I believe crunch64's compression formats go beyond mips-based platforms. Or at the very least, they aren't related to a specific ISA, even if they are only used in mips-based games. I'd potentially just call the 'feature' compression? but as I say that, I'm not sure. anyway there's my thought process. see ya

@AngheloAlf

Copy link
Copy Markdown
Collaborator Author

Renamed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants