Skip to content

Update minimum required CMake version#181

Merged
bjosv merged 2 commits intovalkey-io:mainfrom
bjosv:ci-verify-min-cmake-version
Mar 21, 2025
Merged

Update minimum required CMake version#181
bjosv merged 2 commits intovalkey-io:mainfrom
bjosv:ci-verify-min-cmake-version

Conversation

@bjosv
Copy link
Copy Markdown
Collaborator

@bjosv bjosv commented Mar 20, 2025

CMake 3.7.0 (released Nov. 2016) is required to get FIND_PACKAGE(OpenSSL REQUIRED)
to successfully generate the needed target OpenSSL::SSL.

This PR also adds a CI job to test that the minimum required CMake version can be used.
This showed that building with CMake 3.0 - 3.6 failed.

As reference Ubuntu 18.04 ships with CMake 3.10.

bjosv added 2 commits March 20, 2025 11:52
Adds a CI job to test that the minimum required CMake version
can be used. The CMakeLists.txt currently contains:
CMAKE_MINIMUM_REQUIRED(VERSION 3.0.0)

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
CMake 3.7.0 (released Nov. 2016) is required to get
  FIND_PACKAGE(OpenSSL REQUIRED)
to successfully generate the needed target `OpenSSL::SSL`.

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
@bjosv bjosv requested a review from michael-grunder March 20, 2025 12:15
@bjosv bjosv merged commit 517e626 into valkey-io:main Mar 21, 2025
44 checks passed
@bjosv bjosv deleted the ci-verify-min-cmake-version branch March 21, 2025 14:55
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.

2 participants