This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author gregory.p.smith
Recipients gregory.p.smith
Date 2022-03-22.20:08:54
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1647979734.3.0.881589682781.issue47090@roundup.psfhosted.org>
In-reply-to
Content
Bringing this up on discord, others point out that the windows build requires zlib for convenience when we transitioned from having a vendored copy in our repo and that smaller "embedded" use cases may not like this if they don't already need the dep. But there have been no complaints about it on the Windows side.

`binascii.crc32` was one thing that motivated me to look into "just require zlib" to get out of carrying our own suboptimal crc32 code.  along those lines, we should recommend people choose https://github.com/zlib-ng/zlib-ng rather than zlib.net for better performance.

looking over my PR, it can make for some awkward code with zlib right next to others that we treat as optionals.  good bad or indifferent?  i'm leaning towards indifferent and still enjoying fewer lines of code.
History
Date User Action Args
2022-03-22 20:08:54gregory.p.smithsetrecipients: + gregory.p.smith
2022-03-22 20:08:54gregory.p.smithsetmessageid: <1647979734.3.0.881589682781.issue47090@roundup.psfhosted.org>
2022-03-22 20:08:54gregory.p.smithlinkissue47090 messages
2022-03-22 20:08:54gregory.p.smithcreate