Skip to content

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