Skip to content

cmake: raise minimum required to 3.5#795

Merged
jforissier merged 1 commit into
OP-TEE:masterfrom
sudeep-holla:cmake_4.0_fix
May 5, 2025
Merged

cmake: raise minimum required to 3.5#795
jforissier merged 1 commit into
OP-TEE:masterfrom
sudeep-holla:cmake_4.0_fix

Conversation

@sudeep-holla
Copy link
Copy Markdown
Contributor

CMake 4.0 was released [1] which dropped compatibility with versions older than 3.5 and causes cmake_minimum_required(3.4) to fail. Bump the minimum required version to 3.5, which was released 9 years ago.

[1] https://www.kitware.com/cmake-4-0-0-available-for-download/

With v4.0.0 the following build error is observed:

| CMake Error at CMakeLists.txt:1 (cmake_minimum_required):
| Compatibility with CMake < 3.5 has been removed from CMake.
|
| Update the VERSION argument value. Or, use the ... syntax
| to tell CMake that the project requires at least but has been updated
| to work with policies introduced by or earlier.
|
| Or, add -DCMAKE_POLICY_VERSION_MINIMUM=3.5 to try configuring anyway.

Copy link
Copy Markdown
Contributor

@jforissier jforissier left a comment

Choose a reason for hiding this comment

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

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

@jforissier
Copy link
Copy Markdown
Contributor

s/mimimum/minimum/ in the subject, though 😉

@jenswikl
Copy link
Copy Markdown
Contributor

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

@etienne-lms
Copy link
Copy Markdown
Contributor

Acked-by: Etienne Carriere <etienne.carriere@foss.st.com>

CMake 4.0 was released [1] which dropped compatibility with versions
older than 3.5 and causes cmake_minimum_required(3.4) to fail.
Bump the minimum required version to 3.5, which was released 9 years
ago.

[1] https://www.kitware.com/cmake-4-0-0-available-for-download/

With v4.0.0 the following build error is observed:

  |  CMake Error at CMakeLists.txt:1 (cmake_minimum_required):
  |  Compatibility with CMake < 3.5 has been removed from CMake.
  |
  |  Update the VERSION argument <min> value.  Or, use the <min>...<max> syntax
  |  to tell CMake that the project requires at least <min> but has been updated
  |  to work with policies introduced by <max> or earlier.
  |
  |  Or, add -DCMAKE_POLICY_VERSION_MINIMUM=3.5 to try configuring anyway.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@foss.st.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
@sudeep-holla sudeep-holla changed the title cmake: raise mimimum required to 3.5 cmake: raise minimum required to 3.5 May 2, 2025
@jforissier
Copy link
Copy Markdown
Contributor

OP-TEE/manifest#324 should address the CI failure.

@jforissier jforissier merged commit 6897e9c into OP-TEE:master May 5, 2025
1 of 6 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.

4 participants