Closed sigmaaa closed 5 months ago
The updates pertain to the Espressif IDF Eclipse plugin, with a focus on enhancing dependency management, streamlining the build configuration process, and refining the DFU (Device Firmware Update) command utility. These refinements include enforcing a version requirement for a dependent bundle, cleaning up the code by removing unused imports, reorganizing code for clarity, and altering how build and flash commands are handled.
File Path | Change Summary |
---|---|
.../MANIFEST.MF .../DfuCommandsUtil.java |
Added version requirement for org.eclipse.cdt.core bundle.Code cleanup, logic adjustments, and method refactoring in DFU-related classes. |
.../IDFBuildConfiguration.java |
Code cleanup, reordering, and build process adjustments. |
πβ¨ In the realm of code, we hop and refine, Cleaning up, enhancing, making it shine. With each careful tweak and artful adjust, We craft the bytes that earn your trust. π
(\(\ ( -.-) o_(")(")
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@sigmaaa hi !
Tested under: OS - Windows 10 ESP-IDF: v5.1.2
1) Build -> Stop -> Build -> Flash -> Monitor π 2) Build -> Stop -> Close Project -> Open Project -> Build -> Flash -> Monitor π 3) Build -> Stop -> Close Eclipse -> Open Eclipse -> Build -> Flash -> Monitor π 4) Build -> Stop -> Restart -> Build -> Flash - Monitor π 5) select DFU -> Build -> Stop -> Build -> Flash - Monitor π 6) Custom Build Folder -> Build -> Stop -> Build -> Flash - Monitor π 7) New Project -> click "Run" -> Stop -> can't stop build process β
Hi @AndriiFilippov, It was fixed with the last commit. @alirana01, @kolipakakondal PTAL at the last commit. I had to add a workaround and custom handler, which terminates the build process, which is part of the Launching job. To avoid handler conflicts, I created a context with id com.espressif.idf.ui.espLaunchScope and added a condition for activating a custom handler.
@sigmaaa hi !
Retested all scenarios. LGTM π
Description
Pressing Stop now terminates building processes. Also did a small refactoring, which improves build time for DFU.
Fixes # (IEP-1026)
Type of change
Please delete options that are not relevant.
How has this been tested?
Test 1: build -> Stop Test 2: select DFU -> Build -> Stop
Test Configuration:
Dependent components impacted by this PR:
Checklist
Summary by CodeRabbit
Refactor
Documentation