Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Fix build when using latest arduino-esp32 master due to IDF update #999

Merged
merged 2 commits into from
Jul 26, 2021

Conversation

blackhack
Copy link
Contributor

Latest idf update on arduino-esp32 (espressif/arduino-esp32@a618fc1) causes a linker failure.

This change was also applied to the original arduino-esp32 webserver.


Linking everything together...
d:/david/development/arduino/sketch/hardware/espressif/esp32/tools/xtensa-esp32-elf/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: C:\Users\david\AppData\Local\Temp\arduino_build_567038\libraries\ESPAsyncWebServer\WebAuthentication.cpp.o:(.literal._ZL6getMD5PhtPc+0x4): undefined reference to `mbedtls_md5_starts'
d:/david/development/arduino/sketch/hardware/espressif/esp32/tools/xtensa-esp32-elf/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: C:\Users\david\AppData\Local\Temp\arduino_build_567038\libraries\ESPAsyncWebServer\WebAuthentication.cpp.o: in function `getMD5(unsigned char*, unsigned short, char*)':
D:\david\Development\Arduino\Sketch\libraries\ESPAsyncWebServer\src/WebAuthentication.cpp:73: undefined reference to `mbedtls_md5_starts'
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board ESP32 Dev Module.

@dominikawysocka
Copy link

Save my day! :) Merge, working with ArduinoIDE with ESP32 Dev Module board 👍

@1e1
Copy link

1e1 commented Jul 2, 2021

Great fix!
I guess there is a similar issue for ESP8266.
As soon as the sketch uses onSslFileRequest, the compilation fails:

collect2: error: ld returned 1 exit status
exit status 1
Error compiling for board Generic ESP8266 Module.

@CelliesProjects
Copy link

CelliesProjects commented Jul 26, 2021

Hit my head on this issue too: espressif/arduino-esp32/issues/5457

@blackhack Is there any chance you get this merged in https://github.com/ul-gh/ESPAsyncWebServer/tree/dev ?
I have not seen a single merge by me-no-dev in this repo for a very long time.

@CelliesProjects
Copy link

Spoke too soon.
@me-no-dev Thanks!

DmytroKorniienko pushed a commit to DmytroKorniienko/ESPAsyncWebServer that referenced this pull request Oct 1, 2021
…e-no-dev#999)

* Fix build when using latest arduino-esp32 master due to IDF update
espressif/arduino-esp32@a618fc1

* Fix build when using WebSockets
flofricke added a commit to flofricke/ESPAsyncWebServer that referenced this pull request Dec 27, 2021
* Fix authentication for callback web handlers

* URL template match by extension (me-no-dev#751)

Changes to be committed:
modified:   src/WebHandlerImpl.h

* Redefine DYNAMIC_JSON_DOCUMENT_SIZE if not defined (me-no-dev#922)

* Fix build when using latest arduino-esp32 master due to IDF update (me-no-dev#999)

* Fix build when using latest arduino-esp32 master due to IDF update
espressif/arduino-esp32@a618fc1

* Fix build when using WebSockets

* Stop PIO CI

Co-authored-by: Me No Dev <[email protected]>
Co-authored-by: Viktr <[email protected]>
Co-authored-by: Pablo Clemente Maseda <[email protected]>
Co-authored-by: JDavid <[email protected]>
luciadarsa pushed a commit to luciadarsa/ESPAsyncWebServer that referenced this pull request Sep 19, 2022
…e-no-dev#999)

* Fix build when using latest arduino-esp32 master due to IDF update
espressif/arduino-esp32@a618fc1

* Fix build when using WebSockets
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants