Bug 16656 – move embedded zlib to a separate library

Status
ASSIGNED
Severity
enhancement
Priority
P4
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2016-11-02T12:51:46Z
Last change time
2024-12-01T16:28:10Z
Assigned to
No Owner
Creator
Martin Nowak
Moved to GitHub: phobos#10206 →

Comments

Comment #0 by code — 2016-11-02T12:51:46Z
We'd like to get rid of the embedded zlib in phobos. https://github.com/dlang/phobos/tree/fd518eb310a9494cccf28c54892542b052c49669/etc/c/zlib As a shared zlib is available on literally any non-Windows OS, we should prefer using that over statically linking. For Windows we can integrate a DLL and import library with out binary releases, similarly to how we handle libcurl. That means someone documents and builds zlib [¹], we upload the results to downloads.dlang.org/other, and integrate it with the build script [²][³]. [¹]: https://wiki.dlang.org/Curl_on_Windows [²]: https://github.com/dlang/installer/blob/8b3c7efe1e049f73ec5fc526a849904bf80b434d/create_dmd_release/build_all.d#L451 [³]: https://github.com/dlang/installer/blob/8b3c7efe1e049f73ec5fc526a849904bf80b434d/create_dmd_release/build_all.d#L232
Comment #1 by greeenify — 2016-12-27T10:56:51Z
Why don't we remove zlib entirely from Phobos and move it to the DUB universe? Isn't this something that we are better off putting into the userland? (same for the sqlite bindings etc.) If there are blocking bugs/issues with DUB, we should rather try to fix those. FYI is already e.g. archive: https://code.dlang.org/packages/archive
Comment #2 by robert.schadek — 2024-12-01T16:28:10Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10206 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB