Skip to content

Bump the minimum version of CMake to 3.10, to be able to compile with CMake 4.0#14

Merged
jmcarcell merged 2 commits into
iLCSoft:masterfrom
jmcarcell:cmake-4
May 19, 2026
Merged

Bump the minimum version of CMake to 3.10, to be able to compile with CMake 4.0#14
jmcarcell merged 2 commits into
iLCSoft:masterfrom
jmcarcell:cmake-4

Conversation

@jmcarcell
Copy link
Copy Markdown
Contributor

@jmcarcell jmcarcell commented Mar 31, 2025

BEGINRELEASENOTES

  • Bump the minimum version of CMake to 3.10, to be able to compile with CMake 4.0
  • Don't set CMP0005 to OLD since it has to be set to NEW with CMake 4.0 and onwards. Escape
    the DLCCD_DB_INIT_DEFAULT definition accordingly

ENDRELEASENOTES

See https://cmake.org/cmake/help/latest/release/4.0.html#deprecated-and-removed-features

Here I'm not sure what's best. I added the following:

get_directory_property(COMPILE_DEFINITIONS COMPILE_DEFINITIONS)
message(WARNING "Current definitions: ${COMPILE_DEFINITIONS}")

after DLCCD_DB_INIT_DEFAULT is set and I get an empty string in the current version, while with this PR it will become

Current definitions:
LCCD_DB_INIT_DEFAULT="localhost:lccd_test:calvin:hobbes"

I see this is only used in https://github.com/iLCSoft/LCCD/blob/master/source/src/lccd.cc#L44. One possibility would be to remove the definition and just return an empty string, which is the current behaviour.

@jmcarcell jmcarcell changed the title Bump the minimum version of CMake to 3.5, to be able to compile with CMake 4.0 Bump the minimum version of CMake to 3.10, to be able to compile with CMake 4.0 May 19, 2026
@jmcarcell
Copy link
Copy Markdown
Contributor Author

Let's give this a try.

@jmcarcell jmcarcell merged commit 2b9070b into iLCSoft:master May 19, 2026
4 of 5 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.

1 participant