cmake: (re-)add zlib to Libs.private in libssh2.pc
#1131
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We mistakently added transitive zlib to `Requires.private` before, then removed it. This patch re-adds zlib, but this time to `Libs.private`, which is listing raw libs and should include transitive libs as well. Also add zlib when used as a direct dependency when zlib compression support is enabled. Follow-up to ef53806 Closes libssh2#1131
d181f8f to
9d0da8a
Compare
lampmanyao
pushed a commit
to lampmanyao/libssh2
that referenced
this pull request
Sep 3, 2023
We mistakently added transitive zlib to `Requires.private` before, then removed it. This patch re-adds zlib, but this time to `Libs.private`, which is listing raw libs and should include transitive libs as well. Also add zlib when used as a direct dependency when zlib compression support is enabled. Follow-up to ef53806 Closes libssh2#1131
agreppin
pushed a commit
to agreppin/libssh2
that referenced
this pull request
Jul 14, 2024
We mistakently added transitive zlib to `Requires.private` before, then removed it. This patch re-adds zlib, but this time to `Libs.private`, which is listing raw libs and should include transitive libs as well. Also add zlib when used as a direct dependency when zlib compression support is enabled. Follow-up to ef53806 Closes libssh2#1131
vszakats
added a commit
to vszakats/libssh2
that referenced
this pull request
Jul 13, 2025
And the libs listed in `Libs.private` in `libssh2.pc` with it. The order and repetition might be significant for linkers that rely on strict lib order and unable to resolve symbols without it. Such linker is binutils `ld`. This also syncs this snippet with curl. De-duplication was originally introduced to avoid a repeat `-lz`. It means after this patch a repeat `-lz` is back again. (with a zlib-enabled OpenSSL or wolfSSL build.) One safe way to de-duplicate would be to remove dupes when they appear right next to each other. This is an excercise for another PR, because cmake doesn't have such built-in function. Meanwhile, such duplicates don't cause harm, and only make the output less clean-looking and introduce potential differences between `libssh2.pc` generated by cmake and autotools. (those aren't necessary identical anyway, yet.) Follow-up to 6464301 libssh2#1131
vszakats
added a commit
that referenced
this pull request
Jul 13, 2025
`LIBSSH2_PC_LIBS_PRIVATE` ends up in `Libs.private` in `libssh2.pc`. The order and duplication may be significant for linkers that rely on strict lib order and unable to resolve symbols without it. Such linker is binutils `ld`. De-duplication can break it. As of now there is no purposeful duplication in libs in libssh2, thus the de-duplication most likely did not affect actual builds. It was originally introduced to avoid a repeat `-lz` (with a zlib-enabled OpenSSL or wolfSSL build.) To keep this feature, this patch makes sure to only delete duplicates that are next to each other. Follow-up to 6464301 #1131 Closes #1621
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We mistakently added transitive zlib to
Requires.privatebefore, then removed it. This patch re-adds zlib, but this time toLibs.private, which is listing raw libs and should include transitive libs as well.Also add zlib when used as a direct dependency when zlib compression support is enabled.
Follow-up to ef53806
Closes #1131