toptobes commented on code in PR #582: URL: https://github.com/apache/cassandra-cpp-driver/pull/582#discussion_r3393125948
########## CMakeLists.txt: ########## @@ -1,4 +1,4 @@ -cmake_minimum_required(VERSION 3.10) +cmake_minimum_required(VERSION 3.5) Review Comment: Looking at the git logs, I'm fairly certain what happened is that they were originally basing this off the version of the c++ driver 2 weeks before this PR was created, when it was still using cmake version `2.8.12` (so this was intended as an upgrade, not a downgrade) See: ```rb ❯ git lg * a22a11a3 golddevelopment - (HEAD -> gold-development/trunk) add support for appleclang and update cmake (5 months ago) * e5a486ac Sigmanificient - Bump minimum required version for cmake4 compatibility (5 months ago) ``` ```rb ❯ git log -L 1,1:CMakeLists.txt commit a22a11a37c3d942bf249ef0501252b27d371df67 (HEAD -> gold-development/trunk) Author: golddevelopment <[email protected]> Date: Wed Jan 14 12:30:43 2026 +0100 add support for appleclang and update cmake diff --git a/CMakeLists.txt b/CMakeLists.txt --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1,1 +1,1 @@ -cmake_minimum_required(VERSION 3.10) +cmake_minimum_required(VERSION 3.5) commit e5a486ac52b99ae74d40e285b4438252fcf3b07a Author: Sigmanificient <[email protected]> Date: Wed Dec 31 14:20:25 2025 +0100 Bump minimum required version for cmake4 compatibility patch by Sigmanificient; reviewed by Bret McGuire and Yifan Cai reference: https://github.com/apache/cassandra-cpp-driver/pull/580 diff --git a/CMakeLists.txt b/CMakeLists.txt --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1,1 +1,1 @@ -cmake_minimum_required(VERSION 2.8.12) +cmake_minimum_required(VERSION 3.10) ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
