Skip to content

build(cmake): link to proper OpenSSL libraries#146

Merged
plexoos merged 1 commit into
mainfrom
fix-openssl-cmake
Sep 3, 2025
Merged

build(cmake): link to proper OpenSSL libraries#146
plexoos merged 1 commit into
mainfrom
fix-openssl-cmake

Conversation

@plexoos

@plexoos plexoos commented Sep 3, 2025

Copy link
Copy Markdown
Member

No description provided.

@plexoos plexoos requested a review from Copilot September 3, 2025 12:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CMake build configuration to properly link OpenSSL libraries using the modern CMake approach with imported targets instead of the legacy method.

  • Replaces direct ssl library linking with proper OpenSSL package discovery
  • Updates target linking to use CMake imported targets (OpenSSL::SSL and OpenSSL::Crypto)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@plexoos plexoos merged commit 7aa998c into main Sep 3, 2025
3 checks passed
@plexoos plexoos deleted the fix-openssl-cmake branch September 3, 2025 14:18
@plexoos plexoos added this to simphony Mar 13, 2026
@github-project-automation github-project-automation Bot moved this to Done in simphony Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants