Skip to content
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

deps: replace python-lzo by lzallright. #76

Merged
merged 1 commit into from
May 12, 2023
Merged

deps: replace python-lzo by lzallright. #76

merged 1 commit into from
May 12, 2023

Conversation

qkaiser
Copy link
Contributor

@qkaiser qkaiser commented May 12, 2023

We proposed to move away from python-lzo because:

  • it doesn't have a wheel, so you still need a C compiler and lzo sources
  • licensing issues

This commit replace python-lzo with https://github.com/vlaci/lzallright, a Python binding for https://crates.io/crates/lzokay

@qkaiser qkaiser self-assigned this May 12, 2023
@qkaiser
Copy link
Contributor Author

qkaiser commented May 12, 2023

Once we get this in, we can tag it with a new version and perform the release locally so we're clear on unblob side to fully get rid of python-lzo.

Subsequent work on improving ubireader can happen afterwards (see #75).

@vlaci
Copy link
Contributor

vlaci commented May 12, 2023

This also means dropping Python 2 support.

We proposed to move away from python-lzo because:

- it doesn't have a wheel, so you still need a C compiler and lzo sources
- licensing issues

This commit replace python-lzo with https://github.com/vlaci/lzallright, a Python binding for https://crates.io/crates/lzokay
@vlaci vlaci merged commit 1cc34b2 into main May 12, 2023
@vlaci vlaci deleted the replace-python-lzo branch May 12, 2023 07:46
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