Skip to content

fix(scons): prioritize RT-Thread FreeRTOS headers - #19

Open
CYFS3 wants to merge 1 commit into
RT-Thread-packages:rtt-devfrom
CYFS3:fix/scons-freertos-include-order
Open

CYFS3 wants to merge 1 commit into
RT-Thread-packages:rtt-devfrom
CYFS3:fix/scons-freertos-include-order

Conversation

@CYFS3

@CYFS3 CYFS3 commented Oct 23, 2026 •

Copy link
Copy Markdown

ESP-IDF 适配头通过 #include_next 引入 FreeRTOS-Wrapper,但原脚本只保证适配头位于搜索路径最前面,没有保证 Wrapper 和 BSP 配置、移植头优先于 IDF 原生头文件。软件包加载顺序变化时,可能误用 IDF 自带的 FreeRTOS 头文件,导致 configNUMBER_OF_CORES 等配置缺失,出现编译失败。

因此,在所有编译组注册完成后,统一确定头文件优先级:ESP 适配头 → BSP 配置和移植头 → FreeRTOS-Wrapper → IDF 原生头文件,消除对软件包加载顺序的依赖。路径从现有编译组获取,避免写死 BSP 位置或软件包版本目录。

@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 98ad21fa-fc11-4c6e-8e62-6a7d338ef618
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant