Skip to content

fix for vcpkg build#495

Merged
dgarske merged 2 commits intowolfSSL:masterfrom
JacobBarthelmeh:cmake_build
Apr 28, 2026
Merged

fix for vcpkg build#495
dgarske merged 2 commits intowolfSSL:masterfrom
JacobBarthelmeh:cmake_build

Conversation

@JacobBarthelmeh
Copy link
Copy Markdown
Contributor

This is the patch used to get vcpkg building (microsoft/vcpkg#51406).

@JacobBarthelmeh JacobBarthelmeh self-assigned this Apr 27, 2026
Copilot AI review requested due to automatic review settings April 27, 2026 16:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the CMake build/install logic to support vcpkg builds by avoiding hard dependence on pkg-config and ensuring exported targets can be installed successfully.

Changes:

  • Guarded pkg_check_modules() behind PkgConfig_FOUND to prevent configure-time failure when PkgConfig isn’t available.
  • Installed/exported wolftpm_wolfssl_dep alongside wolftpm to satisfy export-set dependencies during install.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CMakeLists.txt Outdated
Comment thread CMakeLists.txt
Comment thread CMakeLists.txt
Copy link
Copy Markdown

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fenrir Automated Review — PR #495

No scan targets match the changed files in this PR. Review skipped.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/cmake-build.yml
Comment thread .github/workflows/cmake-build.yml
Comment thread CMakeLists.txt
@dgarske dgarske merged commit ebaeb7a into wolfSSL:master Apr 28, 2026
128 checks passed
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.

5 participants