Skip to content

Conversation

@respondersGY
Copy link

The package is used but not defined:

import zstandard as zstd
.

[project.optional-dependencies]
full = [
"pycryptodome",
"zstandard",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we could immediately switch to backport.zstd as noted here: fox-it/dissect.target#1380

And change the code to use that?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a quick go at this myself and ended up with #69 (needed to make a new PR to push the test data), so I'll close this one!

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