-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Tools: don't add the esptool directory with wrappers to the PATH
Esptool.py is available from the esptool package which is installed in every ESP-IDF v5.0+ shells. Therefore, the path to the legacy wrapper scripts is not needed to be in the PATH. In some cases like #10926 it even cases problem because the wrapper doesn't have a shebang line and is not executable. Note that the wrapper is kept only for invoking esptool as "python $IDF_PATH/..../esptool.py" which was previously used mostly in CMake files. There should be no other use-case where the esptool wrappers are still needed.
- Loading branch information
1 parent
3050ea6
commit c7a70b6
Showing
3 changed files
with
1 addition
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters