Skip to content

Conversation

xuxin930
Copy link
Contributor

@xuxin930 xuxin930 commented Aug 22, 2025

Summary

  1. sync defconfig preprocess for CMake build
  2. add APPDIR for default preprocess search path

Impact

this allow defconfig use #include such as makefile

Testing

CI testing

Signed-off-by: xuxin19 <xuxin19@xiaomi.com>
this allows defconfig #include to be freely extensible

Signed-off-by: xuxin19 <xuxin19@xiaomi.com>
@xuxin930 xuxin930 marked this pull request as ready for review August 22, 2025 10:00
# Preprocess DEFCONFIG if necessary
include(nuttx_preprocess_defconfig)

# Generate inital .config ###################################################
Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix this typo:

Used config files:
1: .codespellrc
/home/runner/work/nuttx/nuttx/nuttx/CMakeLists.txt:164: inital ==> initial

@wangchdo
Copy link
Contributor

wangchdo commented Aug 25, 2025

I would recomend to process defconfig with python scripts, which is cross-platform, please refer to my new PR: #16897

I closed my previous PR #16891, since it changed the Cmake scripts too much.

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: S The size of the change in this PR is small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants