Skip to content

Link zlib-ng native if available#574

Merged
nmoinvaz merged 4 commits into
zlib-ng:devfrom
Diatrus:dev
Mar 5, 2022
Merged

Link zlib-ng native if available#574
nmoinvaz merged 4 commits into
zlib-ng:devfrom
Diatrus:dev

Conversation

@Diatrus

@Diatrus Diatrus commented May 17, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@nmoinvaz

nmoinvaz commented Mar 5, 2022

Copy link
Copy Markdown
Member

I think one problem is that we don't know if system zlib-ng was built with ZLIB_COMPAT or not.

@nmoinvaz

nmoinvaz commented Mar 5, 2022

Copy link
Copy Markdown
Member

Ah, actually if it has -ng suffix it was built with ZLIB_COMPAT=OFF

@nmoinvaz nmoinvaz merged commit 2d10201 into zlib-ng:dev Mar 5, 2022
nmoinvaz pushed a commit that referenced this pull request Mar 5, 2022
@Diatrus Diatrus deleted the dev branch March 5, 2022 21:38
@jhudsonWA

Copy link
Copy Markdown
Contributor

Is my understanding correct that if we don't build with ZLIB_COMPAT we will have to include zlib-ng now? (if I don't I get a error that zlib-ng.h is missing)

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.

3 participants