Skip to content

docs: refresh Tab5 README build guidance#142

Merged
baba-dev merged 1 commit into
mainfrom
codex/update-readme-with-new-prerequisites-and-steps
Sep 20, 2025
Merged

docs: refresh Tab5 README build guidance#142
baba-dev merged 1 commit into
mainfrom
codex/update-readme-with-new-prerequisites-and-steps

Conversation

@baba-dev

Copy link
Copy Markdown
Owner

Summary

  • document ESP-IDF v5.4.2 as the tested toolchain for ESP32-P4 builds
  • update the quickstart flow to use platforms/tab5 with the latest idf.py build commands
  • remove outdated devcontainer guidance and add golden snapshot maintenance notes

Testing

  • not run (docs-only change)

https://chatgpt.com/codex/tasks/task_e_68ce5d9892cc83249ef3ed10cdc67b4f

@baba-dev baba-dev merged commit a2d8284 into main Sep 20, 2025
1 of 6 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment thread README.md
Comment on lines +127 to +131
1. **Clone the repo**

git clone https://github.com/baba-dev/M5Tab5-UserDemo.git
cd M5Tab5-UserDemo
python fetch_repos.py # fetch or update M5 & LVGL components
cd M5Tab5-UserDemo

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[P1] Restore dependency fetch step before building Tab5 firmware

The new quickstart flow no longer fetches the external components that the Tab5 build requires. Following these steps from a fresh clone leaves dependencies/ empty, but platforms/tab5/main/CMakeLists.txt declares mooncake, mooncake_log, and smooth_ui_toolkit in REQUIRES, so idf.py build will fail with missing-component errors. Please reintroduce a step (e.g., python fetch_repos.py) or document another way to populate those dependencies before invoking the build.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant