Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PlatformIO unified debugger does not properly escape tool installation process into MI2 #4565

Closed
elmot opened this issue Mar 6, 2023 · 1 comment

Comments

@elmot
Copy link
Contributor

elmot commented Mar 6, 2023

[x] PlatformIO Core.

Configuration

Operating system: Win10

PlatformIO Version (platformio --version): 6.1.6

Description of problem

When pio debug --interface=gdb --interpreter=mi2 --iex "set mi-async on" is run and a required tool is not installed,
pio downloads and installs the tool, and reports the progress to debuggers stdout. Those messages are not properly escaped for MI2

Setting PLATFORMIO_DISABLE_PROGRESSBAR=true does not help.

Steps to Reproduce

  1. Create a project for STM Nucleo-L476 board and arduino framework (most probably any framework and any STM32 board will have same effect)
  2. ensure that ~/.platformio/packages/tool-openocd does not exist
  3. Run debugger as pio debug --interface=gdb --interpreter=mi2 --iex "set mi-async on"

Actual Results

Unescaped download reports are printed to stdout:

Tool Manager: Installing platformio/tool-openocd @ ~2.1100.0
Unpacking...
Tool Manager: tool-openocd@2.1100.211028 has been installed!

Expected Results

The message is either escaped according to MI2 format, or printed to stderr, or not printed at all

@ivankravets
Copy link
Member

Thanks for the report! Please re-test with pio upgrade --dev.

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

No branches or pull requests

2 participants