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.

Sign up for GitHub

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

Jump to bottom

[BUG] python: No module named esptool #27060

Closed
lboue opened this issue Jun 5, 2023 · 4 comments · Fixed by #27141
Closed

[BUG] python: No module named esptool #27060

lboue opened this issue Jun 5, 2023 · 4 comments · Fixed by #27141
Labels
esp32

Comments

@lboue
Copy link
Contributor

lboue commented Jun 5, 2023

Reproduction steps

Build bridge-app esp32

git clone https://github.com/project-chip/connectedhomeip.git
cd connectedhomeip
source scripts/bootstrap.sh
source scripts/activate.sh

cd ~/connectedhomeip/examples/bridge-app/esp32
gn gen out/debug
ninja -C out/debug
idf.py set-target esp32c3
idf.py build

Bug prevalence

always

GitHub hash of the SDK that was being used

275341e

Platform

esp32

Platform Version(s)

N/A

Anything else?

Do you know why esptool is not installed by bootstrap.sh script ?

I had to run that command:
pip install esptool

Log:
esptool-error.log
@bzbarsky-apple
Copy link
Contributor

@lboue I don't see you doing the export from the esp stuff anywhere in your commands above. Are you doing it somewhere?

@bzbarsky-apple
Copy link
Contributor

And in particular, the steps above don't look very much like what https://github.com/project-chip/connectedhomeip/blob/master/docs/guides/esp32/build_app_and_commission.md has listed.

@shubhamdp
Copy link
Contributor

@lboue What is the IDF version? In IDF v4.4.x esptool.py is part of repository, but from IDF v5.x and onwards it was moved out as an installable package. Since, docs still recommends using v4.4.x we have not added it to scripts/setup/requirements.esp32.txt.

This should have been noted in the docs, will fix it.

@shubhamdp shubhamdp mentioned this issue Jun 8, 2023
Merged
@lboue
Copy link
Contributor Author

lboue commented Jun 8, 2023

Thanks. It will be fixed by PR #27141.

@lboue lboue closed this as completed Jun 8, 2023
kpschoedel added a commit to kpschoedel/connectedhomeip that referenced this issue Nov 23, 2023
@kpschoedel
[flashing scripts] [ESP32] Fix running esptool.py
b6c260c 
In Espressif IDF 5.x, esptool.py changed from a script under $IDF_PATH to an
installable Python package (project-chip#27060). This change udpates the flashing script
accordingly.
@kpschoedel kpschoedel mentioned this issue Nov 23, 2023
Merged
andy31415 pushed a commit that referenced this issue Nov 23, 2023
@kpschoedel
[flashing scripts] [ESP32] Fix running esptool.py (#30627)
89e0bde 
In Espressif IDF 5.x, esptool.py changed from a script under $IDF_PATH to an
installable Python package (#27060). This change udpates the flashing script
accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
esp32
Projects
None yet
Milestone
No milestone
3 participants
@lboue @shubhamdp @bzbarsky-apple