Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmake: log ${CMAKE_VERSION} #53286

Merged
merged 1 commit into from
Dec 22, 2022

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Dec 22, 2022

Different CMake versions can have very subtle differences, for instance CMake 3.21 links object files in a different order compared to CMake 3.20; this produces different binaries.

CMAKE_VERSION is required information to track binary differences between two build systems.

Signed-off-by: Marc Herbert [email protected]

Different CMake versions can have very subtle differences, for
instance CMake 3.21 links object files in a different order compared
to CMake 3.20; this produces different binaries.

CMAKE_VERSION is required information to track binary differences
between two build systems.

Signed-off-by: Marc Herbert <[email protected]>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Dec 22, 2022

Sample output:

-- west build: generating a build system
Loading Zephyr default modules (Zephyr base).
-- Application: /home/me/SOF/sof/app
-- CMake version: 3.25.1
-- Found Python3: /usr/bin/python3.10 (found suitable exact version "3.10.8") found components: Interpreter 
-- Cache files will be written to: /home/me/.cache/zephyr
-- Zephyr version: 3.2.99 (/home/me/SOF/zephyr)
-- Found west (found suitable version "0.14.0", minimum required is "0.7.1")
-- Board: intel_adsp_cavs25
-- Found toolchain: xcc (/home/me/XCC/install/tools)
-- Found Dtc: /usr/bin/dtc (found suitable version "1.6.1", minimum required is "1.4.6") 

@carlescufi carlescufi merged commit 3ec1bb2 into zephyrproject-rtos:main Dec 22, 2022
@marc-hb marc-hb deleted the cmake-version-log branch December 22, 2022 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants