cmake: de-duplicate LIBSSH2_PC_LIBS_PRIVATE more carefully
#1621
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
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
LIBSSH2_PC_LIBS_PRIVATELIBSSH2_PC_LIBS_PRIVATE more carefully
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.
LIBSSH2_PC_LIBS_PRIVATEends up inLibs.privateinlibssh2.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(witha 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