Skip to content

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Jul 14, 2023

(libssh2.pc not tested as a project dependency.)

autotools was using `libwolfssl`. CMake left it empty. The `.pc`
file provided by wolfSSL is called `wolfssl.pc`.

This patch sets `Requires.private: wolfssl` with both build tools.

Not tested.

Closes libssh2#1123
@vszakats vszakats changed the title build: fix Requires.private for wolfSSL in libssh2.pc build: add/fix Requires.private for wolfSSL/libgcrypt in libssh2.pc Jul 14, 2023
Also delete just added lib dedupe, it's now redundant.
Ref: 2fc3679
@vszakats vszakats changed the title build: add/fix Requires.private for wolfSSL/libgcrypt in libssh2.pc build: add/fix Requires.private packages in libssh2.pc Jul 14, 2023
@vszakats vszakats closed this in ef53806 Jul 14, 2023
@vszakats vszakats deleted the fix-requires-private-for-wolfssl branch July 14, 2023 18:38
lampmanyao pushed a commit to lampmanyao/libssh2 that referenced this pull request Sep 3, 2023
- autotools was using `libwolfssl`. CMake left it empty. wolfSSL
  provides `wolfssl.pc`. This patch sets `Requires.private: wolfssl`
  with both build tools.

- add `libgcrypt` to `Requires.private` with both autotools and CMake.
  Ref:
    https://github.com/gpg/libgcrypt/blob/e76e88eef7811ada4c6e1d57520ba8c439139782/src/libgcrypt.pc.in
  Present since 2005-04-22:
    gpg/libgcrypt@32bf3f1
  Released in v1.3.0 2007-05-04:
    https://github.com/gpg/libgcrypt/releases/tag/libgcrypt-1.3.0

- also stop adding transitive `zlib` deps to `Requires.private`.
  The referenced crypto package is adding it as nedded.
  This makes deduplication of the list redundant, so stop doing it.
  Follow-up to 2fc3679

(`libssh2.pc` not tested as a project dependency.)

Closes libssh2#1123
agreppin pushed a commit to agreppin/libssh2 that referenced this pull request Jul 14, 2024
- autotools was using `libwolfssl`. CMake left it empty. wolfSSL
  provides `wolfssl.pc`. This patch sets `Requires.private: wolfssl`
  with both build tools.

- add `libgcrypt` to `Requires.private` with both autotools and CMake.
  Ref:
    https://github.com/gpg/libgcrypt/blob/e76e88eef7811ada4c6e1d57520ba8c439139782/src/libgcrypt.pc.in
  Present since 2005-04-22:
    gpg/libgcrypt@32bf3f1
  Released in v1.3.0 2007-05-04:
    https://github.com/gpg/libgcrypt/releases/tag/libgcrypt-1.3.0

- also stop adding transitive `zlib` deps to `Requires.private`.
  The referenced crypto package is adding it as nedded.
  This makes deduplication of the list redundant, so stop doing it.
  Follow-up to 2fc3679

(`libssh2.pc` not tested as a project dependency.)

Closes libssh2#1123
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant