espressif / vscode-esp-idf-extension

Visual Studio Code extension for ESP-IDF projects
Apache License 2.0
1.03k stars 300 forks source link

✨ Hoist linker (ld) errors to VS Code 'Problems' window #1317

Open GillesZunino opened 5 days ago

GillesZunino commented 5 days ago

Description

When building a project in VS Code ESP IDF Extension (v 1.8.1), any linker error is reported in the terminal. These errors are not reported in the 'Problems' list of VS Code. Users have to sift through the build log and extract linker errors. This can also confuse new users as ESP IDF will say "build failed" and the problems list will be empty.

image

Fixes #1315

Type of change

Please delete options that are not relevant.

Steps to test this pull request

Provide a list of steps to test changes in this PR and required output

  1. Open any ESP-IDF project in VS Code,
  2. Ensure there is at least one or more linker (ld) errors. A simple way to create an ld error is to rename 'app_main' to 'app_mainXXX',
  3. Build project in VS Code by clicking the wrench icon (Build).

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Test Configuration:

Dependent components impacted by this PR:

N/A

Checklist