Skip to content

Conversation

@phoeagon
Copy link

PR 描述/PR description

添加限定ESP32的命令idf于tos.py中。此命令调用idf.py于构建环境中,以期方便手工调节设定参数

代码质量/Code Quality:

在本次拉取请求中,我已考虑以下事项 As part of this pull request, I've considered the following:

  • 确保代码注释和文档清晰,并使用英文注释以保证代码可读性。Ensure that the code comments and documentation are clear, and use English for comments to ensure code readability.
  • 确保文件头遵循文件头格式。Ensure that the file header follows the File Header Format.
  • 确保函数头遵循 Doxygen 格式。Ensure that function headers follow the Doxygen format as specified in Comments.
  • 已查阅 编码风格指南,并核查代码风格合规性,包括缩进、空格、命名规范及其他风格要求。 Reviewed the Coding Style Guide and verified code style compliance, including indentation, spacing, naming conventions, and other style guidelines.
  • 已使用代码格式化工具确保符合 TuyaOpen 编码规范。Have used the code-formatting source code formatting tool to ensure compliance with TuyaOpen coding standards.

本更改不包含任何C语言更改。所有更改存在于构建python脚本中。

此新功能仅限ESP32系列。
此工具对于目前issues中#436 https://github.com/tuya/TuyaOpen/issues/144等或有帮助

应用实例:
tos.py idf menuconfig
可以调用idf.py menuconfig以修改idf编译设定

此流程较以下显著方便:
TOP_DIR=/xxx/TuyaOpen/platform/ESP32
IDF_PATH=${TOP_DIR}/esp-idf
IDF_TOOLS_PATH=${TOP_DIR}/.espressif
export IDF_PATH
export IDF_TOOLS_PATH
. ${IDF_PATH}/export.sh
idf.py menuconfig

@adwuard adwuard changed the title 添加tos.py命令以调用idf feat: add tos.py cli command for build defs passing to idf SDK Dec 24, 2025
@ve2opn
Copy link

ve2opn commented Dec 26, 2025

On my system it throws error:
TypeError: execute_idf_commands() takes 3 positional arguments but 4 were given
Fix:
cli_idf.py

@phoeagon
Copy link
Author

Thanks! It looks like I manually tweaked the ESP folder (tools/util.py) that wasn't captured in the git workflow. Thanks for the fix! Applied.

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.

2 participants