micropython / micropython-esp32

Old port of MicroPython to the ESP32 -- new port is at https://github.com/micropython/micropython
MIT License
675 stars 217 forks source link

Toolchain setup in ports/esp32/README is out of date #237

Closed aykevl closed 5 years ago

aykevl commented 6 years ago

The README for the esp32 gives this text as part of "Setting up the toolchain and ESP-IDF":

Follow the guide "Setting Up ESP-IDF", for Windows, Mac or Linux. You only need to perform up to "Step 2" of the guide, by which stage you should have installed the cross-compile and cloned the ESP-IDF repository.

Apparently they expect the guide to cover cloning of the repository, but it only gives instructions for how to download a binary toolchain from Espressif. I worked around it by also downloading the esp-idf repository, checking out the right hash and doing a submodule update (without compiling), but I'm not sure whether this is the correct approach. Now I have set ESPIDF to the esp-idf repository and have added the binary toolchain from Espressif to my PATH variable. This works, mostly.

dpgeorge commented 5 years ago

The README was improved upstream in commit https://github.com/micropython/micropython/commit/f16c775a0782961a841f77cc48a789b3246d7ef7