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

Consider replacing zlib dependency with header-only equivalent #100

Open
mr-smidge opened this issue Jul 6, 2023 · 1 comment
Open

Consider replacing zlib dependency with header-only equivalent #100

mr-smidge opened this issue Jul 6, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@mr-smidge
Copy link
Contributor

mr-smidge commented Jul 6, 2023 •
edited
Loading

Overview

The library currently depends on zlib. This is a commonplace dependency that is widely available, but C++ friendly header-only alternatives exist, such as zstr and bxzstr.

By using one of these libraries, it may be possible to lower the dependency footprint of libdjinterop. In fact, as Sqlite can also be linked in via the amalgamation, this change could make libdjinterop a zero-dependency library.

Acceptance Criteria

  • The library must not have a dependency on zlib.
  • Additional dependencies must not be added to libdjinterop as a result of this change.
@mr-smidge mr-smidge added the enhancement New feature or request label Jul 6, 2023
@JoergAtGithub
Copy link

Also a much faster fork of zlib exist: https://github.com/zlib-ng/zlib-ng

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mr-smidge @JoergAtGithub