-
Notifications
You must be signed in to change notification settings - Fork 306
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
Switch ESP-IDF versions #1173
Switch ESP-IDF versions #1173
Conversation
Download the artifacts for this pull request: |
@brianignacio5 hi ! Tested under: there is no command "ESP-IDF: Select Current ESP-IDF Version" |
I've tested on Windows and indeed there is. Can you provide troubleshooting information @AndriiFilippov ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Tested on Linux Fedora WS 39
@brianignacio5 hi ! Great, now I see only actual ESP-IDF versions. Because I have installed some IDF using Eclipse IDE(v4.4.7) and cmd(master) -> restart VS Code -> can't see those IDFs in the list. |
This only shown the ESP-IDF setups previously configured with EDIT: Actually it will try to populate all ESP-IDF setups specified in |
Looks great |
@brianignacio5 hi ! Tested under: everything works fine but current version does not get updated in Status Bar during version switch |
4fc0873
to
8692fb6
Compare
@brianignacio5 hi ! LGTM 👍 |
Hi @brianignacio5, I've re-tested on Linux Fedora WS 39, it seems that there is a bug if you have IDF Tools installed in the same folder (ex: .espressif): When you use command Steps to reproduce:
I've also notice one scenario that can be a bit confusing for some users. For example: They have worskpace folder as their scope and have the settings set for v4.4.7. (In global settings they use v5.2.1) They create a new project using the I'm not sure if there is a better way than documentation to inform the users about this scenario or about how switching the versions work. Maybe there is a way to analyze the project (at least for esp-idf example projects) to see if it's compatible with the esp-idf version they are using? What do you think? |
Hi @radurentea I fixed this status bar update in the last commit. With regards to ESP-IDF examples, this issue was already present before. Thing is that examples are loaded using IDF_PATH setting and when you read them the workspace folder always have preference on top of global User settings. We could
|
Description
Add ESP-IDF version switcher.
Fixes #1095
Type of change
Steps to test this pull request
Provide a list of steps to test changes in this PR and required output
.vscode/settings.json
Expected behaviour:
New selected version is applied to local workspace folder.
Expected output:
How has this been tested?
Test Configuration:
Checklist