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.

Sign up for GitHub

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

Jump to bottom

Turn on zlib for larger builds #1789

Closed
rdagger opened this issue Apr 13, 2019 · 3 comments
Closed

Turn on zlib for larger builds #1789

rdagger opened this issue Apr 13, 2019 · 3 comments
Labels
cpython api modules from cpython
Milestone
Long term

Comments

@rdagger
Copy link

rdagger commented Apr 13, 2019

The docs indicate zlib support: https://circuitpython.readthedocs.io/en/latest/docs/library/uzlib.html

However, it doesn't look like it's implemented for the nRF52840. I tried the Feather nRF52840 Express and the PCA10059.

I'm trying to write a CircuitPython Png decoder and this would be very helpful.
@dhalbert dhalbert added this to the Long term milestone Apr 15, 2019
@dhalbert dhalbert added the cpython api modules from cpython label Oct 31, 2019
@dhalbert
Copy link
Collaborator

uzlib is not turned on for any CircuitPython ports. It might be nice to have if it's not too big. It should be moved to shared-bindings and shared-module.

@dhalbert dhalbert changed the title nRF52840 zlib support Turn on zlib for larger builds Oct 31, 2019
@dhalbert
Copy link
Collaborator

See PR #1274

@dhalbert
Copy link
Collaborator

Partly addressed by #6069. See #6284 for further followup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cpython api modules from cpython
Projects
None yet
Milestone
Long term
Development

No branches or pull requests
2 participants
@rdagger @dhalbert