idf-eclipse-plugin icon indicating copy to clipboard operation
idf-eclipse-plugin copied to clipboard

[WIP] IEP-897: FreeRTOS Task View

Open alirana01 opened this issue 2 years ago • 1 comments

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 ToolsUtility class. This change enhances the software's reliability and maintainability.
  • New Feature: Added the ability to install the freertos-gdb package, 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.

alirana01 avatar Oct 17 '23 10:10 alirana01

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 @coderabbitai in 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: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.
  • Additionally, enter @coderabbitai pause to halt the bot's ongoing reviews as any issue or review comment. To resume reviews, enter @coderabbitai resume as any issue or review comment.

coderabbitai[bot] avatar Oct 17 '23 10:10 coderabbitai[bot]