plerup / makeEspArduino

A makefile for ESP8266 and ESP32 Arduino projects
GNU Lesser General Public License v2.1
513 stars 131 forks source link

Add support for Arduino Core 3.0.0 #200

Closed everslick closed 11 months ago

everslick commented 11 months ago

It's this time of year when major changes in the Arduino Core break makeEspArduino. But maybe this needs only a small fix. I believe the build breaks because the IDF headers are now in tools/esp32-arduino-libs/esp32/ instead of tools/sdk/esp32/. But maybe the linking stage needs a fix too. I didn't come that far. ;-)

everslick commented 11 months ago

~changing SDK_ROOT to SDK_ROOT = $(ESP_ROOT)/tools/esp32-arduino-libs gets me further:~

SDK_ROOT is not used anywhere, so my initial build issues have been unrelated (i.e. old object files in the build dir). Anyway...

Arduino-ESP32-3.0.0/cores/esp32/chip-debug-report.cpp:212:67: error: expected primary-expression before ')' token
  212 |   chip_report_printf("  Compile Host OS   : %s\n", ARDUINO_HOST_OS);

Arduino-ESP32-3.0.0/cores/esp32/chip-debug-report.cpp:224:64: error: expected primary-expression before ')' token
  224 |   chip_report_printf("  Arduino FQBN      : %s\n", ARDUINO_FQBN);

I guess it's missing defines for ARDUINO_HOST_OS and ARDUINO_FQBN.

everslick commented 11 months ago

Adding:

DEFINES += -DARDUINO_HOST_OS=\"$(OS)\" -DARDUINO_FQBN=\"generic\"

helps, but should probably done in makeEspArduino.mk.

plerup commented 11 months ago

Thanks for reporting. Fix coming tonight.

plerup commented 11 months ago

Fixed in latest commit

everslick commented 11 months ago

Thanks!