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.

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

Closed
wants to merge 1 commit into from
Closed

Made Ethernet useable on 'Esp32 Dev Module'. #107

wants to merge 1 commit into from

Conversation

reitowo
Copy link

@reitowo reitowo commented Aug 6, 2019

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

…EthernetServer because of the change in the Client and Server classes made by espressif.
@per1234
Copy link
Contributor

per1234 commented Aug 6, 2019

Similar (but not exactly the same) to #95
As I commented there:

It seems like it would be better for the ESP32 core developers to follow the standard Arduino core API, rather than forcing every library to be patched to deal with the breakage they caused.

Related: #88

Copy link
Contributor

@per1234 per1234 left a 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.)

@berguinsson
Copy link

Hi guys!
Only one quick question. What does "BSP" mean?
The user called cnSchwarzer said Espressif's BSP changes...
I hope you can help me.
Thank you!

@per1234
Copy link
Contributor

per1234 commented Jun 13, 2020

@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:
https://github.com/espressif/arduino-esp32
and more specifically, its core:
https://github.com/espressif/arduino-esp32/tree/master/cores/esp32
The officially recommended term is "platform", not "BSP".

@nmaas87
Copy link

nmaas87 commented Feb 20, 2021

Is there any updates on this? It is still failing builds on ESP32 :(
Thank you for your support :)!

@reitowo
Copy link
Author

reitowo commented Feb 20, 2021

Reasonable, it is esp's fault, not this library.

@reitowo reitowo closed this Feb 20, 2021
@nmaas87
Copy link

nmaas87 commented Feb 20, 2021

Thank you for the information.
While I can understand Espressif should fix the interface, it will probably hurt the overall experience and for the users look like the Arduino library is faulty.

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

sumnerboy12 added a commit to OXRS-IO/Ethernet that referenced this pull request Nov 5, 2021
@per1234 per1234 self-assigned this Sep 1, 2024
@per1234 per1234 added conclusion: declined Will not be worked on topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project labels Sep 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: declined Will not be worked on topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants