-
Notifications
You must be signed in to change notification settings - Fork 266
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.
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
Made Ethernet useable on 'Esp32 Dev Module'. #107
Conversation
…EthernetServer because of the change in the Client and Server classes made by espressif.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The need to have special declarations of EthernetClient::connect() for ESP32 was removed by espressif/arduino-esp32#3191.
Unfortunately, the situation with EthernetServer::begin() remains.
(Sorry for repeat comment. I realized this is more appropriate as a review than a comment.)
Hi guys! |
@berguinsson BSP is an acronym for "board support package". This is the term occasionally used to refer to the collection of software required to add support for specific boards to the Arduino development software. In this case, it's: |
Is there any updates on this? It is still failing builds on ESP32 :( |
Reasonable, it is esp's fault, not this library. |
Thank you for the information. For those who come after me, because they badly need to use the sheer amount of libraries that build upon the Arduino Ethernet (especially MQTT, Webserver and Websocket) and now stand before a failing build: You'll need to frankenstein the src/Ethernet.h as shown in the pull request and it will make your other libaries work again |
Espressif's BSP changes the Client class and Server class's definition. When compiling on 'Esp32 Dev Module, we will receive messages indicating there are some pure virtual functions which are not implemented.
Adding these override functions will solve the following issues:
#97
#106
espressif/arduino-esp32#2786