[WIP] IEP-897: FreeRTOS Task View
Description
Initial work to dump the data from the freertos gdb python package to a simple text file. Will work on windows as the pip package depends on gdb with python support which has issues on mac and linux.
Fixes # (IEP-897)
Type of change
Please delete options that are not relevant.
- New feature (non-breaking change which adds functionality)
- This change requires a documentation update
How has this been tested?
WIP
Test Configuration:
- ESP-IDF Version: 5.1+
- OS (Windows):
Checklist
- [ ] PR Self Reviewed
- [ ] Applied Code formatting
- [ ] Added Documentation
- [ ] Added Unit Test
- [ ] Verified on all platforms - Windows,Linux and macOS
Summary by CodeRabbit
- Refactor: Streamlined the installation process by centralizing the configuration of environment variables, installation of pip packages, and copying of OpenOCD rules into the
ToolsUtilityclass. This change enhances the software's reliability and maintainability. - New Feature: Added the ability to install the
freertos-gdbpackage, improving the range of debugging tools available to users. - Usability: Updated user-facing messages and tooltips for better clarity during the installation process, enhancing the user experience.
- Cleanup: Removed unused code and imports, improving the software's efficiency and readability.
Walkthrough
The changes primarily involve refactoring and localization. Functionality related to environment configuration, OpenOCD rules copying, and pip package installation has been moved to ToolsUtility. Localization strings have been updated accordingly. Unused code has been removed.
Changes
| File | Summary |
|---|---|
.../core/tools/util/Messages.java |
New class Messages added for localization. |
.../core/tools/util/ToolsUtility.java |
New methods added for pip package installation, environment configuration, and OpenOCD rules copying. |
.../core/tools/util/messages.properties |
New localization strings added for OpenOCD rules copying. |
.../ui/tools/Messages.java |
Removed unused string constants. |
.../ui/tools/ToolsInstallationHandler.java |
Refactored to use ToolsUtility. Added installFreertosGdbPipPackage() method. |
.../ui/tools/messages.properties |
Removed unused localization strings. Added new tooltip. |
.../ui/update/InstallToolsHandler.java |
Refactored to use ToolsUtility. Removed unused imports and methods. |
.../ui/update/Messages.java |
Added new string constant for FreeRTOS GDB installation. |
.../ui/update/messages.properties |
Added localization string for FreeRTOS GDB installation. |
🐇
"In the land of code, where the shadows lie,
Refactoring reigns, under the cloudless sky.
With each pull request, we make our stand,
Crafting art with a keyboard in hand.
Localization strings, in their rightful place,
Give our software a friendly face.
So here's to the changes, big and small,
In the end, they matter to us all." 🎉
Tips
Chat with CodeRabbit Bot (@coderabbitai)
- Mention
@coderabbitaiin any review comment for bot assistance. - Note: Review comments are made on code diffs or files, not on the PR overview.
Pause Incremental Reviews
- Insert
@coderabbitai: ignorein the PR description to halt the bot's ongoing reviews. Remove the line to resume. - Additionally, enter
@coderabbitai pauseto halt the bot's ongoing reviews as any issue or review comment. To resume reviews, enter@coderabbitai resumeas any issue or review comment.