Skip to content

Cmake: add defconfig preprocess capability in Cmake build environment(recursively expand #include) #16897

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

Merged
merged 1 commit into from
Aug 25, 2025

Conversation

wangchdo
Copy link
Contributor

Closed the previous PR and opened this new PR to minimize the Cmake scripts change.....

Add:
cmake/nuttx_process_config.cmake
tools/process_config.py

Update nuttx/CMakeLists.txt to call process_config defined in nuttx_process_config.cmake to process defconfig before actually using it

Note: Please adhere to Contributing Guidelines.

Summary

Add deconfig "#include" expand capability in Cmake build system

Impact

Cmake build system will have the deconfig "#include" expand capability, which is also cross-platform

Testing

Split defconfig into two files:
image

Cmake Expand defconfig and generate .defconfig.processed:

image

@github-actions github-actions bot added Area: Tooling Area: Build system Size: M The size of the change in this PR is medium labels Aug 25, 2025
@wangchdo wangchdo force-pushed the add_cmake_defconfig_process branch 2 times, most recently from 420e20e to 58d84ea Compare August 25, 2025 01:05
@wangchdo wangchdo force-pushed the add_cmake_defconfig_process branch from c601f1b to badc50d Compare August 25, 2025 05:19
@wangchdo wangchdo force-pushed the add_cmake_defconfig_process branch 2 times, most recently from 16b9a9f to 26d68e0 Compare August 25, 2025 07:13
…(recursively expand #include)

Add:
    cmake/nuttx_process_config.cmake
    tools/process_config.py

    Update nuttx/CMakeLists.txt to call process_config defined ini
    nuttx_process_config.cmake to process defconfig before actually using it
@wangchdo wangchdo force-pushed the add_cmake_defconfig_process branch from 26d68e0 to 0aa63c2 Compare August 25, 2025 07:18
@xiaoxiang781216 xiaoxiang781216 merged commit be36d42 into apache:master Aug 25, 2025
39 checks passed
@wangchdo wangchdo deleted the add_cmake_defconfig_process branch August 26, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Build system Area: Tooling Size: M The size of the change in this PR is medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants